From: Brion Vibber <brion@pobox.com>
Date: Tue, 1 Dec 2009 23:44:07 +0000 (-0800)
Subject: Fix regression in password settings: users have been unable to change their passwords... 
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=a3a2168292855ce99246b6244084be58552e8172;p=quix0rs-gnu-social.git

Fix regression in password settings: users have been unable to change their passwords since introduction of ChangePassword event (later StartChangePassword)  November 5 in commit d6ddb84132d7b9510ba82064c67f2a39822dab49

Logic was inversed; new password was only being saved if a plugin claimed the event; so when no auth plugin was present to take it, passwords never got saved.
---

diff --git a/actions/passwordsettings.php b/actions/passwordsettings.php
index 4395f772bd..3bb8e3bb9f 100644
--- a/actions/passwordsettings.php
+++ b/actions/passwordsettings.php
@@ -170,7 +170,7 @@ class PasswordsettingsAction extends AccountSettingsAction
         }
 
         $success = false;
-        if(! Event::handle('StartChangePassword', array($user, $oldpassword, $newpassword))){
+        if(Event::handle('StartChangePassword', array($user, $oldpassword, $newpassword))){
             //no handler changed the password, so change the password internally
             $original = clone($user);