wikimedia / mediawiki

includes/debug/MWDebug.php

Doc Comments +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
 	 * Adds a line to the log
 	 *
 	 * @since 1.19
-	 * @param mixed $str
+	 * @param string $str
 	 */
 	public static function log( $str ) {
 		if ( !self::$enabled ) {

includes/jobqueue/JobQueue.php

Doc Comments +1 added lines, -1 removed lines

@@ -377,7 +377,7 @@
 
 	/**
 	 * @see JobQueue::pop()
-	 * @return Job|bool
+	 * @return Job
 	 */
 	abstract protected function doPop();
 

includes/session/CookieSessionProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -302,8 +302,8 @@
 	 * @param \WebRequest $request
 	 * @param string $key
 	 * @param string $prefix
-	 * @param mixed $default
-	 * @return mixed
+	 * @param boolean $default
+	 * @return string
 	 */
 	protected function getCookie( $request, $key, $prefix, $default = null ) {
 		$value = $request->getCookie( $key, $prefix, $default );

includes/search/SearchSuggestion.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,9 +55,9 @@
 	/**
 	 * Construct a new suggestion
 	 * @param float $score the suggestion score
-	 * @param string $text|null the suggestion text
 	 * @param Title|null $suggestedTitle the suggested title
 	 * @param int|null $suggestedTitleID the suggested title ID
+	 * @param string $text
 	 */
 	public function __construct( $score, $text = null, Title $suggestedTitle = null,
 			$suggestedTitleID = null ) {

includes/MagicWordArray.php

Doc Comments +3 added lines, -3 removed lines

@@ -771,7 +771,7 @@ 
 
 	/**
 	 * Get the base regex
-	 * @return array
+	 * @return string[]
 	 */
 	function getBaseRegex() {
 		if ( is_null( $this->baseRegex ) ) {
@@ -815,7 +815,7 @@ 
 	/**
 	 * Get a regex for matching variables with parameters
 	 *
-	 * @return string
+	 * @return string[]
 	 */
 	function getVariableRegex() {
 		return str_replace( "\\$1", "(.*?)", $this->getRegex() );
@@ -868,7 +868,7 @@ 
 	 * Returns array(magic word ID, parameter value)
 	 * If there is no parameter value, that element will be false.
 	 *
-	 * @param array $m
+	 * @param string[] $m
 	 *
 	 * @throws MWException
 	 * @return array

includes/logging/PatrolLog.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	/**
 	 * Record a log event for a change being patrolled
 	 *
-	 * @param int|RecentChange $rc Change identifier or RecentChange object
+	 * @param RecentChange $rc Change identifier or RecentChange object
 	 * @param bool $auto Was this patrol event automatic?
 	 * @param User $user User performing the action or null to use $wgUser
 	 * @param string|string[] $tags Change tags to add to the patrol log entry

maintenance/findHooks.php

Doc Comments +1 added lines, -1 removed lines

@@ -296,7 +296,7 @@
 	/**
 	 * Get hooks from a directory of PHP files.
 	 * @param string $dir Directory path to start at
-	 * @param int $recursive Pass self::FIND_RECURSIVE
+	 * @param int $recurse Pass self::FIND_RECURSIVE
 	 * @return array Array: key => hook name; value => array of arguments or string 'unknown'
 	 */
 	private function getHooksFromDir( $dir, $recurse = 0 ) {

includes/WatchedItemStore.php

Doc Comments +5 added lines

@@ -410,6 +410,11 @@
 		return true;
 	}
 
+	/**
+	 * @param WatchedItem|null $item
+	 * @param string $force
+	 * @param integer $oldid
+	 */
 	private function getNotificationTimestamp( User $user, Title $title, $item, $force, $oldid ) {
 		if ( !$oldid ) {
 			// No oldid given, assuming latest revision; clear the timestamp.

includes/filerepo/FileRepo.php

Doc Comments +5 added lines, -5 removed lines

@@ -220,7 +220,7 @@ 
 	 * Get an explanatory message if this repo is read-only.
 	 * This checks if an administrator disabled writes to the backend.
 	 *
-	 * @return string|bool Returns false if the repo is not read-only
+	 * @return string|false Returns false if the repo is not read-only
 	 */
 	public function getReadOnlyReason() {
 		return $this->backend->getReadOnlyReason();
@@ -716,7 +716,7 @@ 
 	 *
 	 * @param string $query Query string to append
 	 * @param string $entry Entry point; defaults to index
-	 * @return string|bool False on failure
+	 * @return string|false False on failure
 	 */
 	public function makeUrl( $query = '', $entry = 'index' ) {
 		if ( isset( $this->scriptDirUrl ) ) {
@@ -797,7 +797,7 @@ 
 	/**
 	 * Get the URL of the stylesheet to apply to description pages
 	 *
-	 * @return string|bool False on failure
+	 * @return string|false False on failure
 	 */
 	public function getDescriptionStylesheetUrl() {
 		if ( isset( $this->scriptDirUrl ) ) {
@@ -958,7 +958,7 @@ 
 	 *
 	 * @param string|FSFile $src Source file system path, storage path, or virtual URL
 	 * @param string $dst Virtual URL or storage path
-	 * @param array|string|null $options An array consisting of a key named headers
+	 * @param string $options An array consisting of a key named headers
 	 *   listing extra headers. If a string, taken as content-disposition header.
 	 *   (Support for array of options new in 1.23)
 	 * @return FileRepoStatus
@@ -1158,7 +1158,7 @@ 
 	 * Options to $options include:
 	 *   - headers : name/value map of HTTP headers to use in response to GET/HEAD requests
 	 *
-	 * @param string|FSFile $src The source file system path, storage path, or URL
+	 * @param string $src The source file system path, storage path, or URL
 	 * @param string $dstRel The destination relative path
 	 * @param string $archiveRel The relative path where the existing file is to
 	 *   be archived, if there is one. Relative to the public zone root.