MaxSem has submitted this change and it was merged.

Change subject: Don't set $wgResourceLoaderDebug
......................................................................


Don't set $wgResourceLoaderDebug

https://gerrit.wikimedia.org/r/#/c/50442/ made it enabled every time
ResourceLoaderTestModules hook gets executed. Which gets executed every time
$wgEnableJavaScriptTest is true.

Change-Id: I3c219ff300ba3267d06c22769b5e4a700126b337
---
M includes/MobileFrontend.hooks.php
1 file changed, 0 insertions(+), 4 deletions(-)

Approvals:
  MaxSem: Verified; Looks good to me, approved
  awjrichards: Verified; Looks good to me, approved



diff --git a/includes/MobileFrontend.hooks.php 
b/includes/MobileFrontend.hooks.php
index bf31612..5a50fe3 100644
--- a/includes/MobileFrontend.hooks.php
+++ b/includes/MobileFrontend.hooks.php
@@ -191,10 +191,6 @@
        public static function onResourceLoaderTestModules( array 
&$testModules, ResourceLoader &$resourceLoader ) {
                global $wgResourceModules, $wgResourceLoaderDebug;
 
-               // run RL in debug mode so that we get real line numbers on 
errors and
-               // so that sinon.js is loaded within global context
-               $wgResourceLoaderDebug = true;
-
                $testModuleBoilerplate = array(
                        'localBasePath' => dirname( __DIR__ ),
                        'remoteExtPath' => 'MobileFrontend',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3c219ff300ba3267d06c22769b5e4a700126b337
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>
Gerrit-Reviewer: MaxSem <[email protected]>
Gerrit-Reviewer: awjrichards <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to