Fomafix has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/346045 )

Change subject: SiteModule.php: Use ResourceLoader::makeConfigSetScript
......................................................................

SiteModule.php: Use ResourceLoader::makeConfigSetScript

Change-Id: Iac5147450664e5accf1413102f694352566fee20
---
M client/includes/Modules/SiteModule.php
M client/tests/phpunit/includes/Modules/SiteModuleTest.php
2 files changed, 2 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/45/346045/1

diff --git a/client/includes/Modules/SiteModule.php 
b/client/includes/Modules/SiteModule.php
index e2183ab..2818909 100644
--- a/client/includes/Modules/SiteModule.php
+++ b/client/includes/Modules/SiteModule.php
@@ -43,11 +43,7 @@
                        );
                }
 
-               return Xml::encodeJsCall(
-                       'mw.config.set',
-                       [ 'wbCurrentSite', $currentSite ],
-                       ResourceLoader::inDebugMode()
-               );
+               return ResourceLoader::makeConfigSetScript( [ 'wbCurrentSite' 
=> $currentSite ] );
        }
 
 }
diff --git a/client/tests/phpunit/includes/Modules/SiteModuleTest.php 
b/client/tests/phpunit/includes/Modules/SiteModuleTest.php
index 1353eea..69e2b56 100644
--- a/client/tests/phpunit/includes/Modules/SiteModuleTest.php
+++ b/client/tests/phpunit/includes/Modules/SiteModuleTest.php
@@ -29,7 +29,7 @@
        public function testGetScript() {
                $module = new SiteModule();
                $script = $module->getScript( $this->getContext() );
-               $this->assertStringStartsWith( 
'mw.config.set("wbCurrentSite",', $script );
+               $this->assertStringStartsWith( 
'mw.config.set({"wbCurrentSite":', $script );
                $this->assertStringEndsWith( ');', $script );
        }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iac5147450664e5accf1413102f694352566fee20
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Fomafix <foma...@googlemail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to