wikimedia / mediawiki

includes/api/ApiAuthManagerHelper.php

Doc Comments +1 added lines, -2 removed lines

@@ -275,8 +275,6 @@ 
 
 	/**
 	 * Clean up a field array for output
-	 * @param ApiBase $module For context and parameters 'mergerequestfields'
-	 *  and 'messageformat'
 	 * @param array $fields
 	 * @return array
 	 */
@@ -315,6 +313,7 @@ 
 	 * Fetch the standard parameters this helper recognizes
 	 * @param string $action AuthManager action
 	 * @param string $param... Parameters to use
+	 * @param string $param
 	 * @return array
 	 */
 	public static function getStandardParams( $action, $param /* ... */ ) {

includes/cache/CacheDependency.php

Doc Comments +10 added lines, -1 removed lines

@@ -172,7 +172,7 @@ 
 	}
 
 	/**
-	 * @return array
+	 * @return string[]
 	 */
 	function __sleep() {
 		$this->loadDependencyValues();
@@ -228,6 +228,9 @@ 
 	private $name;
 	private $value;
 
+	/**
+	 * @param string $name
+	 */
 	function __construct( $name ) {
 		$this->name = $name;
 		$this->value = $GLOBALS[$name];
@@ -252,6 +255,9 @@ 
 	private $name;
 	private $value;
 
+	/**
+	 * @param string $name
+	 */
 	function __construct( $name ) {
 		$this->name = $name;
 		$this->value = $this->getConfig()->get( $this->name );
@@ -280,6 +286,9 @@ 
 	private $name;
 	private $value;
 
+	/**
+	 * @param string $name
+	 */
 	function __construct( $name ) {
 		$this->name = $name;
 		$this->value = constant( $name );

includes/libs/StatusValue.php

Doc Comments +2 added lines, -2 removed lines

@@ -139,7 +139,7 @@ 
 	/**
 	 * Add a new warning
 	 *
-	 * @param string|MessageSpecifier $message Message key or object
+	 * @param Message $message Message key or object
 	 */
 	public function warning( $message /*, parameters... */ ) {
 		$this->errors[] = [
@@ -167,7 +167,7 @@ 
 	 * Add an error and set OK to false, indicating that the operation
 	 * as a whole was fatal
 	 *
-	 * @param string|MessageSpecifier $message Message key or object
+	 * @param string $message Message key or object
 	 */
 	public function fatal( $message /*, parameters... */ ) {
 		$this->errors[] = [

includes/skins/QuickTemplate.php

Doc Comments +1 added lines

@@ -97,6 +97,7 @@
 	/**
 	 * Main function, used by classes that subclass QuickTemplate
 	 * to show the actual HTML output
+	 * @return string
 	 */
 	abstract public function execute();
 

includes/specialpage/AuthManagerSpecialPage.php

Doc Comments +3 added lines, -2 removed lines

@@ -197,7 +197,7 @@ 
 	 * Get the default action for this special page, if none is given via URL/POST data.
 	 * Subclasses should override this (or override loadAuth() so this is never called).
 	 * @param string $subPage Subpage of the special page.
-	 * @return string an AuthManager::ACTION_* constant.
+	 * @return string|null an AuthManager::ACTION_* constant.
 	 */
 	protected function getDefaultAction( $subPage ) {
 		throw new BadMethodCallException( 'Subclass did not implement getDefaultAction' );
@@ -206,6 +206,7 @@ 
 	/**
 	 * Return custom message key.
 	 * Allows subclasses to customize messages.
+	 * @param string $defaultKey
 	 * @return string
 	 */
 	protected function messageKey( $defaultKey ) {
@@ -708,7 +709,7 @@ 
 	 * Get an array value, or a default if it does not exist.
 	 * @param array $array
 	 * @param string $fieldName
-	 * @param mixed $default
+	 * @param integer $default
 	 * @return mixed
 	 */
 	protected static function getField( array $array, $fieldName, $default = null ) {

includes/specials/helpers/LoginHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
 	}
 
 	/**
-	 * @return array
+	 * @return string[]
 	 */
 	function getOrderFields() {
 		return array( 'rd_namespace', 'rd_title', 'rd_from' );

includes/diff/DairikiDiff.php

Doc Comments +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
 	 *    $diff = new Diff($lines1, $lines2);
 	 *    $rev = $diff->reverse();
 	 *
-	 * @return Object A Diff object representing the inverse of the
+	 * @return Diff A Diff object representing the inverse of the
 	 *   original diff.
 	 */
 	public function reverse() {

includes/diff/DiffEngine.php

Doc Comments +18 added lines

@@ -222,6 +222,13 @@ 
 		return $ranges;
 	}
 
+	/**
+	 * @param integer $bottoml1
+	 * @param integer $topl1
+	 * @param integer $bottoml2
+	 * @param integer $topl2
+	 * @param integer[] $snake
+	 */
 	private function lcs_rec( $bottoml1, $topl1, $bottoml2, $topl2, &$V, &$snake ) {
 		// check that both sequences are non-empty
 		if ( $bottoml1 > $topl1 || $bottoml2 > $topl2 ) {
@@ -490,6 +497,11 @@ 
 		*/
 	}
 
+	/**
+	 * @param double $M
+	 * @param double $N
+	 * @param double $limit
+	 */
 	private static function findMostProgress( $M, $N, $limit, $V ) {
 		$delta = $N - $M;
 
@@ -609,6 +621,12 @@ 
 	/** @var int */
 	public $rightlength;
 
+	/**
+	 * @param integer $leftstart
+	 * @param integer $leftend
+	 * @param integer $rightstart
+	 * @param integer $rightend
+	 */
 	function __construct( $leftstart, $leftend, $rightstart, $rightend ) {
 		$this->leftstart = $leftstart;
 		$this->leftend = $leftend;

includes/interwiki/ClassicInterwikiLookup.php

Doc Comments +3 added lines

@@ -112,6 +112,9 @@
 		}
 	}
 
+	/**
+	 * @return string
+	 */
 	function stripParameters( $text ) {
 		if ( !$this->stripParametersEnabled ) {
 			return $text;