wikimedia / mediawiki

includes/jobqueue/aggregator/JobQueueAggregatorRedis.php

Doc Comments +2 added lines, -2 removed lines

@@ -98,7 +98,7 @@ 
 	/**
 	 * Get a connection to the server that handles all sub-queues for this queue
 	 *
-	 * @return RedisConnRef|bool Returns false on failure
+	 * @return boolean Returns false on failure
 	 * @throws MWException
 	 */
 	protected function getConnection() {
@@ -122,7 +122,7 @@ 
 
 	/**
 	 * @param string $name
-	 * @return string
+	 * @return string[]
 	 */
 	private function decodeQueueName( $name ) {
 		list( $type, $wiki ) = explode( '/', $name, 2 );

includes/jobqueue/Job.php

Doc Comments +3 added lines, -3 removed lines

@@ -130,7 +130,7 @@ 
 	}
 
 	/**
-	 * @return int|null UNIX timestamp to delay running this job until, otherwise null
+	 * @return string|null UNIX timestamp to delay running this job until, otherwise null
 	 * @since 1.22
 	 */
 	public function getReleaseTimestamp() {
@@ -140,7 +140,7 @@ 
 	}
 
 	/**
-	 * @return int|null UNIX timestamp of when the job was queued, or null
+	 * @return string|null UNIX timestamp of when the job was queued, or null
 	 * @since 1.26
 	 */
 	public function getQueuedTimestamp() {
@@ -150,7 +150,7 @@ 
 	}
 
 	/**
-	 * @return int|null UNIX timestamp of when the job was runnable, or null
+	 * @return string|null UNIX timestamp of when the job was runnable, or null
 	 * @since 1.26
 	 */
 	public function getReadyTimestamp() {

includes/jobqueue/JobQueueGroup.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
 	 * This pops a job off a queue as specified by $wgJobTypeConf and
 	 * updates the aggregate job queue information cache as needed.
 	 *
-	 * @param int|string $qtype JobQueueGroup::TYPE_* constant or job type string
+	 * @param integer $qtype JobQueueGroup::TYPE_* constant or job type string
 	 * @param int $flags Bitfield of JobQueueGroup::USE_* constants
 	 * @param array $blacklist List of job types to ignore
 	 * @return Job|bool Returns false on failure

includes/jobqueue/JobQueueRedis.php

Doc Comments +3 added lines, -3 removed lines

@@ -378,7 +378,7 @@ 
 	/**
 	 * @see JobQueue::doAck()
 	 * @param Job $job
-	 * @return Job|bool
+	 * @return boolean
 	 * @throws UnexpectedValueException
 	 * @throws JobQueueError
 	 */
@@ -480,7 +480,7 @@ 
 
 	/**
 	 * @see JobQueue::doDelete()
-	 * @return bool
+	 * @return boolean|null
 	 * @throws JobQueueError
 	 */
 	protected function doDelete() {
@@ -693,7 +693,7 @@ 
 
 	/**
 	 * @param array $fields
-	 * @return Job|bool
+	 * @return Job
 	 */
 	protected function getJobFromFields( array $fields ) {
 		$title = Title::makeTitle( $fields['namespace'], $fields['title'] );

includes/jobqueue/jobs/ThumbnailRenderJob.php

Doc Comments +4 added lines

@@ -77,6 +77,10 @@
 		}
 	}
 
+	/**
+	 * @param File $file
+	 * @param string $thumbUrl
+	 */
 	protected function hitThumbUrl( $file, $transformParams, &$thumbUrl ) {
 		global $wgUploadThumbnailRenderHttpCustomHost, $wgUploadThumbnailRenderHttpCustomDomain;
 

includes/libs/ArrayUtils.php

Doc Comments -1 removed lines

@@ -151,7 +151,6 @@
 	 * @since 1.23
 	 *
 	 * @param array $array1 The array to compare from
-	 * @param array $array2,... More arrays to compare against
 	 * @return array An array containing all the values from array1
 	 *               that are not present in any of the other arrays.
 	 */

includes/libs/BufferingStatsdDataFactory.php

Doc Comments +3 added lines

@@ -35,6 +35,9 @@
 class BufferingStatsdDataFactory extends StatsdDataFactory {
 	protected $buffer = array();
 
+	/**
+	 * @param string $prefix
+	 */
 	public function __construct( $prefix ) {
 		parent::__construct();
 		$this->prefix = $prefix;

includes/libs/Cookie.php

Doc Comments +4 added lines

@@ -33,6 +33,10 @@
 	// TO IMPLEMENT? protected $version
 	// TO IMPLEMENT? protected $comment
 
+	/**
+	 * @param string $name
+	 * @param string $value
+	 */
 	function __construct( $name, $value, $attr ) {
 		$this->name = $name;
 		$this->set( $value, $attr );

includes/libs/CSSMin.php

Doc Comments +3 added lines, -3 removed lines

@@ -128,7 +128,7 @@ 
 	 * @param bool $ie8Compat By default, a data URI will only be produced if it can be made short
 	 *     enough to fit in Internet Explorer 8 (and earlier) URI length limit (32,768 bytes). Pass
 	 *     `false` to remove this limitation.
-	 * @return string|bool Image contents encoded as a data URI or false.
+	 * @return false|string Image contents encoded as a data URI or false.
 	 */
 	public static function encodeImageAsDataURI( $file, $type = null, $ie8Compat = true ) {
 		// Fast-fail for files that definitely exceed the maximum data URI length
@@ -156,7 +156,7 @@ 
 	 * @param string $contents File contents to encode.
 	 * @param string $type File's MIME type.
 	 * @param bool $ie8Compat See encodeImageAsDataURI().
-	 * @return string|bool Image contents encoded as a data URI or false.
+	 * @return string|false Image contents encoded as a data URI or false.
 	 */
 	public static function encodeStringAsDataURI( $contents, $type, $ie8Compat = true ) {
 		// Try #1: Non-encoded data URI
@@ -200,7 +200,7 @@ 
 	}
 
 	/**
-	 * @param $file string
+	 * @param string $file string
 	 * @return bool|string
 	 */
 	public static function getMimeType( $file ) {