Aaron Schulz has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/258402

Change subject: Avoid calling lazyImportLocalNames() in a few spaces
......................................................................

Avoid calling lazyImportLocalNames() in a few spaces

Simplify addLocalName()/removeLocalName() to skip calling this.
They both do the same unconditional DB update anyway.

Change-Id: I46bfef71e1cd544377017273f7be1ae989d72332
---
M includes/CentralAuthUser.php
1 file changed, 0 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CentralAuth 
refs/changes/02/258402/1

diff --git a/includes/CentralAuthUser.php b/includes/CentralAuthUser.php
index 187b112..9790b98 100644
--- a/includes/CentralAuthUser.php
+++ b/includes/CentralAuthUser.php
@@ -1930,7 +1930,6 @@
         */
        function addLocalName( $wikiID ) {
                $dbw = self::getCentralDB();
-               $this->lazyImportLocalNames();
                $dbw->insert( 'localnames',
                        array(
                                'ln_wiki' => $wikiID,
@@ -1945,7 +1944,6 @@
         */
        function removeLocalName( $wikiID ) {
                $dbw = self::getCentralDB();
-               $this->lazyImportLocalNames();
                $dbw->delete( 'localnames',
                        array(
                                'ln_wiki' => $wikiID,

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I46bfef71e1cd544377017273f7be1ae989d72332
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>

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

Reply via email to