wikimedia / mediawiki

includes/filebackend/FSFile.php

Doc Comments +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
 	/**
 	 * Get the file size in bytes
 	 *
-	 * @return int|bool
+	 * @return integer
 	 */
 	public function getSize() {
 		return filesize( $this->path );
@@ -72,7 +72,7 @@ 
 	/**
 	 * Get the file's last-modified timestamp
 	 *
-	 * @return string|bool TS_MW timestamp or false on failure
+	 * @return string|false TS_MW timestamp or false on failure
 	 */
 	public function getTimestamp() {
 		MediaWiki\suppressWarnings();
@@ -195,7 +195,7 @@ 
 	 * fairly neatly.
 	 *
 	 * @param bool $recache
-	 * @return bool|string False on failure
+	 * @return string|false False on failure
 	 */
 	public function getSha1Base36( $recache = false ) {
 
@@ -248,7 +248,7 @@ 
 	 * fairly neatly.
 	 *
 	 * @param string $path
-	 * @return bool|string False on failure
+	 * @return string|false False on failure
 	 */
 	public static function getSha1Base36FromPath( $path ) {
 		$fsFile = new self( $path );

includes/filebackend/SwiftFileBackend.php

Doc Comments +5 added lines, -5 removed lines

@@ -1290,7 +1290,7 @@ 
 	 * (lists are truncated to 10000 item with no way to page), and is just a performance risk.
 	 *
 	 * @param string $container Resolved Swift container
-	 * @param array $readGrps List of the possible criteria for a request to have
+	 * @param string[] $readGrps List of the possible criteria for a request to have
 	 * access to read a container. Each item is one of the following formats:
 	 *   - account:user        : Grants access if the request is by the given user
 	 *   - ".r:<regex>"        : Grants access if the request is from a referrer host that
@@ -1298,7 +1298,7 @@ 
 	 *                           Setting this to '*' effectively makes a container public.
 	 *   -".rlistings:<regex>" : Grants access if the request is from a referrer host that
 	 *                           matches the expression and the request is for a listing.
-	 * @param array $writeGrps A list of the possible criteria for a request to have
+	 * @param string[] $writeGrps A list of the possible criteria for a request to have
 	 * access to write to a container. Each item is of the following format:
 	 *   - account:user       : Grants access if the request is by the given user
 	 * @return Status
@@ -1681,7 +1681,7 @@ 
 	 * @param array $creds From getAuthentication()
 	 * @param string $container
 	 * @param string $object
-	 * @return array
+	 * @return string
 	 */
 	protected function storageUrl( array $creds, $container = null, $object = null ) {
 		$parts = array( $creds['storage_url'] );
@@ -1880,7 +1880,7 @@ 
 class SwiftFileBackendDirList extends SwiftFileBackendList {
 	/**
 	 * @see Iterator::current()
-	 * @return string|bool String (relative path) or false
+	 * @return string String (relative path) or false
 	 */
 	public function current() {
 		return substr( current( $this->bufferIter ), $this->suffixStart, -1 );
@@ -1897,7 +1897,7 @@ 
 class SwiftFileBackendFileList extends SwiftFileBackendList {
 	/**
 	 * @see Iterator::current()
-	 * @return string|bool String (relative path) or false
+	 * @return string String (relative path) or false
 	 */
 	public function current() {
 		list( $path, $stat ) = current( $this->bufferIter );

includes/filerepo/file/ArchivedFile.php

Doc Comments +2 added lines, -2 removed lines

@@ -215,7 +215,7 @@ 
 
 	/**
 	 * Fields in the filearchive table
-	 * @return array
+	 * @return string[]
 	 */
 	static function selectFields() {
 		return array(
@@ -421,7 +421,7 @@ 
 	/**
 	 * Returns the number of pages of a multipage document, or false for
 	 * documents which aren't multipage documents
-	 * @return bool|int
+	 * @return string|boolean
 	 */
 	function pageCount() {
 		if ( !isset( $this->pageCount ) ) {

includes/filerepo/file/ForeignAPIFile.php

Doc Comments +5 added lines, -5 removed lines

@@ -33,7 +33,7 @@ 
 	protected $repoClass = 'ForeignApiRepo';
 
 	/**
-	 * @param Title|string|bool $title
+	 * @param Title|null $title
 	 * @param ForeignApiRepo $repo
 	 * @param array $info
 	 * @param bool $exists
@@ -151,7 +151,7 @@ 
 
 	/**
 	 * @param int $page
-	 * @return int|number
+	 * @return integer
 	 */
 	public function getWidth( $page = 1 ) {
 		return isset( $this->mInfo['width'] ) ? intval( $this->mInfo['width'] ) : 0;
@@ -166,7 +166,7 @@ 
 	}
 
 	/**
-	 * @return bool|null|string
+	 * @return string|null
 	 */
 	public function getMetadata() {
 		if ( isset( $this->mInfo['metadata'] ) ) {
@@ -205,7 +205,7 @@ 
 	}
 
 	/**
-	 * @return bool|int|null
+	 * @return integer|null
 	 */
 	public function getSize() {
 		return isset( $this->mInfo['size'] ) ? intval( $this->mInfo['size'] ) : null;
@@ -268,7 +268,7 @@ 
 	}
 
 	/**
-	 * @return bool|string
+	 * @return string|false
 	 */
 	function getTimestamp() {
 		return wfTimestamp( TS_MW,

includes/filerepo/file/OldLocalFile.php

Doc Comments +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
 
 	/**
 	 * Fields in the oldimage table
-	 * @return array
+	 * @return string[]
 	 */
 	static function selectFields() {
 		return array(
@@ -232,7 +232,7 @@ 
 
 	/**
 	 * @param string $prefix
-	 * @return array
+	 * @return string[]
 	 */
 	function getCacheFields( $prefix = 'img_' ) {
 		$fields = parent::getCacheFields( $prefix );

includes/filerepo/file/UnregisteredLocalFile.php

Doc Comments +2 added lines, -2 removed lines

@@ -178,7 +178,7 @@ 
 	}
 
 	/**
-	 * @return bool
+	 * @return boolean|string
 	 */
 	function getMetadata() {
 		if ( !isset( $this->metadata ) ) {
@@ -193,7 +193,7 @@ 
 	}
 
 	/**
-	 * @return bool|string
+	 * @return string|false
 	 */
 	function getURL() {
 		if ( $this->repo ) {

includes/filerepo/ForeignAPIRepo.php

Doc Comments +2 added lines, -2 removed lines

@@ -211,7 +211,7 @@ 
 	}
 
 	/**
-	 * @param array $data
+	 * @param string $data
 	 * @return bool|array
 	 */
 	function getImageInfo( $data ) {
@@ -546,7 +546,7 @@ 
 	 * @param string $target Used in cache key creation, mostly
 	 * @param array $query The query parameters for the API request
 	 * @param int $cacheTTL Time to live for the memcached caching
-	 * @return null
+	 * @return string
 	 */
 	public function httpGetCached( $target, $query, $cacheTTL = 3600 ) {
 		if ( $this->mApiBase ) {

includes/filerepo/ForeignDBViaLBRepo.php

Doc Comments +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 	 * Get a key on the primary cache for this repository.
 	 * Returns false if the repository's cache is not accessible at this site.
 	 * The parameters are the parts of the key, as for wfMemcKey().
-	 * @return bool|string
+	 * @return string|false
 	 */
 	function getSharedCacheKey( /*...*/ ) {
 		if ( $this->hasSharedCache() ) {

includes/filerepo/RepoGroup.php

Doc Comments +2 added lines, -2 removed lines

@@ -105,7 +105,7 @@ 
 	 * Search repositories for an image.
 	 * You can also use wfFindFile() to do this.
 	 *
-	 * @param Title|string $title Title object or string
+	 * @param string $title Title object or string
 	 * @param array $options Associative array of options:
 	 *   time:           requested time for an archived image, or false for the
 	 *                   current version. An image object will be returned which was
@@ -315,7 +315,7 @@ 
 	/**
 	 * Get the repo instance with a given key.
 	 * @param string|int $index
-	 * @return bool|LocalRepo
+	 * @return string
 	 */
 	function getRepo( $index ) {
 		if ( !$this->reposInitialised ) {