wikimedia / mediawiki

includes/changes/RecentChange.php

Doc Comments +6 added lines, -3 removed lines

@@ -119,7 +119,7 @@ 
 	 * @since 1.24
 	 * @param string|array $type
 	 * @throws MWException
-	 * @return int|array RC_TYPE
+	 * @return string RC_TYPE
 	 */
 	public static function parseToRCType( $type ) {
 		if ( is_array( $type ) ) {
@@ -172,7 +172,7 @@ 
 	 * Find the first recent change matching some specific conditions
 	 *
 	 * @param array $conds Array of conditions
-	 * @param mixed $fname Override the method name in profiling/logs
+	 * @param string $fname Override the method name in profiling/logs
 	 * @param int $dbType DB_* constant
 	 *
 	 * @return RecentChange|null
@@ -194,7 +194,7 @@ 
 	/**
 	 * Return the list of recentchanges fields that should be selected to create
 	 * a new recentchanges object.
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectFields() {
 		return array(
@@ -919,6 +919,9 @@ 
 		return ChangesList::showCharacterDifference( $old, $new );
 	}
 
+	/**
+	 * @param string $ip
+	 */
 	private static function checkIPAddress( $ip ) {
 		global $wgRequest;
 		if ( $ip ) {

includes/changetags/ChangeTags.php

Doc Comments +4 added lines, -1 removed lines

@@ -89,7 +89,7 @@ 
 	 * we consider the tag hidden, and return false.
 	 *
 	 * @param string $tag Tag
-	 * @return string|bool Tag description or false if tag is to be hidden.
+	 * @return string|false Tag description or false if tag is to be hidden.
 	 * @since 1.25 Returns false if tag is to be hidden.
 	 */
 	public static function tagDescription( $tag ) {
@@ -145,6 +145,9 @@ 
 	 * Pass a variable whose value is null if the log_id is not relevant or unknown.
 	 * @param string $params Params to put in the ct_params field of table
 	 * 'change_tag' when adding tags
+	 * @param integer $rc_id
+	 * @param integer $rev_id
+	 * @param integer $log_id
 	 *
 	 * @throws MWException When $rc_id, $rev_id and $log_id are all null
 	 * @return array Index 0 is an array of tags actually added, index 1 is an

includes/Collation.php

Doc Comments +16 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
 	/**
 	 * @throws MWException
 	 * @param string $collationName
-	 * @return Collation
+	 * @return string
 	 */
 	static function factory( $collationName ) {
 		switch ( $collationName ) {
@@ -296,6 +296,9 @@ 
 
 	const RECORD_LENGTH = 14;
 
+	/**
+	 * @param string $locale
+	 */
 	function __construct( $locale ) {
 		if ( !extension_loaded( 'intl' ) ) {
 			throw new MWException( 'An ICU collation was requested, ' .
@@ -512,6 +515,9 @@ 
 		return $data;
 	}
 
+	/**
+	 * @param integer $index
+	 */
 	function getLetterByIndex( $index ) {
 		if ( $this->firstLetterData === null ) {
 			$this->getFirstLetterData();
@@ -563,7 +569,7 @@ 
 	 * currently in use, or false when it can't be determined.
 	 *
 	 * @since 1.21
-	 * @return string|bool
+	 * @return false|string
 	 */
 	static function getUnicodeVersionForICU() {
 		$icuVersion = IcuCollation::getICUVersion();
@@ -621,6 +627,11 @@ 
 		parent::__construct( 'et' );
 	}
 
+	/**
+	 * @param string $string
+	 *
+	 * @return string
+	 */
 	private static function mangle( $string ) {
 		return str_replace(
 			array( 'w', 'W' ),
@@ -629,6 +640,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $string
+	 */
 	private static function unmangle( $string ) {
 		// Casing data is lost…
 		return str_replace(

includes/content/AbstractContent.php

Doc Comments +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
 	 * @param bool $recursive
 	 * @param ParserOutput $parserOutput
 	 *
-	 * @return DataUpdate[]
+	 * @return LinksUpdate[]
 	 *
 	 * @see Content::getSecondaryDataUpdates()
 	 */

includes/content/ContentHandler.php

Doc Comments +4 added lines, -4 removed lines

@@ -649,7 +649,7 @@ 
 	 *
 	 * @param IContextSource $context Context to use, anything else will be ignored.
 	 * @param int $old Revision ID we want to show and diff with.
-	 * @param int|string $new Either a revision ID or one of the strings 'cur', 'prev' or 'next'.
+	 * @param integer $new Either a revision ID or one of the strings 'cur', 'prev' or 'next'.
 	 * @param int $rcid FIXME: Deprecated, no longer used. Defaults to 0.
 	 * @param bool $refreshCache If set, refreshes the diff cache. Defaults to false.
 	 * @param bool $unhide If set, allow viewing deleted revs. Defaults to false.
@@ -787,7 +787,7 @@ 
 	 * @param Content $myContent One of the page's conflicting contents.
 	 * @param Content $yourContent One of the page's conflicting contents.
 	 *
-	 * @return Content|bool Always false.
+	 * @return boolean Always false.
 	 */
 	public function merge3( Content $oldContent, Content $myContent, Content $yourContent ) {
 		return false;
@@ -879,7 +879,7 @@ 
 	 * @param Title $title The page's title
 	 * @param bool &$hasHistory Whether the page has a history
 	 *
-	 * @return mixed String containing deletion reason or empty string, or
+	 * @return false|string String containing deletion reason or empty string, or
 	 *    boolean false if no revision occurred
 	 *
 	 * @todo &$hasHistory is extremely ugly, it's here because
@@ -989,7 +989,7 @@ 
 	 * @param Revision $undo The revision to undo
 	 * @param Revision $undoafter Must be an earlier revision than $undo
 	 *
-	 * @return mixed String on success, false on failure
+	 * @return null|Content String on success, false on failure
 	 */
 	public function getUndoContent( Revision $current, Revision $undo, Revision $undoafter ) {
 		$cur_content = $current->getContent();

includes/context/DerivativeContext.php

Doc Comments -1 removed lines

@@ -329,7 +329,6 @@
 	 * it would set only the original context, and not take
 	 * into account any changes.
 	 *
-	 * @param mixed $args,... Arguments to wfMessage
 	 * @return Message
 	 */
 	public function msg() {

includes/context/RequestContext.php

Doc Comments +1 added lines, -1 removed lines

@@ -323,7 +323,7 @@
 	/**
 	 * Set the Language object
 	 *
-	 * @param Language|string $l Language instance or language code
+	 * @param Language $l Language instance or language code
 	 * @throws MWException
 	 * @since 1.19
 	 */

includes/db/Database.php

Doc Comments +6 added lines, -6 removed lines

@@ -184,7 +184,7 @@ 
 	 *   - false to disable debugging
 	 *   - omitted or null to do nothing
 	 *
-	 * @return bool|null Previous value of the flag
+	 * @return boolean Previous value of the flag
 	 */
 	public function debug( $debug = null ) {
 		return wfSetBit( $this->mFlags, DBO_DEBUG, $debug );
@@ -635,7 +635,7 @@ 
 	}
 
 	/**
-	 * @return bool|string
+	 * @return string|false
 	 */
 	protected function restoreErrorHandler() {
 		restore_error_handler();
@@ -1080,7 +1080,7 @@ 
 	 *
 	 * @param array $options Associative array of options to be turned into
 	 *   an SQL query, valid keys are listed in the function.
-	 * @return array
+	 * @return string[]
 	 * @see DatabaseBase::select()
 	 */
 	public function makeSelectOptions( $options ) {
@@ -2921,8 +2921,8 @@ 
 	 * on object's error ignore settings).
 	 *
 	 * @param string $filename File name to open
-	 * @param bool|callable $lineCallback Optional function called before reading each line
-	 * @param bool|callable $resultCallback Optional function called for each MySQL result
+	 * @param boolean $lineCallback Optional function called before reading each line
+	 * @param boolean $resultCallback Optional function called for each MySQL result
 	 * @param bool|string $fname Calling function name or false if name should be
 	 *   generated dynamically using $filename
 	 * @param bool|callable $inputCallback Optional function called for each
@@ -3235,7 +3235,7 @@ 
 	}
 
 	/**
-	 * @return string|bool Reason this DB is read-only or false if it is not
+	 * @return string|false Reason this DB is read-only or false if it is not
 	 */
 	protected function getReadOnlyReason() {
 		$reason = $this->getLBInfo( 'readOnlyReason' );

includes/db/DatabaseError.php

Doc Comments +1 added lines, -1 removed lines

@@ -328,7 +328,7 @@
 	/**
 	 * @param DatabaseBase $db
 	 * @param string $error
-	 * @param int|string $errno
+	 * @param integer $errno
 	 * @param string $sql
 	 * @param string $fname
 	 */