wikimedia / mediawiki-extensions-Wikibase

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() )

repo/includes/LinkedData/EntityDataFormatProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 	 *
 	 * @param string $format A canonical format name, as returned by getFormatName() or getSupportedFormats().
 	 *
-	 * @return string|null
+	 * @return string
 	 */
 	public function getMimeType( $format ) {
 		$this->initFormats();

repo/tests/phpunit/includes/FederatedProperties/Api/SetClaimTest.php

Doc Comments +3 added lines

@@ -68,6 +68,9 @@
 		] );
 	}
 
+	/**
+	 * @param Statement $statement
+	 */
 	private function getSerializedStatement( $statement ) {
 		$statementSerializer = ( new SerializerFactory( new DataValueSerializer() ) )->newStatementSerializer();
 		return $statementSerializer->serialize( $statement );

repo/tests/phpunit/includes/FederatedProperties/FederatedPropertiesTestTrait.php

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
 		$this->setWbSetting( 'federatedPropertiesEnabled', true );
 	}
 
+	/**
+	 * @param string|boolean $value
+	 */
 	private function setWbSetting( string $name, $value ) {
 		$this->setWbSettingInGlobalIfMwIntegrationTest( $name, $value );
 		$this->setWbSettingInSettings( $name, $value );

repo/includes/FederatedProperties/FederatedPropertiesError.php

Doc Comments +1 added lines

@@ -21,6 +21,7 @@
 	 * @param LabelsProvider $entity
 	 * @param string $msg Message key (string) for error text
 	 * @param array $params Array with parameters to wfMessage()
+	 * @param string|null $languageCode
 	 *
 	 * @suppress SecurityCheck-DoubleEscaped
 	 */

repo/includes/FederatedProperties/SummaryParsingPrefetchHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
 
 	/**
 	 * @param IResultWrapper|array $rows
-	 * @param array $languageCodes
+	 * @param string[] $languageCodes
 	 * @param array $termTypes
 	 */
 	public function prefetchFederatedProperties( $rows, array $languageCodes, array $termTypes ) : void {

repo/tests/phpunit/includes/FederatedProperties/Hooks/DifferenceEngineViewHeaderHookHandlerTest.php

Doc Comments +3 added lines

@@ -182,6 +182,9 @@
 		return $rows;
 	}
 
+	/**
+	 * @param string $titleText
+	 */
 	private function getMockedDiffEngine( $getOldRevision, $getNewRevision, $titleText ) {
 		$diffEngine = $this->createMock( DifferenceEngine::class );
 		$diffEngine->expects( $this->once() )

repo/tests/phpunit/includes/Hooks/LabelPrefetchHookHandlerTestBase.php

Doc Comments +4 added lines, -1 removed lines

@@ -55,7 +55,6 @@ 
 	 * @param string[] $termTypes
 	 * @param string[] $languageCodes
 	 * @param PrefetchingTermLookup|null $prefetchingTermLookup
-	 * @param EntityLookup|null $entityLookup
 	 * @param bool $federatedPropertiesEnabled
 	 * @return LabelPrefetchHookHandler
 	 */
@@ -103,6 +102,10 @@ 
 		);
 	}
 
+	/**
+	 * @param string[] $expectedTermTypes
+	 * @param string[] $expectedLanguageCodes
+	 */
 	protected function getPrefetchTermsCallback( $expectedIds, $expectedTermTypes, $expectedLanguageCodes ) {
 		$prefetchTerms = function (
 			array $entityIds,