wikimedia / mediawiki

includes/media/SVGMetadataExtractor.php

Doc Comments +5 added lines, -1 removed lines

@@ -29,6 +29,10 @@ 
  * @ingroup Media
  */
 class SVGMetadataExtractor {
+
+	/**
+	 * @param string $filename
+	 */
 	static function getMetadata( $filename ) {
 		$svg = new SVGReader( $filename );
 
@@ -372,7 +376,7 @@ 
 	 * http://www.w3.org/TR/SVG11/coords.html#UnitIdentifiers
 	 *
 	 * @param string $length CSS/SVG length.
-	 * @param float|int $viewportSize Optional scale for percentage units...
+	 * @param integer $viewportSize Optional scale for percentage units...
 	 * @return float Length in pixels
 	 */
 	static function scaleSVGUnit( $length, $viewportSize = 512 ) {

includes/media/TransformationalImageHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -287,7 +287,7 @@ 
 	/**
 	 * Get the source file for the transform
 	 *
-	 * @param $file File
+	 * @param File $file File
 	 * @param $params Array
 	 * @return Array Array with keys  width, height and path.
 	 */
@@ -562,7 +562,7 @@ 
 	 * @param array $params Rotate parameters.
 	 *   'rotation' clockwise rotation in degrees, allowed are multiples of 90
 	 * @since 1.24 Is non-static. From 1.21 it was static
-	 * @return bool
+	 * @return MediaTransformError
 	 */
 	public function rotate( $file, $params ) {
 		return new MediaTransformError( 'thumbnail_error', 0, 0,

includes/media/WebP.php

Doc Comments +4 added lines, -2 removed lines

@@ -43,6 +43,9 @@ 
 	const VP8X_XMP = 4;
 	const VP8X_ANIM = 2;
 
+	/**
+	 * @param string $filename
+	 */
 	public function getMetadata( $image, $filename ) {
 		$parsedWebPData = self::extractMetadata( $filename );
 		if ( !$parsedWebPData ) {
@@ -86,7 +89,6 @@ 
 	/**
 	 * Extracts the image size and WebP type from a file
 	 *
-	 * @param string $chunks Chunks as extracted by RiffExtractor
 	 * @return array|bool Header data array with entries 'compression', 'width' and 'height',
 	 * where 'compression' can be 'lossy', 'lossless', 'animated' or 'unknown'. False if
 	 * file is not a valid WebP file.
@@ -289,7 +291,7 @@ 
 	 * @param $ext
 	 * @param $mime
 	 * @param $params
-	 * @return array
+	 * @return string[]
 	 */
 	public function getThumbType( $ext, $mime, $params = null ) {
 		return array( 'png', 'image/png' );

includes/media/XMP.php

Doc Comments +1 added lines, -1 removed lines

@@ -491,7 +491,7 @@
 	 * <exif:DigitalZoomRatio>0/10</exif:DigitalZoomRatio>
 	 * and are processing the 0/10 bit.
 	 *
-	 * @param XMLParser $parser XMLParser reference to the xml parser
+	 * @param resource $parser XMLParser reference to the xml parser
 	 * @param string $data Character data
 	 * @throws RuntimeException On invalid data
 	 */

includes/MimeMagic.php

Doc Comments +1 added lines, -1 removed lines

@@ -620,7 +620,7 @@
 	 * @todo Remove $ext param
 	 *
 	 * @param string $file
-	 * @param mixed $ext
+	 * @param boolean|string $ext
 	 * @return bool|string
 	 * @throws MWException
 	 */

includes/MovePage.php

Doc Comments +3 added lines

@@ -42,6 +42,9 @@
 		$this->newTitle = $newTitle;
 	}
 
+	/**
+	 * @param string $reason
+	 */
 	public function checkPermissions( User $user, $reason ) {
 		$status = new Status();
 

includes/objectcache/ObjectCache.php

Doc Comments +1 added lines, -1 removed lines

@@ -262,7 +262,7 @@
 	}
 
 	/**
-	 * @param array $params [optional] Array key 'fallback' for $fallback.
+	 * @param integer $params [optional] Array key 'fallback' for $fallback.
 	 * @param int|string $fallback Fallback cache, e.g. (CACHE_NONE, "hash") (since 1.24)
 	 * @return BagOStuff
 	 * @deprecated 1.27

includes/objectcache/RedisBagOStuff.php

Doc Comments +2 added lines, -2 removed lines

@@ -307,7 +307,7 @@ 
 
 	/**
 	 * @param string $data
-	 * @return mixed
+	 * @return boolean|string
 	 */
 	protected function unserialize( $data ) {
 		return ctype_digit( $data ) ? intval( $data ) : unserialize( $data );
@@ -316,7 +316,7 @@ 
 	/**
 	 * Get a Redis object with a connection suitable for fetching the specified key
 	 * @param string $key
-	 * @return array (server, RedisConnRef) or (false, false)
+	 * @return false[] (server, RedisConnRef) or (false, false)
 	 */
 	protected function getConnection( $key ) {
 		$candidates = array_keys( $this->serverTagMap );

includes/page/Article.php

Doc Comments +7 added lines, -7 removed lines

@@ -95,7 +95,7 @@ 
 	/**
 	 * Constructor from a page id
 	 * @param int $id Article ID to load
-	 * @return Article|null
+	 * @return Article
 	 */
 	public static function newFromID( $id ) {
 		$t = Title::newFromID( $id );
@@ -353,7 +353,7 @@ 
 	 * @note Code that wants to retrieve page content from the database should
 	 * use WikiPage::getContent().
 	 *
-	 * @return Content|null|bool
+	 * @return null|Content
 	 *
 	 * @since 1.21
 	 */
@@ -441,7 +441,7 @@ 
 	 * on failure.
 	 *
 	 * @since 1.19
-	 * @return Revision|null
+	 * @return Revision
 	 */
 	public function getRevisionFetched() {
 		$this->fetchContentObject();
@@ -1531,7 +1531,7 @@ 
 	 * Chances are you should just be using the ParserOutput from
 	 * WikitextContent::getParserOutput instead of calling this for redirects.
 	 *
-	 * @param Title|array $target Destination(s) to redirect
+	 * @param Title $target Destination(s) to redirect
 	 * @param bool $appendSubtitle [optional]
 	 * @param bool $forceKnown Should the image be shown as a bluelink regardless of existence?
 	 * @return string Containing HTML with redirect link
@@ -2119,7 +2119,7 @@ 
 	 * @param string $reason
 	 * @param int $cascade
 	 * @param array $expiry
-	 * @return bool
+	 * @return Status
 	 */
 	public function updateRestrictions( $limit = array(), $reason = '',
 		&$cascade = 0, $expiry = array()
@@ -2176,7 +2176,7 @@ 
 
 	/**
 	 * @param bool $hasHistory
-	 * @return mixed
+	 * @return false|string
 	 */
 	public function generateReason( &$hasHistory ) {
 		$title = $this->mPage->getTitle();
@@ -2187,7 +2187,7 @@ 
 	// ****** B/C functions for static methods ( __callStatic is PHP>=5.3 ) ****** //
 
 	/**
-	 * @return array
+	 * @return string[]
 	 *
 	 * @deprecated since 1.24, use WikiPage::selectFields() instead
 	 */