Cicalese has submitted this change and it was merged.
Change subject: fixed null user id
......................................................................
fixed null user id
Change-Id: I3b4a4edac3cb35f3931175259c234b4d012b4e6c
---
M PluggableAuth.class.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Cicalese: Verified; Looks good to me, approved
diff --git a/PluggableAuth.class.php b/PluggableAuth.class.php
index cdeef58..eeb030f 100644
--- a/PluggableAuth.class.php
+++ b/PluggableAuth.class.php
@@ -237,7 +237,7 @@
if ( $authorized ) {
if ( $new_user ) {
$user->addToDatabase();
- $instance->saveExtraAttributes( $id );
+ $instance->saveExtraAttributes( $user->mId );
wfDebug( "Added new user: " . $username .
PHP_EOL );
} else {
self::updateUser( $user, $realname, $email );
--
To view, visit https://gerrit.wikimedia.org/r/208148
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3b4a4edac3cb35f3931175259c234b4d012b4e6c
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PluggableAuth
Gerrit-Branch: master
Gerrit-Owner: Cicalese <cical...@mitre.org>
Gerrit-Reviewer: Cicalese <cical...@mitre.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
