wikimedia / mediawiki

includes/page/CategoryPage.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	/**
 	 * Constructor from a page id
 	 * @param int $id Article ID to load
-	 * @return CategoryPage|null
+	 * @return CategoryPage
 	 */
 	public static function newFromID( $id ) {
 		$t = Title::newFromID( $id );

includes/page/ImagePage.php

Doc Comments +4 added lines, -4 removed lines

@@ -50,7 +50,7 @@ 
 	/**
 	 * Constructor from a page id
 	 * @param int $id Article ID to load
-	 * @return ImagePage|null
+	 * @return ImagePage
 	 */
 	public static function newFromID( $id ) {
 		$t = Title::newFromID( $id );
@@ -261,7 +261,7 @@ 
 	 *
 	 * @todo FIXME: Bad interface, see note on MediaHandler::formatMetadata().
 	 *
-	 * @param array $metadata The array containing the Exif data
+	 * @param boolean $metadata The array containing the Exif data
 	 * @return string The metadata table. This is treated as Wikitext (!)
 	 */
 	protected function makeMetadataTable( $metadata ) {
@@ -290,7 +290,7 @@ 
 	 *
 	 * Omit noarticletext if sharedupload; text will be fetched from the
 	 * shared upload server if possible.
-	 * @return string
+	 * @return null|Content
 	 */
 	public function getContentObject() {
 		$this->loadFile();
@@ -631,7 +631,7 @@ 
 	 * Make the text under the image to say what size preview
 	 *
 	 * @param $params Array parameters for thumbnail
-	 * @param $sizeLinkBigImagePreview HTML for the current size
+	 * @param string $sizeLinkBigImagePreview HTML for the current size
 	 * @return string HTML output
 	 */
 	private function getThumbPrevText( $params, $sizeLinkBigImagePreview ) {

includes/page/WikiFilePage.php

Doc Comments +6 added lines, -3 removed lines

@@ -34,6 +34,9 @@ 
 	protected $mFileLoaded = false;		    // !<
 	protected $mDupes = null;				// !<
 
+	/**
+	 * @param Title $title
+	 */
 	public function __construct( $title ) {
 		parent::__construct( $title );
 		$this->mDupes = null;
@@ -66,7 +69,7 @@ 
 	}
 
 	/**
-	 * @return mixed|null|Title
+	 * @return Title|null
 	 */
 	public function getRedirectTarget() {
 		$this->loadFile();
@@ -120,7 +123,7 @@ 
 	}
 
 	/**
-	 * @return bool|File
+	 * @return File
 	 */
 	public function getFile() {
 		$this->loadFile();
@@ -192,7 +195,7 @@ 
 	 * For foreign API files (InstantCommons), this is not supported currently.
 	 * Results will include hidden categories.
 	 *
-	 * @return TitleArray|Title[]
+	 * @return TitleArrayFromResult|null
 	 * @since 1.23
 	 */
 	public function getForeignCategories() {

includes/page/WikiPage.php

Doc Comments +6 added lines, -6 removed lines

@@ -267,7 +267,7 @@ 
 	 * Return the list of revision fields that should be selected to create
 	 * a new page.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectFields() {
 		global $wgContentHandlerUseDB, $wgPageLanguageUseDB;
@@ -528,7 +528,7 @@ 
 
 	/**
 	 * Get the page_links_updated field
-	 * @return string|null Containing GMT timestamp
+	 * @return string Containing GMT timestamp
 	 */
 	public function getLinksTimestamp() {
 		if ( !$this->mDataLoaded ) {
@@ -1157,7 +1157,7 @@ 
 	 * @param IDatabase $dbw
 	 * @param int|null $pageId Custom page ID that will be used for the insert statement
 	 *
-	 * @return bool|int The newly created page_id key; false if the title already existed
+	 * @return integer The newly created page_id key; false if the title already existed
 	 */
 	public function insertOn( $dbw, $pageId = null ) {
 		$pageIdForInsert = $pageId ?: $dbw->nextSequenceValue( 'page_page_id_seq' );
@@ -1484,7 +1484,7 @@ 
 	 * @param int|null $baseRevId
 	 *
 	 * @throws MWException
-	 * @return Content|null New complete article content, or null if error.
+	 * @return Content New complete article content, or null if error.
 	 *
 	 * @since 1.24
 	 */
@@ -3331,7 +3331,7 @@ 
 	 * Returns a list of categories this page is a member of.
 	 * Results will include hidden categories
 	 *
-	 * @return TitleArray
+	 * @return TitleArrayFromResult|null
 	 */
 	public function getCategories() {
 		$id = $this->getId();
@@ -3406,7 +3406,7 @@ 
 	 * Auto-generates a deletion reason
 	 *
 	 * @param bool &$hasHistory Whether the page has a history
-	 * @return string|bool String containing deletion reason or empty string, or boolean false
+	 * @return false|string String containing deletion reason or empty string, or boolean false
 	 *    if no revision occurred
 	 */
 	public function getAutoDeleteReason( &$hasHistory ) {

includes/pager/TablePager.php

Doc Comments +2 added lines, -2 removed lines

@@ -220,7 +220,7 @@ 
 	 *
 	 * @protected
 	 *
-	 * @param object $row The database result row
+	 * @param stdClass $row The database result row
 	 * @return array Array of attribute => value
 	 */
 	function getRowAttrs( $row ) {
@@ -387,7 +387,7 @@ 
 	 * Resubmits all defined elements of the query string, except for a
 	 * blacklist, passed in the $blacklist parameter.
 	 *
-	 * @param array $blacklist Parameters from the request query which should not be resubmitted
+	 * @param string[] $blacklist Parameters from the request query which should not be resubmitted
 	 * @return string HTML fragment
 	 */
 	function getHiddenFields( $blacklist = array() ) {

includes/parser/CoreParserFunctions.php

Doc Comments +4 added lines, -1 removed lines

@@ -264,6 +264,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $func
+	 */
 	public static function urlFunction( $func, $s = '', $arg = null ) {
 		$title = Title::newFromText( $s );
 		# Due to order of execution of a lot of bits, the values might be encoded
@@ -521,7 +524,7 @@ 
 	 * to not break PHP 5.3
 	 * @param Parser $parser
 	 * @param string $title
-	 * @return mixed|string
+	 * @return string
 	 */
 	public static function mwnamespace( $parser, $title = null ) {
 		$t = Title::newFromText( $title );

includes/parser/DateFormatter.php

Doc Comments +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
 	/**
 	 * Get a DateFormatter object
 	 *
-	 * @param Language|string|null $lang In which language to format the date
+	 * @param Language $lang In which language to format the date
 	 * 		Defaults to the site content language
 	 * @return DateFormatter
 	 */
@@ -147,7 +147,7 @@ 
 	/**
 	 * @param string $preference User preference
 	 * @param string $text Text to reformat
-	 * @param array $options Array can contain 'linked' and/or 'match-whole'
+	 * @param string[] $options Array can contain 'linked' and/or 'match-whole'
 	 *
 	 * @return string
 	 */

includes/parser/LinkHolderArray.php

Doc Comments +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
 	 * serializing at present.
 	 *
 	 * Compact the titles, only serialize the text form.
-	 * @return array
+	 * @return string[]
 	 */
 	public function __sleep() {
 		foreach ( $this->internals as &$nsLinks ) {
@@ -119,8 +119,8 @@ 
 	 * strings will be returned.
 	 *
 	 * @param LinkHolderArray $other
-	 * @param array $texts Array of strings
-	 * @return array
+	 * @param string[] $texts Array of strings
+	 * @return string[]
 	 */
 	public function mergeForeign( $other, $texts ) {
 		$this->tempIdOffset = $idOffset = $this->parent->nextLinkID();

includes/parser/Parser.php

Doc Comments +16 added lines, -11 removed lines

@@ -617,7 +617,7 @@ 
 	 * @since 1.25
 	 *
 	 * @param string $text Text extension wants to have parsed
-	 * @param bool|PPFrame $frame The frame to use for expanding any template variables
+	 * @param PPFrame $frame The frame to use for expanding any template variables
 	 * @return string Fully parsed HTML
 	 */
 	public function recursiveTagParseFully( $text, $frame = false ) {
@@ -635,7 +635,7 @@ 
 	 * @param ParserOptions $options
 	 * @param int|null $revid
 	 * @param bool|PPFrame $frame
-	 * @return mixed|string
+	 * @return string
 	 */
 	public function preprocess( $text, Title $title = null,
 		ParserOptions $options, $revid = null, $frame = false
@@ -866,7 +866,7 @@ 
 
 	/**
 	 * Get the language object for language conversion
-	 * @return Language|null
+	 * @return Language
 	 */
 	public function getConverterLanguage() {
 		return $this->getTargetLanguage();
@@ -1938,6 +1938,10 @@ 
 		return $ret;
 	}
 
+	/**
+	 * @param string $component
+	 * @param string $unsafe
+	 */
 	private static function normalizeUrlComponent( $component, $unsafe ) {
 		$callback = function ( $matches ) use ( $unsafe ) {
 			$char = urldecode( $matches[0] );
@@ -2398,6 +2402,7 @@ 
 	 *
 	 * @param string $target The source of the link
 	 * @param string &$text The link text, modified as necessary
+	 * @param string $text
 	 * @return string The full name of the link
 	 * @private
 	 */
@@ -2901,7 +2906,7 @@ 
 	 * @private
 	 *
 	 * @param int $index
-	 * @param bool|PPFrame $frame
+	 * @param PPFrame $frame
 	 *
 	 * @throws MWException
 	 * @return string
@@ -3298,7 +3303,7 @@ 
 	 *
 	 * @param string $s
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public static function splitWhitespace( $s ) {
 		$ltrimmed = ltrim( $s );
@@ -3406,7 +3411,7 @@ 
 	 *   'expansion-depth-exceeded' (corresponding messages:
 	 *       'expansion-depth-exceeded-warning',
 	 *       'expansion-depth-exceeded-category')
-	 * @param string|int|null $current Current value
+	 * @param integer $current Current value
 	 * @param string|int|null $max Maximum allowed, when an explicit limit has been
 	 *	 exceeded, provide the values (optional)
 	 */
@@ -4402,7 +4407,7 @@ 
 	 * @param string $text
 	 * @param string $origText Original, untouched wikitext
 	 * @param bool $isMain
-	 * @return mixed|string
+	 * @return string
 	 * @private
 	 */
 	public function formatHeadings( $text, $origText, $isMain = true ) {
@@ -4971,7 +4976,7 @@ 
 	 * Check that the user's signature contains no bad XML
 	 *
 	 * @param string $text
-	 * @return string|bool An expanded string, or false if invalid.
+	 * @return string|false An expanded string, or false if invalid.
 	 */
 	public function validateSig( $text ) {
 		return Xml::isWellFormedXmlFragment( $text ) ? $text : false;
@@ -5513,7 +5518,7 @@ 
 	 *
 	 * @param Title $title
 	 * @param string $options
-	 * @param LinkHolderArray|bool $holders
+	 * @param LinkHolderArray $holders
 	 * @return string HTML
 	 */
 	public function makeImage( $title, $options, $holders = false ) {
@@ -5731,7 +5736,7 @@ 
 	/**
 	 * @param string $caption
 	 * @param LinkHolderArray|bool $holders
-	 * @return mixed|string
+	 * @return string
 	 */
 	protected function stripAltText( $caption, $holders ) {
 		# Strip bad stuff out of the title (tooltip).  We can't just use
@@ -6134,7 +6139,7 @@ 
 	 * Accessor for $mDefaultSort
 	 * Unlike getDefaultSort(), will return false if none is set
 	 *
-	 * @return string|bool
+	 * @return false|string
 	 */
 	public function getCustomDefaultSort() {
 		return $this->mDefaultSort;