wikimedia / mediawiki

includes/import/UploadSourceAdapter.php

Doc Comments +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 	}
 
 	/**
-	 * @return mixed
+	 * @return integer
 	 */
 	function stream_tell() {
 		return $this->mPosition;

includes/import/WikiRevision.php

Doc Comments +5 added lines, -5 removed lines

@@ -384,14 +384,14 @@ 
 	}
 
 	/**
-	 * @return mixed
+	 * @return string
 	 */
 	function getSrc() {
 		return $this->src;
 	}
 
 	/**
-	 * @return bool|string
+	 * @return false|string
 	 */
 	function getSha1() {
 		if ( $this->sha1base36 ) {
@@ -415,7 +415,7 @@ 
 	}
 
 	/**
-	 * @return mixed
+	 * @return string
 	 */
 	function getFilename() {
 		return $this->filename;
@@ -429,7 +429,7 @@ 
 	}
 
 	/**
-	 * @return mixed
+	 * @return integer
 	 */
 	function getSize() {
 		return $this->size;
@@ -681,7 +681,7 @@ 
 	}
 
 	/**
-	 * @return bool|string
+	 * @return false|string
 	 */
 	function downloadSource() {
 		if ( !$this->config->get( 'EnableUploads' ) ) {

includes/installer/DatabaseInstaller.php

Doc Comments +3 added lines, -3 removed lines

@@ -393,7 +393,7 @@ 
 	/**
 	 * Get a variable, taking local defaults into account.
 	 * @param string $var
-	 * @param mixed|null $default
+	 * @param boolean $default
 	 * @return mixed
 	 */
 	public function getVar( $var, $default = null ) {
@@ -516,8 +516,8 @@ 
 	 * Convenience function to set variables based on form data.
 	 * Assumes that variables containing "password" in the name are (potentially
 	 * fake) passwords.
-	 * @param array $varNames
-	 * @return array
+	 * @param string[] $varNames
+	 * @return string[]
 	 */
 	public function setVarsFromRequest( $varNames ) {
 		return $this->parent->setVarsFromRequest( $varNames, $this->getName() . '_' );

includes/installer/InstallDocFormatter.php

Doc Comments +4 added lines

@@ -21,6 +21,10 @@
  */
 
 class InstallDocFormatter {
+
+	/**
+	 * @param string $text
+	 */
 	static function format( $text ) {
 		$obj = new self( $text );
 

includes/installer/Installer.php

Doc Comments +6 added lines, -6 removed lines

@@ -578,7 +578,7 @@ 
 	 * fake password in which case leave it as it is.
 	 *
 	 * @param string $name
-	 * @param mixed $value
+	 * @param string $value
 	 */
 	public function setPassword( $name, $value ) {
 		if ( !preg_match( '/^\*+$/', $value ) ) {
@@ -1266,7 +1266,7 @@ 
 	 *
 	 * If $versionInfo is not false, only executables with a version
 	 * matching $versionInfo[1] will be returned.
-	 * @return bool|string
+	 * @return string|false
 	 */
 	public static function locateExecutable( $path, $names, $versionInfo = false ) {
 		if ( !is_array( $names ) ) {
@@ -1298,14 +1298,14 @@ 
 	/**
 	 * Same as locateExecutable(), but checks in getPossibleBinPaths() by default
 	 * @see locateExecutable()
-	 * @param array $names Array of possible names.
-	 * @param array|bool $versionInfo Default: false or array with two members:
+	 * @param string[] $names Array of possible names.
+	 * @param string[] $versionInfo Default: false or array with two members:
 	 *   0 => Command to run for version check, with $1 for the full executable name
 	 *   1 => String to compare the output with
 	 *
 	 * If $versionInfo is not false, only executables with a version
 	 * matching $versionInfo[1] will be returned.
-	 * @return bool|string
+	 * @return string|false
 	 */
 	public static function locateExecutableInDefaultPaths( $names, $versionInfo = false ) {
 		foreach ( self::getPossibleBinPaths() as $path ) {
@@ -1324,7 +1324,7 @@ 
 	 * Used only by environment checks.
 	 * @param string $dir
 	 * @param string $url
-	 * @return bool|int|string
+	 * @return string|false
 	 */
 	public function dirIsExecutable( $dir, $url ) {
 		$scriptTypes = array(

includes/installer/MysqlUpdater.php

Doc Comments +1 added lines, -1 removed lines

@@ -289,7 +289,7 @@
 	 * @param string $table Table name
 	 * @param string $field Field name to check
 	 * @param string $patchFile Path to the patch to correct the field
-	 * @return bool
+	 * @return boolean|null
 	 */
 	protected function checkBin( $table, $field, $patchFile ) {
 		if ( !$this->doTable( $table ) ) {

includes/installer/PostgresUpdater.php

Doc Comments +6 added lines

@@ -546,6 +546,9 @@ 
 		return $colnames;
 	}
 
+	/**
+	 * @param string $fkey
+	 */
 	function fkeyDeltype( $fkey ) {
 		$q = <<<END
 SELECT confdeltype FROM pg_constraint, pg_namespace
@@ -726,6 +729,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $default
+	 */
 	protected function setDefault( $table, $field, $default ) {
 
 		$info = $this->db->fieldInfo( $table, $field );

includes/installer/WebInstaller.php

Doc Comments +2 added lines, -2 removed lines

@@ -388,7 +388,7 @@ 
 	/**
 	 * Show an error message in a box. Parameters are like wfMessage(), or
 	 * alternatively, pass a Message object in.
-	 * @param string|Message $msg
+	 * @param string $msg
 	 */
 	public function showError( $msg /*...*/ ) {
 		if ( !( $msg instanceof Message ) ) {
@@ -486,7 +486,7 @@ 
 	 * Set a session variable.
 	 *
 	 * @param string $name Key for the variable
-	 * @param mixed $value
+	 * @param boolean $value
 	 */
 	public function setSession( $name, $value ) {
 		$this->session[$name] = $value;

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