r108374 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r108373‎ | r108374 | r108375 >
Date:22:38, 8 January 2012
Author:reedy
Status:ok
Tags:
Comment:
Fixup documentation

Swap require for require_once

Wrap long line
Modified paths:
  • /trunk/extensions/CodeReview/maintenance/deleteBadTags.php (modified) (history)
  • /trunk/extensions/OpenStackManager/special/SpecialNovaInstance.php (modified) (history)
  • /trunk/phase3/includes/Action.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/Action.php
@@ -146,7 +146,7 @@
147147 /**
148148 * Shortcut to get the user Language being used for this instance
149149 *
150 - * @return Skin
 150+ * @return Language
151151 */
152152 public final function getLanguage() {
153153 return $this->getContext()->getLanguage();
@@ -156,7 +156,7 @@
157157 * Shortcut to get the user Language being used for this instance
158158 *
159159 * @deprecated 1.19 Use getLanguage instead
160 - * @return Skin
 160+ * @return Language
161161 */
162162 public final function getLang() {
163163 wfDeprecated( __METHOD__, '1.19' );
@@ -294,8 +294,6 @@
295295
296296 /**
297297 * Execute the action in a silent fashion: do not display anything or release any errors.
298 - * @param $data Array values that would normally be in the POST request
299 - * @param $captureErrors Bool whether to catch exceptions and just return false
300298 * @return Bool whether execution was successful
301299 */
302300 public abstract function execute();
Index: trunk/extensions/OpenStackManager/special/SpecialNovaInstance.php
@@ -487,6 +487,10 @@
488488 $header .= Html::element( 'th', array(), wfMsg( 'openstackmanager-launchtime' ) );
489489 $header .= Html::element( 'th', array(), wfMsg( 'openstackmanager-actions' ) );
490490 $projectArr = array();
 491+
 492+ /**
 493+ * @var $instance OpenStackNovaInstance
 494+ */
491495 foreach ( $instances as $instance ) {
492496 $project = $instance->getOwner();
493497 if ( ! in_array( $project, $userProjects ) ) {
@@ -591,7 +595,8 @@
592596 $title = Title::newFromText( $this->getOutput()->getPageTitle() );
593597 $job = new OpenStackNovaHostJob( $title, array( 'instanceid' => $instance->getInstanceId() ) );
594598 $job->insert();
595 - $this->getOutput()->addWikiMsg( 'openstackmanager-createdinstance', $instance->getInstanceID(), $instance->getImageId(), $host->getFullyQualifiedHostName() );
 599+ $this->getOutput()->addWikiMsg( 'openstackmanager-createdinstance', $instance->getInstanceID(),
 600+ $instance->getImageId(), $host->getFullyQualifiedHostName() );
596601 } else {
597602 $this->userNova->terminateInstance( $instance->getInstanceId() );
598603 $this->getOutput()->addWikiMsg( 'openstackmanager-createfailedldap' );
Index: trunk/extensions/CodeReview/maintenance/deleteBadTags.php
@@ -4,7 +4,7 @@
55 if ( $IP === false ) {
66 $IP = dirname( __FILE__ ) . '/../../..';
77 }
8 -require "$IP/maintenance/commandLine.inc";
 8+require_once( "$IP/maintenance/commandLine.inc" );
99
1010 echo "Usage: php deleteBadTags.php [commit]\n";
1111

Status & tagging log