wikimedia / mediawiki

includes/PrefixSearch.php

Doc Comments +5 added lines, -1 removed lines

@@ -131,7 +131,7 @@ 
 	 * When implemented in a descendant class, receives an array of titles as strings and returns
 	 * either an unmodified array or an array of Title objects corresponding to strings received.
 	 *
-	 * @param array $strings
+	 * @param string[] $strings
 	 *
 	 * @return array
 	 */
@@ -164,6 +164,10 @@ 
 		return $this->strings( $this->handleResultFromHook( $srchres, $namespaces, $search, $limit ) );
 	}
 
+	/**
+	 * @param string $search
+	 * @param integer $limit
+	 */
 	private function handleResultFromHook( $srchres, $namespaces, $search, $limit ) {
 		$rescorer = new SearchExactMatchRescorer();
 		return $rescorer->rescore( $search, $namespaces, $srchres, $limit );

includes/profiler/TransactionProfiler.php

Doc Comments +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 	 * With conflicting expectations, the most narrow ones will be used
 	 *
 	 * @param array $expects Map of (event => limit)
-	 * @param $fname
+	 * @param string $fname
 	 * @since 1.26
 	 */
 	public function setExpectations( array $expects, $fname ) {

includes/resourceloader/ResourceLoader.php

Doc Comments +6 added lines, -2 removed lines

@@ -207,6 +207,10 @@ 
 		return $result;
 	}
 
+	/**
+	 * @param string $filter
+	 * @param string $data
+	 */
 	private static function applyFilter( $filter, $data ) {
 		$data = trim( $data );
 		if ( $data ) {
@@ -309,7 +313,7 @@ 
 	 * @throws MWException If a duplicate module registration is attempted
 	 * @throws MWException If a module name contains illegal characters (pipes or commas)
 	 * @throws MWException If something other than a ResourceLoaderModule is being registered
-	 * @return bool False if there were any errors, in which case one or more modules were
+	 * @return boolean|null False if there were any errors, in which case one or more modules were
 	 *   not registered
 	 */
 	public function register( $name, $info = null ) {
@@ -572,7 +576,7 @@ 
 	/**
 	 * Get the list of sources.
 	 *
-	 * @return array Like array( id => load.php url, .. )
+	 * @return string Like array( id => load.php url, .. )
 	 */
 	public function getSources() {
 		return $this->sources;

includes/resourceloader/ResourceLoaderFileModule.php

Doc Comments +2 added lines, -2 removed lines

@@ -382,7 +382,7 @@ 
 	 * Get all styles for a given context.
 	 *
 	 * @param ResourceLoaderContext $context
-	 * @return array CSS code for $context as an associative array mapping media type to CSS text.
+	 * @return string CSS code for $context as an associative array mapping media type to CSS text.
 	 */
 	public function getStyles( ResourceLoaderContext $context ) {
 		$styles = $this->readStyleFiles(
@@ -629,7 +629,7 @@ 
 	 * @param array $list List of file paths in any combination of index/path
 	 *     or path/options pairs
 	 * @param string $option Option name
-	 * @param mixed $default Default value if the option isn't set
+	 * @param string $default Default value if the option isn't set
 	 * @return array List of file paths, collated by $option
 	 */
 	protected static function collateFilePathListByOption( array $list, $option, $default ) {

includes/resourceloader/ResourceLoaderImage.php

Doc Comments +1 added lines, -1 removed lines

@@ -312,7 +312,7 @@
 	 * Convert passed image data, which is assumed to be SVG, to PNG.
 	 *
 	 * @param string $svg SVG image data
-	 * @return string|bool PNG image data, or false on failure
+	 * @return string|false PNG image data, or false on failure
 	 */
 	protected function rasterize( $svg ) {
 		/**

includes/resourceloader/ResourceLoaderWikiModule.php

Doc Comments +2 added lines, -2 removed lines

@@ -135,14 +135,14 @@
 	 * NOTE: This ONLY works for getTitleInfo() and isKnownEmpty(), NOT FOR ANYTHING ELSE.
 	 * In particular, it doesn't work for getContent() or getScript() etc.
 	 *
-	 * @return IDatabase|null
+	 * @return DatabaseBase
 	 */
 	protected function getDB() {
 		return wfGetDB( DB_SLAVE );
 	}
 
 	/**
-	 * @param string $title
+	 * @param string $titleText
 	 * @return null|string
 	 */
 	protected function getContent( $titleText ) {

includes/Revision.php

Doc Comments +10 added lines, -10 removed lines

@@ -423,7 +423,7 @@ 
 	/**
 	 * Return the list of revision fields that should be selected to create
 	 * a new revision.
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectFields() {
 		global $wgContentHandlerUseDB;
@@ -454,7 +454,7 @@ 
 	/**
 	 * Return the list of revision fields that should be selected to create
 	 * a new revision from an archive row.
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectArchiveFields() {
 		global $wgContentHandlerUseDB;
@@ -485,7 +485,7 @@ 
 	/**
 	 * Return the list of text fields that should be selected to read the
 	 * revision text
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectTextFields() {
 		return array(
@@ -496,7 +496,7 @@ 
 
 	/**
 	 * Return the list of page fields that should be selected from page table
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectPageFields() {
 		return array(
@@ -511,7 +511,7 @@ 
 
 	/**
 	 * Return the list of user fields that should be selected from user table
-	 * @return array
+	 * @return string[]
 	 */
 	public static function selectUserFields() {
 		return array( 'user_name' );
@@ -756,7 +756,7 @@ 
 	/**
 	 * Returns the base36 sha1 of the text in this revision, or null if unknown.
 	 *
-	 * @return string|null
+	 * @return string
 	 */
 	public function getSha1() {
 		return $this->mSha1;
@@ -838,7 +838,7 @@ 
 	/**
 	 * Fetch revision's user id without regard for the current user's permissions
 	 *
-	 * @return string
+	 * @return integer
 	 * @deprecated since 1.25, use getUser( Revision::RAW )
 	 */
 	public function getRawUser() {
@@ -1533,7 +1533,7 @@ 
 	 * Lazy-load the revision's text.
 	 * Currently hardcoded to the 'text' table storage engine.
 	 *
-	 * @return string|bool The revision's text, or false on failure
+	 * @return string The revision's text, or false on failure
 	 */
 	protected function loadText() {
 		// Caching may be beneficial for massive use of external storage
@@ -1734,7 +1734,7 @@ 
 	 *
 	 * @param Title $title
 	 * @param int $id
-	 * @return string|bool False if not found
+	 * @return string|false False if not found
 	 */
 	static function getTimestampFromId( $title, $id, $flags = 0 ) {
 		$db = ( $flags & self::READ_LATEST )
@@ -1790,7 +1790,7 @@ 
 	 * @since 1.20
 	 * @deprecated since 1.24
 	 *
-	 * @param IDatabase|int $db The Database to perform the check on. May be given as a
+	 * @param integer $db The Database to perform the check on. May be given as a
 	 *        Database object or a database identifier usable with wfGetDB.
 	 * @param int $pageId The ID of the page in question
 	 * @param int $userId The ID of the user in question

includes/revisiondelete/RevDelArchivedFileItem.php

Doc Comments +4 added lines

@@ -23,6 +23,10 @@
  * Item class for a filearchive table row
  */
 class RevDelArchivedFileItem extends RevDelFileItem {
+
+	/**
+	 * @param RevDelArchivedFileList $list
+	 */
 	public function __construct( $list, $row ) {
 		RevDelItem::__construct( $list, $row );
 		$this->file = ArchivedFile::newFromRow( $row );

includes/revisiondelete/RevDelFileItem.php

Doc Comments +3 added lines

@@ -26,6 +26,9 @@
 	/** @var File */
 	public $file;
 
+	/**
+	 * @param RevDelFileList $list
+	 */
 	public function __construct( $list, $row ) {
 		parent::__construct( $list, $row );
 		$this->file = RepoGroup::singleton()->getLocalRepo()->newFileFromRow( $row );