wikimedia / mediawiki-extensions-Wikibase

repo/tests/phpunit/includes/Content/EntityContentTest.php

Doc Comments +7 added lines

@@ -87,6 +87,10 @@ 
 		$this->assertSame( $expected, $entityContent->getTextForSearchIndex() );
 	}
 
+	/**
+	 * @param string $languageCode
+	 * @param string $text
+	 */
 	private function setLabel( EntityDocument $entity, $languageCode, $text ) {
 		if ( !( $entity instanceof LabelsProvider ) ) {
 			throw new InvalidArgumentException( '$entity must be a LabelsProvider' );
@@ -408,6 +412,9 @@ 
 		$this->assertDataUpdates( $updates );
 	}
 
+	/**
+	 * @param \Wikibase\Repo\Content\DataUpdateAdapter[] $updates
+	 */
 	private function assertDataUpdates( $updates ) {
 		$this->assertInternalType( 'array', $updates );
 		$this->assertContainsOnlyInstancesOf( 'DataUpdate', $updates );

client/includes/OtherProjectsSitesGenerator.php

Doc Comments +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
 	}
 
 	/**
-	 * @return Site
+	 * @return string
 	 */
 	private function getLocalSite() {
 		return $this->siteLookup->getSite( $this->localSiteId );

client/includes/WikibaseClient.php

Doc Comments +3 added lines

@@ -1307,6 +1307,9 @@
 		);
 	}
 
+	/**
+	 * @param string $name
+	 */
 	private function fixLegacyContentModelSetting( array $setting, $name ) {
 		if ( isset( $setting[ 'wikibase-item' ] ) || isset( $setting[ 'wikibase-property' ] ) ) {
 			wfWarn( "The specified value for the Wikibase setting '$name' uses content model ids as keys. This is deprecated. " .

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

Doc Comments +1 added lines, -1 removed lines

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

client/tests/phpunit/includes/DataAccess/Scribunto/Scribunto_LuaWikibaseLibraryTest.php

Doc Comments +3 added lines

@@ -206,6 +206,9 @@
 		$this->assertFalse( $cacheSplit );
 	}
 
+	/**
+	 * @return RepoLinker
+	 */
 	private function getRepoLinker() {
 		$repoLinker = $this->getMockBuilder( RepoLinker::class )
 			->disableOriginalConstructor()

client/tests/phpunit/includes/DataAccess/Scribunto/WikibaseLanguageIndependentLuaBindingsTest.php

Doc Comments +3 added lines

@@ -75,6 +75,9 @@
 		);
 	}
 
+	/**
+	 * @param EntityUsage[] $actualUsages
+	 */
 	private function hasUsage( $actualUsages, EntityId $entityId, $aspect ) {
 		$usage = new EntityUsage( $entityId, $aspect );
 		$key = $usage->getIdentityString();

lib/includes/LanguageFallbackChainFactory.php

Doc Comments +3 added lines

@@ -227,6 +227,9 @@
 		return $languageFallbackChain;
 	}
 
+	/**
+	 * @param string $languageCode
+	 */
 	private function getBabel( $languageCode, $user ) {
 		$babel = array();
 

lib/includes/Summary.php

Doc Comments +1 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
 	 * @param string|null $moduleName The module part of the auto comment
 	 * @param string|null $actionName The action part of the auto comment
 	 * @param string|null $languageCode The language code to use as the second auto comment argument
-	 * @param array $commentArgs The arguments to the auto comment
+	 * @param string[] $commentArgs The arguments to the auto comment
 	 * @param array $summaryArgs The arguments to the auto summary
 	 */
 	public function __construct(
@@ -153,7 +153,6 @@ 
 	/**
 	 * Add auto comment arguments.
 	 *
-	 * @param mixed $args,... Parts to be stringed together
 	 */
 	public function addAutoCommentArgs( $args /*...*/ ) {
 		if ( !is_array( $args ) ) {
@@ -166,7 +165,6 @@ 
 	/**
 	 * Add arguments to the summary part.
 	 *
-	 * @param mixed $args,... Parts to be stringed together
 	 */
 	public function addAutoSummaryArgs( $args /*...*/ ) {
 		if ( !is_array( $args ) ) {

lib/tests/phpunit/MockRepository.php

Doc Comments +5 added lines, -5 removed lines

@@ -232,7 +232,7 @@ 
 	 *
 	 * @param EntityDocument $entity
 	 * @param int $revisionId
-	 * @param int|string $timestamp
+	 * @param integer $timestamp
 	 * @param User|string|null $user
 	 *
 	 * @throws StorageException
@@ -295,7 +295,7 @@ 
 	 *
 	 * @param EntityRedirect $redirect
 	 * @param int $revisionId
-	 * @param string|int $timestamp
+	 * @param integer $timestamp
 	 *
 	 * @throws StorageException
 	 */
@@ -644,7 +644,7 @@ 
 	/**
 	 * @param string $idString
 	 *
-	 * @return ItemId|PropertyId
+	 * @return EntityId
 	 */
 	private function parseId( $idString ) {
 		$parser = new BasicEntityIdParser();
@@ -653,7 +653,7 @@ 
 
 	/**
 	 * @param int $revisionId
-	 * @param EntityId|string $entityId
+	 * @param EntityId $entityId
 	 * @param string $summary
 	 * @param User|string $user
 	 */
@@ -690,7 +690,7 @@ 
 	 * Returns the newest (according to the revision id) log entry
 	 * for the given entity.
 	 *
-	 * @param EntityId|string $entityId
+	 * @param ItemId $entityId
 	 *
 	 * @return array|null An associative array containing the fields
 	 * 'revision', 'entity', 'summary', and 'user'.