Jdlrobson has submitted this change and it was merged.

Change subject: Rm unused variables
......................................................................


Rm unused variables

Change-Id: I7e09b0504ab8c3d157091c28819b9dcf8698c40b
---
M includes/MobileFrontend.hooks.php
M includes/skins/SkinMobileBase.php
2 files changed, 1 insertion(+), 4 deletions(-)

Approvals:
  Jdlrobson: Verified; Looks good to me, approved
  jenkins-bot: Checked



diff --git a/includes/MobileFrontend.hooks.php 
b/includes/MobileFrontend.hooks.php
index 92e1304..94ef3ff 100644
--- a/includes/MobileFrontend.hooks.php
+++ b/includes/MobileFrontend.hooks.php
@@ -20,7 +20,6 @@
         * @return boolean
         */
        public static function onEnableMobileModules( $out, $mode ) {
-               global $wgResourceModules;
                if ( $mode === 'alpha' && $out->getTitle()->isSpecialPage() ) {
                        $out->addModules(
                                array(
@@ -275,7 +274,7 @@
         * @return bool
         */
        public static function onResourceLoaderRegisterModules( ResourceLoader 
&$resourceLoader ) {
-               global $wgMFMobileResourceBoilerplate, $wgMFLogEvents, 
$wgResourceModules, $localBasePath;
+               global $wgMFMobileResourceBoilerplate, $wgMFLogEvents, 
$wgResourceModules;
 
                $detector = DeviceDetection::factory();
                foreach ( $detector->getCssFiles() as $file ) {
diff --git a/includes/skins/SkinMobileBase.php 
b/includes/skins/SkinMobileBase.php
index b62e2d8..62ef274 100644
--- a/includes/skins/SkinMobileBase.php
+++ b/includes/skins/SkinMobileBase.php
@@ -117,8 +117,6 @@
         */
        protected function prepareTemplate() {
                wfProfileIn( __METHOD__ );
-               $ctx = MobileContext::singleton();
-               $req = $this->getRequest();
 
                $tpl = $this->setupTemplate( $this->template );
                $tpl->setRef( 'skin', $this );

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

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

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

Reply via email to