r72135 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r72134‎ | r72135 | r72136 >
Date:18:14, 1 September 2010
Author:tomasz
Status:deferred
Tags:
Comment:
Sync'ing up to trunk head at 72131
Modified paths:
  • /branches/wmf/1.16wmf4/extensions/CentralNotice (modified) (history)
  • /branches/wmf/1.16wmf4/extensions/CentralNotice/NoticePage.php (modified) (history)
  • /branches/wmf/1.16wmf4/extensions/CentralNotice/SpecialCentralNotice.php (modified) (history)
  • /branches/wmf/1.16wmf4/extensions/CentralNotice/SpecialNoticeLocal.php (modified) (history)
  • /branches/wmf/1.16wmf4/extensions/CentralNotice/SpecialNoticeText.php (modified) (history)
  • /branches/wmf/1.16wmf4/extensions/CentralNotice/rebuildLocalTemplates.php (modified) (history)
  • /branches/wmf/1.16wmf4/extensions/CentralNotice/rebuildTemplates.php (modified) (history)

Diff [purge]

Index: branches/wmf/1.16wmf4/extensions/CentralNotice/SpecialCentralNotice.php
@@ -614,7 +614,7 @@
615615
616616 // Handle new project name
617617 $projectName = $wgRequest->getVal( 'project_name' );
618 - if ( $projectName ) {
 618+ if ( isset( $projectName ) ) {
619619 $this->updateProjectName ( $notice, $projectName );
620620 }
621621
@@ -946,7 +946,7 @@
947947
948948 /**
949949 * Lookup function for active banners under a given language and project. This function is
950 - * called by SpecialNoticeText::getJsOutput() in order to build the static Javascript files for
 950+ * called by SpecialNoticeText::getOutput() in order to build the static Javascript files for
951951 * each project.
952952 * @return A 2D array of running banners with associated weights and settings
953953 */
Index: branches/wmf/1.16wmf4/extensions/CentralNotice/SpecialNoticeLocal.php
@@ -12,7 +12,7 @@
1313 return 7200;
1414 }
1515
16 - function getJsOutput( $par ) {
 16+ function getOutput( $par ) {
1717 $text = '';
1818 if ( $par == 'anonnotice.js' ) {
1919 $text = wfGetCachedNotice( 'anonnotice' );
Index: branches/wmf/1.16wmf4/extensions/CentralNotice/rebuildTemplates.php
@@ -30,7 +30,7 @@
3131 echo "$key\n";
3232
3333 $builder = new SpecialNoticeText();
34 - $js = $builder->getJsOutput( $key );
 34+ $js = $builder->getOutput( $key );
3535
3636 if ( isset( $options['o'] ) ) {
3737 $outputDir = "$wgNoticeCentralDirectory/$project/$lang";
Index: branches/wmf/1.16wmf4/extensions/CentralNotice/NoticePage.php
@@ -5,13 +5,13 @@
66 global $wgOut;
77 $wgOut->disable();
88 $this->sendHeaders();
9 - $js = $this->getJsOutput( $par );
 9+ $content = $this->getOutput( $par );
1010
11 - if ( strlen( $js ) == 0 ) {
 11+ if ( strlen( $content ) == 0 ) {
1212 /* Hack for IE/Mac 0-length keepalive problem, see RawPage.php */
1313 echo "/* Empty */";
1414 } else {
15 - echo $js;
 15+ echo $content;
1616 }
1717 }
1818
@@ -40,7 +40,7 @@
4141 // header( "Last-modified: $epoch" );
4242 }
4343
44 - function getJsOutput( $par ) {
 44+ function getOutput( $par ) {
4545 return "";
4646 }
4747 }
Index: branches/wmf/1.16wmf4/extensions/CentralNotice/rebuildLocalTemplates.php
@@ -22,7 +22,7 @@
2323 echo "$wgDBname/$key\n";
2424
2525 $builder = new SpecialNoticeLocal();
26 - $js = $builder->getJsOutput( $key );
 26+ $js = $builder->getOutput( $key );
2727
2828 $outputDir = $wgNoticeLocalDirectory;
2929 if ( wfMkDirParents( $outputDir ) ) {
Index: branches/wmf/1.16wmf4/extensions/CentralNotice/SpecialNoticeText.php
@@ -27,7 +27,7 @@
2828 /**
2929 * Given a project key, generate the body for a static Javascript file
3030 */
31 - function getJsOutput( $par ) {
 31+ function getOutput( $par ) {
3232
3333 // Break $par into separate parameters and assign to $this->project and $this->language
3434 $this->setLanguage( $par );
Property changes on: branches/wmf/1.16wmf4/extensions/CentralNotice
___________________________________________________________________
Modified: svn:mergeinfo
3535 Merged /trunk/extensions/CentralNotice:r72058-72131

Status & tagging log