jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/373701 )
Change subject: Fix check ...................................................................... Fix check Bug: T173888 Change-Id: I2643f7cbcdf6c9ad8dff8b6369a5ba6e402684b3 (cherry picked from commit 8acdeae44d231cd7672f1800aac04d67b276bc27) --- M includes/LoginNotify.php 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: MaxSem: Verified; Looks good to me, approved jenkins-bot: Verified diff --git a/includes/LoginNotify.php b/includes/LoginNotify.php index 99c46cf..df2ddc2 100644 --- a/includes/LoginNotify.php +++ b/includes/LoginNotify.php @@ -289,9 +289,9 @@ $dbrLocal ); - if ( $res ) { + if ( $res === self::USER_KNOWN ) { $lb->reuseConnection( $dbrLocal ); - return self::USER_KNOWN; + return $res; } if ( $result === self::USER_NO_INFO && $this->userHasCheckUserData( $user->getId(), $dbr ) -- To view, visit https://gerrit.wikimedia.org/r/373701 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2643f7cbcdf6c9ad8dff8b6369a5ba6e402684b3 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/LoginNotify Gerrit-Branch: wmf/1.30.0-wmf.15 Gerrit-Owner: MaxSem <maxsem.w...@gmail.com> Gerrit-Reviewer: MaxSem <maxsem.w...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits