Dr0ptp4kt has uploaded a new change for review.

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


Change subject: Remove superfluous GetMobileNotice call.
......................................................................

Remove superfluous GetMobileNotice call.

* Currently, GetMobileNotice gets called twice instead of once.
* This change will make it so that GetMobileNotice is only called once, from 
the WML context.
* This change can be implemented independently.
* Change 69336 may result in GetMobileNotice becoming the definitive banner 
generation hook (instead of the current split between MinervaPreRender and 
GetMobileNotice), although to be functional it will need to give the callee 
full control over the banners array at the last possible moment (similar to 
MinervaPreRender). Discussion is underway on change 69336.

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


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/10/71710/1

diff --git a/includes/skins/SkinMobile.php b/includes/skins/SkinMobile.php
index 77ae426..11dbe51 100644
--- a/includes/skins/SkinMobile.php
+++ b/includes/skins/SkinMobile.php
@@ -47,8 +47,6 @@
                $tpl->set( 'headelement', $out->headElement( $this ) );
                $tpl->set( 'bodytext', $html );
                $notice = '';
-               wfRunHooks( 'GetMobileNotice', array( $this, &$notice ) );
-               $tpl->set( 'notice', $notice );
                $tpl->set( 'reporttime', wfReportTime() );
                $tpl->execute();
                wfProfileOut( __METHOD__  . '-tpl' );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I151858a632fbd3456f38d2c28e512e959bd5a757
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Dr0ptp4kt <[email protected]>

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

Reply via email to