From 14c60c5d9a0f07eecec4518275b12efd30ce24af Mon Sep 17 00:00:00 2001
From: Will Rossiter <will.rossiter@gmail.com>
Date: Mon, 25 Apr 2011 22:44:54 +1200
Subject: [PATCH] Landing pull request 17. Fixes Call to a member function
 setValue() on a non-object in ConfirmedPasswordField Fixes #????.

---
 forms/ConfirmedPasswordField.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/forms/ConfirmedPasswordField.php b/forms/ConfirmedPasswordField.php
index e376c6dbb..0da16d93b 100644
--- a/forms/ConfirmedPasswordField.php
+++ b/forms/ConfirmedPasswordField.php
@@ -186,7 +186,7 @@ class ConfirmedPasswordField extends FormField {
 			if($value['_Password'] || (!$value['_Password'] && !$this->canBeEmpty)) {
 				$this->value = $value['_Password'];
 			}
-			if(isset($value['_PasswordFieldVisible'])){
+			if($this->showOnClick && isset($value['_PasswordFieldVisible'])){
 				$this->children->fieldByName($this->Name() . '[_PasswordFieldVisible]')->setValue($value['_PasswordFieldVisible']);
 			}
 		} else {