wikimedia / mediawiki-extensions-Wikibase

repo/tests/phpunit/includes/EntityModificationTestHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -144,7 +144,7 @@
 	 * @param array $data
 	 * @param EntityId|string|null $id
 	 *
-	 * @return object
+	 * @return EntityDocument
 	 */
 	public function unserializeEntity( array $data, $id = null ) {
 		if ( $id !== null ) {

repo/includes/Interactors/ItemMergeInteractor.php

Doc Comments +4 added lines, -1 removed lines

@@ -270,7 +270,7 @@ 
 	 * @param string|null $summary
 	 * @param bool $bot
 	 *
-	 * @return array A list of exactly two EntityRevision objects. The first one represents the
+	 * @return \Wikibase\EntityRevision[] A list of exactly two EntityRevision objects. The first one represents the
 	 *  modified source item, the second one represents the modified target item.
 	 */
 	private function attemptSaveMerge( Item $fromItem, Item $toItem, $summary, $bot ) {
@@ -283,6 +283,9 @@ 
 		return array( $fromRev, $toRev );
 	}
 
+	/**
+	 * @param boolean $bot
+	 */
 	private function saveEntity( Entity $entity, Summary $summary, $bot ) {
 		// Given we already check all constraints in ChangeOpsMerge, it's
 		// fine to ignore them here. This is also needed to not run into

repo/tests/phpunit/includes/ChangeOp/ChangeOpsMergeTest.php

Doc Comments +3 added lines

@@ -144,6 +144,9 @@
 		);
 	}
 
+	/**
+	 * @param string $idString
+	 */
 	private function newItemWithId( $idString ) {
 		return new Item( new ItemId( $idString ) );
 	}

repo/includes/ChangeDispatcher.php

Doc Comments +3 added lines

@@ -451,6 +451,9 @@
 		}
 	}
 
+	/**
+	 * @param string $message
+	 */
 	private function log( $message ) {
 		$this->messageReporter->reportMessage( $message );
 	}

repo/includes/ParserOutput/EntityParserOutputDataUpdater.php

Doc Comments +3 added lines

@@ -56,6 +56,9 @@
 		$this->dataUpdaters = $dataUpdaters;
 	}
 
+	/**
+	 * @param ParserOutputDataUpdater $updater
+	 */
 	private function registerDataUpdater( $updater ) {
 		if ( !( $updater instanceof StatementDataUpdater ) &&
 		     !( $updater instanceof SiteLinkDataUpdater )

client/tests/phpunit/includes/Changes/WikiPageUpdaterTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
 	/**
 	 * @param string $text
 	 *
-	 * @return Title
+	 * @return EntityChange
 	 */
 	private function getTitleMock( $text ) {
 		$title = $this->getMockBuilder( 'Title' )

repo/includes/GenericEventDispatcher.php

Doc Comments +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 	 * @param object $listener
 	 *
 	 * @throws InvalidArgumentException
-	 * @return mixed The listener key, for removing the listener later.
+	 * @return integer The listener key, for removing the listener later.
 	 */
 	public function registerWatcher( $listener ) {
 		if ( !is_subclass_of( $listener, $this->interface ) ) {
@@ -59,7 +59,7 @@ 
 	 * Unregisters a watcher using its registration key. The watcher will no longer
 	 * be called by dispatch().
 	 *
-	 * @param mixed $key A watcher key as returned by registerWatcher().
+	 * @param integer $key A watcher key as returned by registerWatcher().
 	 *
 	 * @throws InvalidArgumentException
 	 */

client/includes/Specials/SpecialUnconnectedPages.php

Doc Comments +2 added lines, -2 removed lines

@@ -162,8 +162,8 @@
 	/**
 	 * @see QueryPage::reallyDoQuery
 	 *
-	 * @param int|bool $limit
-	 * @param int|bool $offset
+	 * @param integer $limit
+	 * @param integer $offset
 	 *
 	 * @return ResultWrapper
 	 */

repo/includes/Api/EditEntity.php

Doc Comments +1 added lines, -1 removed lines

@@ -632,7 +632,7 @@
 
 	/**
 	 * @param mixed $data
-	 * @param array $allowedProps
+	 * @param string[] $allowedProps
 	 */
 	private function checkValidJson( $data, array $allowedProps ) {
 		if ( is_null( $data ) ) {