wikimedia / mediawiki

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/api/ApiCSPReport.php

Doc Comments +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
 	/**
 	 * Log CSP report, with a different severity depending on $flags
 	 * @param $flags Array Flags for this report
-	 * @param $logLine String text of log entry
+	 * @param string $logLine String text of log entry
 	 * @param $context Array logging context
 	 */
 	private function logReport( $flags, $logLine, $context ) {
@@ -165,7 +165,7 @@ 
 	 * Stop processing the request, and output/log an error
 	 *
 	 * @param $code String error code
-	 * @param $method String method that made error
+	 * @param string $method String method that made error
 	 * @throws UsageException Always
 	 */
 	private function error( $code, $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/utils/MWGrants.php

Doc Comments +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
 
 	/**
 	 * Fetch the rights allowed by a set of grants.
-	 * @param string[]|string $grants
+	 * @param string[] $grants
 	 * @return string[]
 	 */
 	public static function getGrantRights( $grants ) {

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;

includes/Category.php

Doc Comments +7 added lines, -7 removed lines

@@ -205,42 +205,42 @@ 
 	}
 
 	/**
-	 * @return mixed DB key name, or false on failure
+	 * @return boolean DB key name, or false on failure
 	 */
 	public function getName() {
 		return $this->getX( 'mName' );
 	}
 
 	/**
-	 * @return mixed Category ID, or false on failure
+	 * @return boolean Category ID, or false on failure
 	 */
 	public function getID() {
 		return $this->getX( 'mID' );
 	}
 
 	/**
-	 * @return mixed Total number of member pages, or false on failure
+	 * @return boolean Total number of member pages, or false on failure
 	 */
 	public function getPageCount() {
 		return $this->getX( 'mPages' );
 	}
 
 	/**
-	 * @return mixed Number of subcategories, or false on failure
+	 * @return boolean Number of subcategories, or false on failure
 	 */
 	public function getSubcatCount() {
 		return $this->getX( 'mSubcats' );
 	}
 
 	/**
-	 * @return mixed Number of member files, or false on failure
+	 * @return boolean Number of member files, or false on failure
 	 */
 	public function getFileCount() {
 		return $this->getX( 'mFiles' );
 	}
 
 	/**
-	 * @return Title|bool Title for this category, or false on failure.
+	 * @return Title Title for this category, or false on failure.
 	 */
 	public function getTitle() {
 		if ( $this->mTitle ) {
@@ -260,7 +260,7 @@ 
 	 * category sort key $offset.
 	 * @param int $limit
 	 * @param string $offset
-	 * @return TitleArray TitleArray object for category members.
+	 * @return TitleArrayFromResult|null TitleArray object for category members.
 	 */
 	public function getMembers( $limit = false, $offset = '' ) {