[MediaWiki-commits] [Gerrit] Don't use protected CentralAuthUser::getPasswordObject - change (operations/mediawiki-config)

[Hoo man (Code Review)]

Hoo man has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/184147

Change subject: Don't use protected CentralAuthUser::getPasswordObject
......................................................................

Don't use protected CentralAuthUser::getPasswordObject

Doing it like CentralAuthPlugin does now... *sigh*

Change-Id: I59766cbf9bc71e32a2b08e6703635a9982b3903b
---
M wmf-config/Bug54847.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config 
refs/changes/47/184147/1

diff --git a/wmf-config/Bug54847.php b/wmf-config/Bug54847.php
index 4d3aea3..6125b80 100644
--- a/wmf-config/Bug54847.php
+++ b/wmf-config/Bug54847.php
@@ -158,7 +158,7 @@
        try {
                if ( $isGlobal ) {
                        $centralUser = CentralAuthUser::getInstance( $user );
-                       if ( $centralUser->getPasswordObject()->equals( 
$newpassword ) ) {
+                       if ( $centralUser->authenticate( $newpassword ) === 
'ok' ) {
                                wfDebugLog( "Bug54847", "User attempted to 
reset with CentralAuth password: " . $user->getName() );
                                $errorMsg = 'password-recycled';
                                return false;

-- 
To view, visit https://gerrit.wikimedia.org/r/184147
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I59766cbf9bc71e32a2b08e6703635a9982b3903b
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: Hoo man <h...@online.de>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits