wikimedia / mediawiki

includes/htmlform/HTMLForm.php

Doc Comments +1 added lines, -2 removed lines

@@ -250,7 +250,6 @@ 
 	 * Construct a HTMLForm object for given display type. May return a HTMLForm subclass.
 	 *
 	 * @param string $displayFormat
-	 * @param mixed $arguments... Additional arguments to pass to the constructor.
 	 * @return HTMLForm
 	 */
 	public static function factory( $displayFormat/*, $arguments...*/ ) {
@@ -1359,7 +1358,7 @@ 
 	 * this message as its "<legend>" element.
 	 * @since 1.19
 	 *
-	 * @param string|Message $msg Message key or Message object
+	 * @param string $msg Message key or Message object
 	 *
 	 * @return HTMLForm $this for chaining calls (since 1.20)
 	 */

includes/htmlform/OOUIHTMLForm.php

Doc Comments +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 	 * @param OOUI\FieldLayout[] $fieldsHtml
 	 * @param string $sectionName
 	 * @param bool $anyFieldHasLabel Unused
-	 * @return string HTML
+	 * @return OOUI\FieldsetLayout HTML
 	 */
 	protected function formatSection( array $fieldsHtml, $sectionName, $anyFieldHasLabel ) {
 		$config = array(

includes/HttpFunctions.php

Doc Comments +2 added lines, -2 removed lines

@@ -186,7 +186,7 @@ 
 	 * @todo FIXME this is wildly inaccurate and fails to actually check most stuff
 	 *
 	 * @param string $uri URI to check for validity
-	 * @return bool
+	 * @return integer
 	 */
 	public static function isValidURI( $uri ) {
 		return preg_match(
@@ -317,7 +317,7 @@ 
 	 * @param array $options (optional) extra params to pass (see Http::request())
 	 * @param string $caller The method making this request, for profiling
 	 * @throws MWException
-	 * @return CurlHttpRequest|PhpHttpRequest
+	 * @return string
 	 * @see MWHttpRequest::__construct
 	 */
 	public static function factory( $url, $options = null, $caller = __METHOD__ ) {

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