wikimedia / mediawiki-extensions-Wikibase

repo/includes/Content/EntityHandler.php

Doc Comments +4 added lines, -4 removed lines

@@ -559,12 +559,12 @@
 	/**
 	 * @see ContentHandler::getUndoContent
 	 *
-	 * @param Revision|Content $latest The current text
-	 * @param Revision|Content $newer The revision to undo
-	 * @param Revision|Content $older Must be an earlier revision than $newer
+	 * @param EntityContent $latest The current text
+	 * @param EntityContent $newer The revision to undo
+	 * @param EntityContent $older Must be an earlier revision than $newer
 	 * @param bool $undoIsLatest Set to true if $newer is from the current revision (since 1.32)
 	 *
-	 * @return EntityContent|bool Content on success, false on failure
+	 * @return null|Content Content on success, false on failure
 	 */
 	public function getUndoContent( $latest, $newer, $older, $undoIsLatest = false ) {
 		$latestContent = ( $latest instanceof Revision ) ? $latest->getContent() : $latest;

client/includes/Hooks/SkinAfterBottomScriptsHandler.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Html;
 use PageImages\PageImages;
 use Title;
-
 use Wikibase\Client\RepoLinker;
 use Wikibase\Client\WikibaseClient;
 use Wikibase\DataModel\Entity\EntityDocument;

client/tests/phpunit/integration/includes/Changes/InjectRCRecordsJobTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -87,7 +87,7 @@ 
 	}
 
 	/**
-	 * @return RecentChangesFinder|MockObject
+	 * @return RecentChangesFinder
 	 */
 	private function getRCDupeDetectorMock() {
 		$rcDupeDetector = $this->getMockBuilder( RecentChangesFinder::class )
@@ -117,7 +117,7 @@ 
 	 * @param string $text
 	 * @param int $id
 	 *
-	 * @return Title|MockObject
+	 * @return \Wikibase\DataModel\Entity\EntityId
 	 */
 	private function getTitleMock( $text, $id = 23 ) {
 		$title = $this->getMockBuilder( Title::class )

lib/tests/phpunit/ServiceByTypeDispatcherTest.php

Doc Comments +3 added lines

@@ -26,6 +26,9 @@
 		yield 'default not matching type' => [ false, EntityUrlLookup::class, [], new stdClass() ];
 	}
 
+	/**
+	 * @param stdClass $fakeServiceToReturn
+	 */
 	private function getSomeServiceReturningCallback( $fakeServiceToReturn = null ) {
 		return function () use ( $fakeServiceToReturn ) {
 			return $fakeServiceToReturn ?? new stdClass();

repo/includes/Hooks/HtmlPageLinkRendererEndHookHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -335,7 +335,7 @@
 	 * @param Title|null $currentTitle
 	 * @param RequestContext $context
 	 *
-	 * @return bool
+	 * @return null|boolean
 	 */
 	private function shouldConvert( ?Title $currentTitle, RequestContext $context ) {
 		static $shouldConvert = null;

repo/includes/RepoHooks.php

Doc Comments +6 added lines, -1 removed lines

@@ -170,6 +170,7 @@ 
 	 *
 	 * @param int $ns Namespace ID
 	 * @param bool &$movable
+	 * @param boolean $movable
 	 */
 	public static function onNamespaceIsMovable( $ns, &$movable ) {
 		if ( self::isNamespaceUsedByLocalEntities( $ns ) ) {
@@ -177,6 +178,9 @@ 
 		}
 	}
 
+	/**
+	 * @param integer $namespace
+	 */
 	private static function isNamespaceUsedByLocalEntities( $namespace ) {
 		$wikibaseRepo = WikibaseRepo::getDefaultInstance();
 		$namespaceLookup = $wikibaseRepo->getEntityNamespaceLookup();
@@ -774,6 +778,7 @@ 
 	 * @param string $contentModel
 	 * @param LinkTarget $title Actually a Title object, but we only require getNamespace
 	 * @param bool &$ok
+	 * @param boolean $ok
 	 *
 	 * @return bool
 	 */
@@ -852,7 +857,7 @@ 
 	/**
 	 * Helper for onAPIQuerySiteInfoStatisticsInfo
 	 *
-	 * @param object $row
+	 * @param \stdClass|null $row
 	 * @return array
 	 */
 	private static function formatDispatchRow( $row ) {

repo/tests/phpunit/includes/ParserOutput/FullEntityParserOutputGeneratorIntegrationTest.php

Doc Comments +6 added lines

@@ -111,6 +111,9 @@ 
 			->getEntityParserOutputGenerator( Language::factory( 'en' ) );
 	}
 
+	/**
+	 * @param string $id
+	 */
 	private function saveItem( $id ) {
 		$this->entityStore->saveEntity(
 			new Item( new ItemId( $id ) ),
@@ -119,6 +122,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $id
+	 */
 	private function saveProperty( $id ) {
 		$this->entityStore->saveEntity(
 			new Property( new PropertyId( $id ), null, 'wikibase-item' ),

client/tests/phpunit/integration/includes/Hooks/DataUpdateHookHandlerTest.php

Doc Comments -1 removed lines

@@ -127,7 +127,6 @@
 	}
 
 	/**
-	 * @param Title $title
 	 * @param array|null $currentUsages
 	 *
 	 * @return UsageLookup

client/includes/Hooks/ChangesListSpecialPageHookHandler.php

Doc Comments +2 added lines

@@ -65,6 +65,8 @@
 	 * @param array &$query_options Array of options for the database request
 	 * @param array &$join_conds Join conditions for the tables
 	 * @param \FormOptions $opts FormOptions for this request
+	 * @param string[] $tables
+	 * @param string[] $fields
 	 */
 	public function onChangesListSpecialPageQuery( $name, &$tables, &$fields,
 			&$conds, &$query_options, &$join_conds, $opts ) {