jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/373699 )
Change subject: Fix check ...................................................................... Fix check Bug: T173888 Change-Id: I2643f7cbcdf6c9ad8dff8b6369a5ba6e402684b3 --- M includes/LoginNotify.php 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Niharika29: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/LoginNotify.php b/includes/LoginNotify.php index 06376f1..aa59964 100644 --- a/includes/LoginNotify.php +++ b/includes/LoginNotify.php @@ -292,9 +292,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/373699 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: master Gerrit-Owner: MaxSem <maxsem.w...@gmail.com> Gerrit-Reviewer: Niharika29 <nko...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits