From 7429387aab07f410a5eee759d8c13415e0c4b2ba Mon Sep 17 00:00:00 2001 From: Sam Minnee Date: Tue, 26 Feb 2008 01:42:29 +0000 Subject: [PATCH] Fixed password emailing for edited members git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.2.2@50193 467b73ca-7a2a-4603-9d3b-597d59a354a9 --- security/Member.php | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/security/Member.php b/security/Member.php index abd125e40..2b39e23f9 100644 --- a/security/Member.php +++ b/security/Member.php @@ -413,16 +413,7 @@ class Member extends DataObject { * found update this record to merge with that member. */ function onBeforeWrite() { - if(isset($this->changed['Password']) && $this->changed['Password']) { - // Password was changed: encrypt the password according the settings - $encryption_details = Security::encrypt_password($this->Password); - $this->Password = $encryption_details['password']; - $this->Salt = $encryption_details['salt']; - $this->PasswordEncryption = $encryption_details['algorithm']; - - $this->changed['Salt'] = true; - $this->changed['PasswordEncryption'] = true; - } + if($this->SetPassword) $this->Password = $this->SetPassword; if($this->Email) { if($this->ID) { @@ -449,13 +440,22 @@ class Member extends DataObject { } } } - - if($this->SetPassword) $this->Password = $this->SetPassword; if(Director::isLive() && $this->changed['Password'] && $this->record['Password'] && Member::$notify_password_change) $this->sendInfo('changePassword'); + if(isset($this->changed['Password']) && $this->changed['Password']) { + // Password was changed: encrypt the password according the settings + $encryption_details = Security::encrypt_password($this->Password); + $this->Password = $encryption_details['password']; + $this->Salt = $encryption_details['salt']; + $this->PasswordEncryption = $encryption_details['algorithm']; + + $this->changed['Salt'] = true; + $this->changed['PasswordEncryption'] = true; + } + parent::onBeforeWrite(); }