wikimedia / mediawiki-extensions-Wikibase

lib/includes/Store/Sql/Terms/TermStoresDelegatingPrefetchingItemTermLookup.php

Doc Comments +2 added lines, -2 removed lines

@@ -59,8 +59,8 @@
 	 * Loads a set of terms into the buffer.
 	 * The source from which to fetch would typically be supplied to the buffer's constructor.
 	 * @param EntityId[] $entityIds
-	 * @param string[]|null $termTypes The desired term types; null means all.
-	 * @param string[]|null $languageCodes The desired languages; null means all.
+	 * @param string[] $termTypes The desired term types; null means all.
+	 * @param string[] $languageCodes The desired languages; null means all.
 	 */
 	public function prefetchTerms( array $entityIds, array $termTypes, array $languageCodes ) {
 		Assert::parameterElementType( ItemId::class, $entityIds, '$entityIds' );

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

Doc Comments +4 added lines

@@ -91,6 +91,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' );

lib/tests/phpunit/Store/ItemTermStoreWriterAdapterTest.php

Doc Comments +6 added lines

@@ -57,6 +57,9 @@ 
 		);
 	}
 
+	/**
+	 * @return EntityDocument
+	 */
 	private function newUnsupportedEntity() {
 		return $this->createMock( EntityDocument::class );
 	}
@@ -68,6 +71,9 @@ 
 		$writer->deleteTermsOfEntity( $this->newUnsupportedId() );
 	}
 
+	/**
+	 * @return EntityId
+	 */
 	public function newUnsupportedId() {
 		return $this->createMock( EntityId::class );
 	}

lib/tests/phpunit/Store/PropertyTermStoreWriterAdapterTest.php

Doc Comments +6 added lines

@@ -57,6 +57,9 @@ 
 		);
 	}
 
+	/**
+	 * @return EntityDocument
+	 */
 	private function newUnsupportedEntity() {
 		return $this->createMock( EntityDocument::class );
 	}
@@ -68,6 +71,9 @@ 
 		$writer->deleteTermsOfEntity( $this->newUnsupportedId() );
 	}
 
+	/**
+	 * @return EntityId
+	 */
 	public function newUnsupportedId() {
 		return $this->createMock( EntityId::class );
 	}

lib/tests/phpunit/Store/Sql/Terms/DatabaseTermStoreWriterTestGetTermsTrait.php

Doc Comments +5 added lines

@@ -25,6 +25,11 @@
  */
 trait DatabaseTermStoreWriterTestGetTermsTrait {
 
+	/**
+	 * @param string $termsTable
+	 * @param string $termInLangField
+	 * @param string $idField
+	 */
 	private function getTerms( Int32EntityId $entityId, $termsTable, $termInLangField, $idField ): Fingerprint {
 		$loadBalancer = new FakeLoadBalancer( [
 			'dbr' => $this->db,

repo/tests/phpunit/includes/Store/Sql/WikiPageEntityMetaDataLookupTest.php

Doc Comments +3 added lines

@@ -85,6 +85,7 @@ 
 	}
 
 	/**
+	 * @param EntityNamespaceLookup $namespaceLookup
 	 * @return WikiPageEntityMetaDataLookup
 	 */
 	private function getWikiPageEntityMetaDataLookup( $namespaceLookup = null ) {
@@ -163,6 +164,8 @@ 
 
 	/**
 	 * Gets a "lagged" database connection: We always leave out the first row on select.
+	 * @param integer $selectCount
+	 * @param integer $selectRowCount
 	 */
 	private function getLaggedDatabase( IDatabase $realDB, $selectCount, $selectRowCount ) {
 		$db = $this->getMockBuilder( IDatabase::class )

repo/tests/phpunit/includes/RepoHooksTest.php

Doc Comments +3 added lines

@@ -514,6 +514,9 @@
 		$this->assertTrue( $canBeMoved );
 	}
 
+	/**
+	 * @param integer $propertyNamespace
+	 */
 	private function newEntitySourceSettings( $itemNamespace, $propertyNamespace ) {
 		global $wgWBRepoSettings;
 

repo/tests/phpunit/includes/Rdf/PropertyRdfBuilderTest.php

Doc Comments +3 added lines

@@ -62,6 +62,9 @@
 		);
 	}
 
+	/**
+	 * @param string $dataSetName
+	 */
 	private function assertOrCreateNTriples( $dataSetName, RdfWriter $writer ) {
 		$actual = $writer->drain();
 		$expected = $this->getTestData()->getNTriples( $dataSetName );

lib/includes/SimpleCacheWithBagOStuff.php

Doc Comments +6 added lines, -6 removed lines

@@ -58,7 +58,7 @@ 
 	 * Fetches a value from the cache.
 	 *
 	 * @param string $key The unique key of this item in the cache.
-	 * @param mixed $default Default value to return if the key does not exist.
+	 * @param string $default Default value to return if the key does not exist.
 	 *
 	 * @return mixed The value of the item from the cache, or $default in case of cache miss.
 	 *
@@ -80,8 +80,8 @@ 
 	 * Persists data in the cache, uniquely referenced by a key with an optional expiration TTL time.
 	 *
 	 * @param string $key The key of the item to store.
-	 * @param mixed $value The value of the item to store, must be serializable.
-	 * @param null|int|DateInterval $ttl Optional. The TTL value of this item. If no value is sent and
+	 * @param string $value The value of the item to store, must be serializable.
+	 * @param integer $ttl Optional. The TTL value of this item. If no value is sent and
 	 *                                      the driver supports TTL then the library may set a default value
 	 *                                      for it or let the driver take care of that.
 	 *
@@ -128,8 +128,8 @@ 
 	/**
 	 * Obtains multiple cache items by their unique keys.
 	 *
-	 * @param iterable $keys A list of keys that can obtained in a single operation.
-	 * @param mixed $default Default value to return for keys that do not exist.
+	 * @param string[] $keys A list of keys that can obtained in a single operation.
+	 * @param string $default Default value to return for keys that do not exist.
 	 *
 	 * @return iterable A list of key => value pairs. Cache keys that do not exist or are stale will have $default as value.
 	 *
@@ -169,7 +169,7 @@ 
 	 * Persists a set of key => value pairs in the cache, with an optional TTL.
 	 *
 	 * @param iterable $values A list of key => value pairs for a multiple-set operation.
-	 * @param null|int|DateInterval $ttl Optional. The TTL value of this item. If no value is sent and
+	 * @param integer $ttl Optional. The TTL value of this item. If no value is sent and
 	 *                                       the driver supports TTL then the library may set a default value
 	 *                                       for it or let the driver take care of that.
 	 *