jenkins-bot has submitted this change and it was merged.

Change subject: Defer the SiteStatsUpdate in attemptAddUser().
......................................................................


Defer the SiteStatsUpdate in attemptAddUser().

Change-Id: Ic11bca52eab593ebde346db81ab56c1f6f74f391
---
M CentralAuthHooks.php
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Demon: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/CentralAuthHooks.php b/CentralAuthHooks.php
index 6dc45d1..d159569 100644
--- a/CentralAuthHooks.php
+++ b/CentralAuthHooks.php
@@ -674,8 +674,7 @@
                wfRunHooks( 'AuthPluginAutoCreate', array( $user ) );
 
                # Update user count
-               $ssUpdate = new SiteStatsUpdate( 0, 0, 0, 0, 1 );
-               $ssUpdate->doUpdate();
+               DeferredUpdates::addUpdate( new SiteStatsUpdate( 0, 0, 0, 0, 1 
) );
 
                return true;
        }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic11bca52eab593ebde346db81ab56c1f6f74f391
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Brion VIBBER <[email protected]>
Gerrit-Reviewer: Demon <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to