wikimedia / mediawiki

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;

includes/linker/LinkRenderer.php

Doc Comments +4 added lines, -1 removed lines

@@ -98,7 +98,7 @@ 
 	}
 
 	/**
-	 * @param string|bool|int $expand A PROTO_* constant or false
+	 * @param string $expand A PROTO_* constant or false
 	 */
 	public function setExpandURLs( $expand ) {
 		$this->expandUrls = $expand;
@@ -189,6 +189,9 @@ 
 		return $options;
 	}
 
+	/**
+	 * @param boolean $isKnown
+	 */
 	private function runBeginHook( LinkTarget $target, &$text, &$extraAttribs, &$query, $isKnown ) {
 		$ret = null;
 		if ( !Hooks::run( 'HtmlPageLinkRendererBegin',

includes/resourceloader/ResourceLoaderFileModule.php

Doc Comments +3 added lines, -2 removed lines

@@ -381,7 +381,7 @@ 
 	 * Get all styles for a given context.
 	 *
 	 * @param ResourceLoaderContext $context
-	 * @return array CSS code for $context as an associative array mapping media type to CSS text.
+	 * @return string CSS code for $context as an associative array mapping media type to CSS text.
 	 */
 	public function getStyles( ResourceLoaderContext $context ) {
 		$styles = $this->readStyleFiles(
@@ -631,7 +631,7 @@ 
 	 * @param array $list List of file paths in any combination of index/path
 	 *     or path/options pairs
 	 * @param string $option Option name
-	 * @param mixed $default Default value if the option isn't set
+	 * @param string $default Default value if the option isn't set
 	 * @return array List of file paths, collated by $option
 	 */
 	protected static function collateFilePathListByOption( array $list, $option, $default ) {
@@ -1007,6 +1007,7 @@ 
 	 * the BOM character is not valid in the middle of a string.
 	 * We already assume UTF-8 everywhere, so this should be safe.
 	 *
+	 * @param string $input
 	 * @return string input minus the intial BOM char
 	 */
 	protected function stripBom( $input ) {

includes/deferred/LinksDeletionUpdate.php

Doc Comments +3 added lines

@@ -163,6 +163,9 @@
 		} );
 	}
 
+	/**
+	 * @param string $table
+	 */
 	private function batchDeleteByPK( $table, array $conds, array $pk, $bSize ) {
 		$dbw = $this->mDb; // convenience
 		$res = $dbw->select( $table, $pk, $conds, __METHOD__ );

includes/changes/ChangesList.php

Doc Comments +9 added lines, -3 removed lines

@@ -153,7 +153,7 @@ 
 	 * @param RecentChange|RCCacheEntry $rc
 	 * @param string|bool $watched Optionally timestamp for adding watched class
 	 *
-	 * @return array of classes
+	 * @return string[] of classes
 	 */
 	protected function getHTMLClasses( $rc, $watched ) {
 		$classes = [];
@@ -228,7 +228,7 @@ 
 	}
 
 	/**
-	 * @param ResultWrapper|array $rows
+	 * @param ResultWrapper $rows
 	 */
 	public function initChangesListRows( $rows ) {
 		Hooks::run( 'ChangesListInitRows', [ $this, $rows ] );
@@ -478,6 +478,8 @@ 
 	 *
 	 * @param string &$s HTML to update
 	 * @param RecentChange &$rc
+	 * @param string $s
+	 * @param RecentChange $rc
 	 */
 	public function insertUserRelatedLinks( &$s, &$rc ) {
 		if ( $this->isDeleted( $rc, Revision::DELETED_USER ) ) {
@@ -636,7 +638,7 @@ 
 
 	/**
 	 * @param RecentChange $rc
-	 * @param array $classes
+	 * @param string[] $classes
 	 * @return string
 	 * @since 1.26
 	 */
@@ -646,6 +648,10 @@ 
 		return $s;
 	}
 
+	/**
+	 * @param string $s
+	 * @param RecentChange $rc
+	 */
 	public function insertExtra( &$s, &$rc, &$classes ) {
 		// Empty, used for subclasses to add anything special.
 	}

includes/gallery/SlideshowImageGallery.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/Sanitizer.php

Doc Comments +4 added lines, -4 removed lines

@@ -1206,7 +1206,7 @@ 
 	 * @since 1.27
 	 *
 	 * @param string $referenceString Space delimited list of ids
-	 * @param string|array $options String or array of strings (default is array()):
+	 * @param string $options String or array of strings (default is array()):
 	 *   'noninitial': This is a non-initial fragment of an id, not a full id,
 	 *       so don't pay attention if the first character isn't valid at the
 	 *       beginning of an id.  Only matters if $wgExperimentalHtmlIds is
@@ -1334,7 +1334,7 @@ 
 	 * Pick the appropriate attribute value from a match set from the
 	 * attribs regex matches.
 	 *
-	 * @param array $set
+	 * @param string[] $set
 	 * @throws MWException When tag conditions are not met.
 	 * @return string
 	 */
@@ -1850,7 +1850,7 @@ 
 
 	/**
 	 * @param string $url
-	 * @return mixed|string
+	 * @return string
 	 */
 	static function cleanUrl( $url ) {
 		# Normalize any HTML entities in input. They will be
@@ -1936,7 +1936,7 @@ 
 	 * @since 1.18
 	 *
 	 * @param string $addr E-mail address
-	 * @return bool
+	 * @return null|boolean
 	 */
 	public static function validateEmail( $addr ) {
 		$result = null;

includes/Block.php

Doc Comments +3 added lines, -3 removed lines

@@ -185,7 +185,7 @@ 
 	/**
 	 * Return the list of ipblocks fields that should be selected to create
 	 * a new block.
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectFields() {
 		return [
@@ -1004,7 +1004,7 @@ 
 	/**
 	 * Get a timestamp of the expiry for autoblocks
 	 *
-	 * @param string|int $timestamp
+	 * @param string|false $timestamp
 	 * @return string
 	 */
 	public static function getAutoblockExpiry( $timestamp ) {
@@ -1361,7 +1361,7 @@ 
 	/**
 	 * @since 1.19
 	 *
-	 * @return mixed|string
+	 * @return string
 	 */
 	public function getExpiry() {
 		return $this->mExpiry;