wikimedia / mediawiki

includes/filebackend/FileBackendStore.php

Doc Comments +4 added lines, -4 removed lines

@@ -753,7 +753,7 @@ 
 	/**
 	 * @see FileBackendStore::getFileSha1Base36()
 	 * @param array $params
-	 * @return bool|string
+	 * @return false|string
 	 */
 	protected function doGetFileSha1Base36( array $params ) {
 		$fsFile = $this->getLocalReference( $params );
@@ -1190,7 +1190,7 @@ 
 	 * @param FileBackendStoreOpHandle[] $fileOpHandles
 	 *
 	 * @throws FileBackendError
-	 * @return array Map of Status objects
+	 * @return Status[] Map of Status objects
 	 */
 	final public function executeOpHandlesInternal( array $fileOpHandles ) {
 		$ps = Profiler::instance()->scopedProfileIn( __METHOD__ . "-{$this->name}" );
@@ -1578,7 +1578,7 @@ 
 	 * Subclasses can override this to be more restrictive.
 	 *
 	 * @param string $container
-	 * @return string|null
+	 * @return string
 	 */
 	protected function resolveContainerName( $container ) {
 		return $container;
@@ -1592,7 +1592,7 @@ 
 	 *
 	 * @param string $container Container name
 	 * @param string $relStoragePath Storage path relative to the container
-	 * @return string|null Path or null if not valid
+	 * @return string Path or null if not valid
 	 */
 	protected function resolveContainerPath( $container, $relStoragePath ) {
 		return $relStoragePath;

includes/filebackend/filejournal/FileJournal.php

Doc Comments +2 added lines, -2 removed lines

@@ -216,7 +216,7 @@ 
 
 	/**
 	 * @see FileJournal::doGetCurrentPosition()
-	 * @return int|bool
+	 * @return boolean
 	 */
 	protected function doGetCurrentPosition() {
 		return false;
@@ -225,7 +225,7 @@ 
 	/**
 	 * @see FileJournal::doGetPositionAtTime()
 	 * @param int|string $time Timestamp
-	 * @return int|bool
+	 * @return boolean
 	 */
 	protected function doGetPositionAtTime( $time ) {
 		return false;

includes/filebackend/FileOp.php

Doc Comments +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
 	 * Get the value of the parameter with the given name
 	 *
 	 * @param string $name
-	 * @return mixed Returns null if the parameter is not set
+	 * @return string Returns null if the parameter is not set
 	 */
 	final public function getParam( $name ) {
 		return isset( $this->params[$name] ) ? $this->params[$name] : null;

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 ) {