r112093 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r112092‎ | r112093 | r112094 >
Date:09:27, 22 February 2012
Author:tstarling
Status:ok
Tags:
Comment:
MFT r112092: mergeMessageFileList.php fixes
Modified paths:
  • /branches/wmf/1.19wmf1/extensions/CustomUserSignup/CustomUserSignup.php (modified) (history)
  • /branches/wmf/1.19wmf1/extensions/FlaggedRevs/FlaggedRevs.php (modified) (history)
  • /branches/wmf/1.19wmf1/extensions/FlaggedRevs/frontend/FlaggedRevsUI.setup.php (modified) (history)
  • /branches/wmf/1.19wmf1/extensions/ZeroRatedMobileAccess/ZeroRatedMobileAccess.body.php (modified) (history)
  • /branches/wmf/1.19wmf1/extensions/ZeroRatedMobileAccess/ZeroRatedMobileAccess.php (modified) (history)

Diff [purge]

Index: branches/wmf/1.19wmf1/extensions/FlaggedRevs/FlaggedRevs.php
@@ -42,6 +42,7 @@
4343
4444 # Define were PHP files and i18n files are located
4545 require( dirname( __FILE__ ) . '/FlaggedRevs.setup.php' );
 46+require( dirname( __FILE__ ) . '/frontend/FlaggedRevsUI.setup.php' );
4647 FlaggedRevsSetup::defineSourcePaths( $wgAutoloadClasses, $wgExtensionMessagesFiles );
4748
4849 # Define JS/CSS modules and file locations
Index: branches/wmf/1.19wmf1/extensions/FlaggedRevs/frontend/FlaggedRevsUI.setup.php
@@ -182,7 +182,7 @@
183183 * @param $ajaxExportList Array $wgAjaxExportList
184184 * @return void
185185 */
186 - public static function defineAjaxFunctions( array &$ajaxExportList ) {
 186+ public static function defineAjaxFunctions( &$ajaxExportList ) {
187187 $ajaxExportList[] = 'RevisionReview::AjaxReview';
188188 $ajaxExportList[] = 'FlaggablePageView::AjaxBuildDiffHeaderItems';
189189 }
@@ -195,7 +195,7 @@
196196 * @return void
197197 */
198198 public static function defineLogBasicDescription(
199 - array &$logNames, array &$logHeaders, array &$filterLogTypes
 199+ &$logNames, &$logHeaders, &$filterLogTypes
200200 ) {
201201 $logNames['review'] = 'review-logpage';
202202 $logHeaders['review'] = 'review-logpagetext';
@@ -213,7 +213,7 @@
214214 * @return void
215215 */
216216 public static function defineLogActionHandlers(
217 - array &$logActions, array &$logActionsHandlers
 217+ &$logActions, &$logActionsHandlers
218218 ) {
219219 # Various actions are used for log filtering ...
220220 $logActions['review/approve'] = 'review-logentry-app'; // checked (again)
Index: branches/wmf/1.19wmf1/extensions/CustomUserSignup/CustomUserSignup.php
@@ -45,5 +45,7 @@
4646 $wgCustomUserSignupVersion = 1;
4747 $wgCustomUserSignupSetBuckets = true;
4848 // For Account Creation Project
49 -ClickTrackingHooks::addCampaign($dir. 'modules', 'CustomUserSignup/modules', 'AccountCreationUserBucket' );
 49+if ( class_exists( 'ClickTrackingHooks' ) ) {
 50+ ClickTrackingHooks::addCampaign($dir. 'modules', 'CustomUserSignup/modules', 'AccountCreationUserBucket' );
 51+}
5052
Index: branches/wmf/1.19wmf1/extensions/ZeroRatedMobileAccess/ZeroRatedMobileAccess.php
@@ -41,6 +41,4 @@
4242
4343 $wgEnableZeroRatedMobileAccessTesting = false;
4444
45 -$wgExtZeroRatedMobileAccess = new ExtZeroRatedMobileAccess();
46 -
47 -$wgHooks['BeforePageDisplay'][] = array( &$wgExtZeroRatedMobileAccess, 'beforePageDisplayHTML' );
\ No newline at end of file
 45+$wgHooks['BeforePageDisplay'][] = 'ExtZeroRatedMobileAccess::onBeforePageDisplay';
Index: branches/wmf/1.19wmf1/extensions/ZeroRatedMobileAccess/ZeroRatedMobileAccess.body.php
@@ -16,6 +16,11 @@
1717 private static $forceClickToViewImages;
1818 public static $useFormat;
1919
 20+ public static function onBeforePageDisplay( &$out, &$text ) {
 21+ $ext = new ExtZeroRatedMobileAccess();
 22+ return $ext->beforePageDisplayHTML( $out, $text );
 23+ }
 24+
2025 /**
2126 * Handler for the BeforePageDisplay hook
2227 *

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r112092More fixes for mergeMessageFileList.php:...tstarling09:21, 22 February 2012

Status & tagging log