r70047 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r70046‎ | r70047 | r70048 >
Date:21:34, 27 July 2010
Author:platonides
Status:reverted
Tags:
Comment:
Prefer ef $eg for extensions.
Modified paths:
  • /trunk/extensions/WikimediaMobile/WikimediaMobile.php (modified) (history)

Diff [purge]

Index: trunk/extensions/WikimediaMobile/WikimediaMobile.php
@@ -19,7 +19,7 @@
2020 /**
2121 * Increment this when the JS file changes
2222 */
23 -$wgWikimediaMobileVersion = '2';
 23+$egWikimediaMobileVersion = '2';
2424
2525 /**
2626 * The base URL of the mobile gateway
@@ -27,19 +27,19 @@
2828 $wgWikimediaMobileUrl = 'http://en.m.wikipedia.org/wiki';
2929
3030
31 -$wgHooks['BeforePageDisplay'][] = 'wfWikimediaMobileAddJs';
32 -$wgHooks['MakeGlobalVariablesScript'][] = 'wfWikimediaMobileVars';
 31+$wgHooks['BeforePageDisplay'][] = 'efWikimediaMobileAddJs';
 32+$wgHooks['MakeGlobalVariablesScript'][] = 'efWikimediaMobileVars';
3333
3434 function wfWikimediaMobileAddJs( &$outputPage, &$skin ) {
35 - global $wgOut, $wgExtensionAssetsPath, $wgWikimediaMobileVersion;
 35+ global $wgOut, $wgExtensionAssetsPath, $egWikimediaMobileVersion;
3636
3737 $wgOut->addScript( Html::linkedScript(
38 - "$wgExtensionAssetsPath/WikimediaMobile/MobileRedirect.js?$wgWikimediaMobileVersion"
 38+ "$egExtensionAssetsPath/WikimediaMobile/MobileRedirect.js?$egWikimediaMobileVersion"
3939 ) );
4040 return true;
4141 }
4242
43 -function wfWikimediaMobileVars( &$vars ) {
 43+function efWikimediaMobileVars( &$vars ) {
4444 global $wgWikimediaMobileUrl;
4545 $vars['wgWikimediaMobileUrl'] = $wgWikimediaMobileUrl;
4646 return true;

Follow-up revisions

RevisionCommit summaryAuthorDate
r70755Revert r70047, causes fatal error. Besides, the idea of it is incorrect: wf a...tstarling08:05, 9 August 2010

Status & tagging log