r107898 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r107897‎ | r107898 | r107899 >
Date:16:54, 3 January 2012
Author:reedy
Status:ok
Tags:
Comment:
MFT r107595

* (bug 32702) Removed method Skin::makeGlobalVariablesScript() has been readded
for backward compatibility.
Modified paths:
  • /branches/REL1_18/phase3 (modified) (history)
  • /branches/REL1_18/phase3/RELEASE-NOTES-1.18 (modified) (history)
  • /branches/REL1_18/phase3/includes (modified) (history)
  • /branches/REL1_18/phase3/includes/OutputPage.php (modified) (history)
  • /branches/REL1_18/phase3/includes/Skin.php (modified) (history)

Diff [purge]

Index: branches/REL1_18/phase3/RELEASE-NOTES-1.18
@@ -23,6 +23,8 @@
2424 * (bug 33240) Sort images are missing but referenced in css.
2525 * (bug 31921) Magic words REVISIONDAY, REVISIONMONTH and REVISIONYEAR were
2626 not showing their values on preview.
 27+* (bug 32702) Removed method Skin::makeGlobalVariablesScript() has been readded
 28+ for backward compatibility.
2729
2830 == MediaWiki 1.18 ==
2931 2011-11-24
Index: branches/REL1_18/phase3/includes/OutputPage.php
@@ -2618,12 +2618,15 @@
26192619 /**
26202620 * Get an array containing the variables to be set in mw.config in JavaScript.
26212621 *
 2622+ * DO NOT CALL THIS FROM OUTSIDE OF THIS CLASS OR Skin::makeGlobalVariablesScript().
 2623+ * This is only public until that function is removed. You have been warned.
 2624+ *
26222625 * Do not add things here which can be evaluated in ResourceLoaderStartupScript
26232626 * - in other words, page-indendent/site-wide variables (without state).
26242627 * You will only be adding bloat to the html page and causing page caches to
26252628 * have to be purged on configuration changes.
26262629 */
2627 - protected function getJSVars() {
 2630+ public function getJSVars() {
26282631 global $wgUseAjax, $wgEnableMWSuggest, $wgContLang;
26292632
26302633 $title = $this->getTitle();
Property changes on: branches/REL1_18/phase3/includes/OutputPage.php
___________________________________________________________________
Modified: svn:mergeinfo
26312634 Merged /trunk/phase3/includes/OutputPage.php:r107595
Index: branches/REL1_18/phase3/includes/Skin.php
@@ -403,6 +403,20 @@
404404 }
405405
406406 /**
 407+ * Make a <script> tag containing global variables
 408+ *
 409+ * @param $unused Unused
 410+ * @return string HTML fragment
 411+ */
 412+ public static function makeGlobalVariablesScript( $unused ) {
 413+ global $wgOut;
 414+
 415+ wfDeprecated( __METHOD__, '1.19' );
 416+
 417+ return self::makeVariablesScript( $wgOut->getJSVars() );
 418+ }
 419+
 420+ /**
407421 * Get the query to generate a dynamic stylesheet
408422 *
409423 * @return array
Property changes on: branches/REL1_18/phase3/includes
___________________________________________________________________
Modified: svn:mergeinfo
410424 Merged /trunk/phase3/includes:r107595
Property changes on: branches/REL1_18/phase3
___________________________________________________________________
Modified: svn:mergeinfo
411425 Merged /trunk/phase3:r107595

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r107595* (bug 32702) Fix for r82273: readded Skin::makeGlobalVariablesScript() for b...ialex20:39, 29 December 2011

Status & tagging log