Cicalese has uploaded a new change for review.

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

Change subject: added editmyprivateinfo right check
......................................................................

added editmyprivateinfo right check

Change-Id: Idfcf5fccfdbf8fdffe8ebb7b02fa6dd80912a10f
---
M PluggableAuth.class.php
1 file changed, 21 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PluggableAuth 
refs/changes/17/208517/1

diff --git a/PluggableAuth.class.php b/PluggableAuth.class.php
index eeb030f..901f3a3 100644
--- a/PluggableAuth.class.php
+++ b/PluggableAuth.class.php
@@ -329,17 +329,27 @@
 
        private static function updateUser( $user, $realname, $email ) {
                if ( $user->mRealName != $realname || $user->mEmail != $email ) 
{
-                       $user->mRealName = $realname;
-                       $user->mEmail = $email;
-                       $dbw = wfGetDB( DB_MASTER );
-                       $dbw->update( 'user',
-                               array( // SET
-                                       'user_real_name' => $realname,
-                                       'user_email' => $email
-                               ), array( // WHERE
-                                       'user_id' => $user->mId
-                               ), __METHOD__
-                       );
+                       $rights = $user->getRights();
+                       if ( in_array( "editmyprivateinfo", $rights ) ) {
+                               wfDebug( "updateUser(): User has 
editmyprivateinfo right." . PHP_EOL );
+                               wfDebug( "updateUser(): Did not save updated 
real name and email address." . PHP_EOL );
+                       } else {
+                               wfDebug( "updateUser(): User does not have 
editmyprivateinfo right." . PHP_EOL );
+                               $user->mRealName = $realname;
+                               $user->mEmail = $email;
+                               $dbw = wfGetDB( DB_MASTER );
+                               $dbw->update( 'user',
+                                       array( // SET
+                                               'user_real_name' => $realname,
+                                               'user_email' => $email
+                                       ), array( // WHERE
+                                               'user_id' => $user->mId
+                                       ), __METHOD__
+                               );
+                               wfDebug( "updateUser(): Saved updated real name 
and email address." . PHP_EOL );
+                       }
+               } else {
+                               wfDebug( "updateUser(): Real name and email 
address did not change." . PHP_EOL );
                }
        }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idfcf5fccfdbf8fdffe8ebb7b02fa6dd80912a10f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PluggableAuth
Gerrit-Branch: master
Gerrit-Owner: Cicalese <cical...@mitre.org>

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

Reply via email to