CSteipp has submitted this change and it was merged.

Change subject: Fixed local case for ID => name lookup
......................................................................


Fixed local case for ID => name lookup

Change-Id: I8ba1dc2bd16988c5afbdd5064fef572c1b240a67
---
M backend/MWOAuthUtils.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  CSteipp: Verified; Looks good to me, approved



diff --git a/backend/MWOAuthUtils.php b/backend/MWOAuthUtils.php
index e168837..d9a5a43 100644
--- a/backend/MWOAuthUtils.php
+++ b/backend/MWOAuthUtils.php
@@ -263,7 +263,7 @@
                        }
                } else {
                        $name = '';
-                       $user = User::newFromName( $userId );
+                       $user = User::newFromId( $userId );
                        if ( !$user->isHidden()
                                || ( $audience instanceof User && 
$audience->isAllowed( 'hideuser' ) )
                        ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8ba1dc2bd16988c5afbdd5064fef572c1b240a67
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/OAuth
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: CSteipp <cste...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to