wikimedia / mediawiki

languages/FakeConverter.php

Doc Comments +4 added lines, -1 removed lines

@@ -32,6 +32,9 @@ 
 	 */
 	public $mLang;
 
+	/**
+	 * @param Language $langobj
+	 */
 	function __construct( $langobj ) {
 		$this->mLang = $langobj;
 	}
@@ -54,7 +57,7 @@ 
 
 	/**
 	 * @param Title $t
-	 * @return mixed
+	 * @return string
 	 */
 	function convertTitle( $t ) {
 		return $t->getPrefixedText();

languages/LanguageConverter.php

Doc Comments +5 added lines, -5 removed lines

@@ -145,7 +145,7 @@ 
 
 	/**
 	 * Get the title produced by the conversion rule.
-	 * @return string The converted title text
+	 * @return boolean The converted title text
 	 */
 	public function getConvRuleTitle() {
 		return $this->mConvRuleTitle;
@@ -207,7 +207,7 @@ 
 	/**
 	 * Validate the variant
 	 * @param string $variant The variant to validate
-	 * @return mixed Returns the variant if it is valid, null otherwise
+	 * @return string|null Returns the variant if it is valid, null otherwise
 	 */
 	public function validateVariant( $variant = null ) {
 		if ( $variant !== null && in_array( $variant, $this->mVariants ) ) {
@@ -219,7 +219,7 @@ 
 	/**
 	 * Get the variant specified in the URL
 	 *
-	 * @return mixed Variant if one found, false otherwise.
+	 * @return string|null Variant if one found, false otherwise.
 	 */
 	public function getURLVariant() {
 		global $wgRequest;
@@ -242,7 +242,7 @@ 
 	/**
 	 * Determine if the user has a variant set.
 	 *
-	 * @return mixed Variant if one found, false otherwise.
+	 * @return string|null Variant if one found, false otherwise.
 	 */
 	protected function getUserVariant() {
 		global $wgUser, $wgContLang;
@@ -276,7 +276,7 @@ 
 	/**
 	 * Determine the language variant from the Accept-Language header.
 	 *
-	 * @return mixed Variant if one found, false otherwise.
+	 * @return string|null Variant if one found, false otherwise.
 	 */
 	protected function getHeaderVariant() {
 		global $wgRequest;

maintenance/archives/upgradeLogging.php

Doc Comments +4 added lines

@@ -178,6 +178,10 @@
 		echo "Copied $numRowsCopied rows\n";
 	}
 
+	/**
+	 * @param string $srcTable
+	 * @param string $dstTable
+	 */
 	function copyExactMatch( $srcTable, $dstTable, $copyPos ) {
 		$numRowsCopied = 0;
 		$srcRes = $this->dbw->select( $srcTable, '*', array( 'log_timestamp' => $copyPos ), __METHOD__ );

maintenance/benchmarks/bench_HTTP_HTTPS.php

Doc Comments +3 added lines

@@ -45,6 +45,9 @@
 		print $this->getFormattedResults();
 	}
 
+	/**
+	 * @param string $proto
+	 */
 	static function doRequest( $proto ) {
 		Http::get( "$proto://localhost/", array(), __METHOD__ );
 	}

maintenance/benchmarks/bench_strtr_str_replace.php

Doc Comments +6 added lines

@@ -25,10 +25,16 @@
 
 require_once __DIR__ . '/Benchmarker.php';
 
+/**
+ * @param string $str
+ */
 function bfNormalizeTitleStrTr( $str ) {
 	return strtr( $str, '_', ' ' );
 }
 
+/**
+ * @param string $str
+ */
 function bfNormalizeTitleStrReplace( $str ) {
 	return str_replace( '_', ' ', $str );
 }

maintenance/checkSyntax.php

Doc Comments +11 added lines

@@ -363,6 +363,12 @@ 
 		$this->checkRegex( $file, $text, '/^[\xFF\xFE\xEF]/', 'byte-order mark' );
 	}
 
+	/**
+	 * @param string $file
+	 * @param string $text
+	 * @param string $regex
+	 * @param string $desc
+	 */
 	private function checkRegex( $file, $text, $regex, $desc ) {
 		if ( !preg_match( $regex, $text ) ) {
 			return;
@@ -375,6 +381,11 @@ 
 		$this->output( "Warning in file $file: $desc found.\n" );
 	}
 
+	/**
+	 * @param string $file
+	 * @param string $evilToken
+	 * @param string $desc
+	 */
 	private function checkEvilToken( $file, $tokens, $evilToken, $desc ) {
 		if ( !in_array( $evilToken, $tokens ) ) {
 			return;

maintenance/compareParsers.php

Doc Comments +3 added lines

@@ -112,6 +112,9 @@
 		}
 	}
 
+	/**
+	 * @return string
+	 */
 	function stripParameters( $text ) {
 		if ( !$this->stripParametersEnabled ) {
 			return $text;

maintenance/convertExtensionToRegistration.php

Doc Comments +6 added lines

@@ -185,6 +185,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $dir
+	 */
 	private function stripPath( $val, $dir ) {
 		if ( $val === $dir ) {
 			$val = '';
@@ -266,6 +269,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $path
+	 */
 	protected function needsComposerAutoloader( $path ) {
 		$path .= '/composer.json';
 		if ( file_exists( $path ) ) {

maintenance/copyJobQueue.php

Doc Comments +3 added lines

@@ -70,6 +70,9 @@
 		}
 	}
 
+	/**
+	 * @param Iterator $jobs
+	 */
 	protected function copyJobs( JobQueue $src, JobQueue $dst, $jobs ) {
 		$total = 0;
 		$totalOK = 0;