wikimedia / mediawiki-extensions-Wikibase

repo/tests/phpunit/includes/Api/SetReferenceTest.php

Doc Comments +3 added lines

@@ -245,6 +245,9 @@
 		) );
 	}
 
+	/**
+	 * @param string|null $referenceHash
+	 */
 	protected function makeInvalidRequest(
 		string $statementGuid,
 		?string $referenceHash,

lib/tests/phpunit/Formatters/ItemPropertyIdHtmlLinkFormatterTest.php

Doc Comments +33 added lines

@@ -333,6 +333,11 @@ 
 		$this->assertEquals( $expectedResult, $result );
 	}
 
+	/**
+	 * @param string $entityId
+	 *
+	 * @return EntityTitleTextLookup
+	 */
 	private function getEntityTitleTextLookup( $entityId ) {
 		$entityTitleTextLookup = $this->createMock( EntityTitleTextLookup::class );
 		$entityTitleTextLookup
@@ -342,6 +347,9 @@ 
 		return $entityTitleTextLookup;
 	}
 
+	/**
+	 * @return EntityUrlLookup
+	 */
 	private function getEntityUrlLookup() {
 		$entityUrlLookup = $this->createMock( EntityUrlLookup::class );
 		$entityUrlLookup
@@ -445,16 +453,25 @@ 
 		);
 	}
 
+	/**
+	 * @param string $itemId
+	 */
 	private function itemPageUrl( $itemId ) {
 		return "/index.php/{$itemId}";
 	}
 
+	/**
+	 * @param string $languageCode
+	 */
 	private function givenUserLanguageIs( $languageCode ) {
 		$this->setUserLang( $languageCode );
 		$this->currentUserLanguage = $languageCode;
 
 		return new class( $this ) extends ItemPropertyIdHtmlLinkFormatterTest {
 
+			/**
+			 * @param ItemPropertyIdHtmlLinkFormatterTest $testCase
+			 */
 			public function __construct( $testCase ) {
 				$this->testCase = $testCase;
 			}
@@ -463,6 +480,9 @@ 
 				$this->testCase->fallbackChain = $languageCodes;
 			}
 
+			/**
+			 * @param string $anotherLanguage
+			 */
 			public function canBeTransliteratedFrom( $anotherLanguage ) {
 				$this->testCase->transliterationMap[ $anotherLanguage ] = $this->testCase->currentUserLanguage;
 			}
@@ -470,6 +490,11 @@ 
 		};
 	}
 
+	/**
+	 * @param string $itemId
+	 * @param string $labelLanguage
+	 * @param string $labelText
+	 */
 	private function givenItemHasLabel( $itemId, $labelLanguage, $labelText ) {
 		$this->givenItemExists( 'Q1' );
 
@@ -575,12 +600,20 @@ 
 		};
 	}
 
+	/**
+	 * @param string $propertyId
+	 * @param string $labelLanguage
+	 * @param string $labelText
+	 */
 	private function givenPropertyHasLabel( $propertyId, $labelLanguage, $labelText ) {
 		$this->givenPropertyExists( 'P1' );
 
 		$this->givenEntityHasLabel( new PropertyId( $propertyId ), $labelLanguage, $labelText );
 	}
 
+	/**
+	 * @param string $propertyId
+	 */
 	private function propertyPageUrl( $propertyId ) {
 		return "/index.php/Property:{$propertyId}";
 	}

repo/includes/Hooks/ShowSearchHitHandler.php

Doc Comments +10 added lines

@@ -82,6 +82,15 @@ 
 	 * @param string &$date
 	 * @param string &$related
 	 * @param string &$html
+	 * @param string $link
+	 * @param string $redirect
+	 * @param string $section
+	 * @param string $extract
+	 * @param string $score
+	 * @param string $size
+	 * @param string $date
+	 * @param string $related
+	 * @param string $html
 	 * @return void
 	 */
 	public function onShowSearchHit( $searchPage, $result,
@@ -200,6 +209,7 @@ 
 	 * @param SpecialSearch $specialSearch
 	 * @param string[] &$query
 	 * @param string[] &$attributes
+	 * @param string $titleSnippet
 	 * @return void
 	 */
 	public function onShowSearchHitTitle(

repo/tests/phpunit/includes/MediawikiEditEntityTest.php

Doc Comments +4 added lines, -1 removed lines

@@ -38,6 +38,9 @@ 
  */
 class MediawikiEditEntityTest extends MediaWikiIntegrationTestCase {
 
+	/**
+	 * @param string $name
+	 */
 	private function getUser( $name ) {
 		$user = User::newFromName( $name );
 
@@ -120,7 +123,7 @@ 
 	 * @param EntityId $entityId
 	 * @param EntityTitleStoreLookup $titleLookup
 	 * @param User|null $user
-	 * @param bool $baseRevId
+	 * @param integer $baseRevId
 	 * @param bool[]|null $permissions map of actions to bool, indicating which actions are allowed.
 	 * @param EditFilterHookRunner|null $editFilterHookRunner
 	 *

lib/tests/phpunit/Store/CachingFallbackLabelDescriptionLookupTest.php

Doc Comments +6 added lines

@@ -325,6 +325,9 @@ 
 		$this->assertEquals( $expectedLabel, $gotLabel );
 	}
 
+	/**
+	 * @return RedirectResolvingLatestRevisionLookup
+	 */
 	private function newRedirectResolvingLatestRevisionLookup( int $revision, EntityId $entityId ) {
 		$revLookup = $this->prophesize( RedirectResolvingLatestRevisionLookup::class );
 		$revLookup->lookupLatestRevisionResolvingRedirect( Argument::any() )
@@ -333,6 +336,9 @@ 
 		return $revLookup->reveal();
 	}
 
+	/**
+	 * @return TermLanguageFallbackChain
+	 */
 	private function newFallbackChain() {
 		$fallbackChain = $this->prophesize( TermLanguageFallbackChain::class );
 		$fallbackChain->getFetchLanguageCodes()->willReturn( [ 'en' ] );

lib/includes/TermLanguageFallbackChain.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
 	/**
 	 * @param LanguageWithConversion[] $chain
-	 * @param ContentLanguages|null $termLanguages
+	 * @param ContentLanguages $termLanguages
 	 */
 	public function __construct( array $chain, ContentLanguages $termLanguages ) {
 		$this->chain = array_values( array_filter(

lib/includes/Store/CachingFallbackLabelDescriptionLookup.php

Doc Comments +3 added lines

@@ -117,6 +117,9 @@
 		return $label;
 	}
 
+	/**
+	 * @param string $languageCode
+	 */
 	private function getTerm( EntityId $entityId, $languageCode, $termName = self::LABEL ) {
 		$resolutionResult = $this->redirectResolvingRevisionLookup->lookupLatestRevisionResolvingRedirect( $entityId );
 		if ( $resolutionResult === null ) {

lib/includes/Store/CachingPrefetchingTermLookup.php

Doc Comments +1 added lines, -1 removed lines

@@ -269,7 +269,7 @@
 	 * @param EntityId $entityId
 	 * @param string $termType
 	 * @param string $languageCode
-	 * @param string|string[] $freshTerm string for existing label or description, string[] for existing aliases
+	 * @param string $freshTerm string for existing label or description, string[] for existing aliases
 	 *                                   Should never be null or false @see bufferAndCacheMissingTerm
 	 */
 	private function bufferAndCacheExistingTerm( EntityId $entityId, string $termType, string $languageCode, $freshTerm ): void {

repo/tests/phpunit/includes/Store/CachingPrefetchingTermLookupTest.php

Doc Comments +6 added lines

@@ -646,6 +646,9 @@ 
 		return $revisionAndRedirectResolver;
 	}
 
+	/**
+	 * @return string
+	 */
 	private function buildTestCacheKey(
 		string $itemId,
 		string $termType,
@@ -663,6 +666,9 @@ 
 		return $mockCache;
 	}
 
+	/**
+	 * @param boolean $fillValue
+	 */
 	private function newMockCacheExpectingKeys( array $expectedCacheKeys, $fillValue = null ): CacheInterface {
 		$cache = $this->createMock( CacheInterface::class );
 		$cache->expects( $this->once() )