Completed
Push — master ( 0a57e3...259aca )
by
unknown
07:19 queued 10s
created
client/includes/DataAccess/Scribunto/CachingFallbackBasedTermLookup.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -139,7 +139,7 @@
 block discarded – undo
139 139
 	/**
140 140
 	 * @param EntityId $entityId
141 141
 	 * @param Language $language
142
-	 * @param $termType
142
+	 * @param string $termType
143 143
 	 * @return TermFallback|null
144 144
 	 */
145 145
 	private function lookupWithoutCache( EntityId $entityId, Language $language, $termType ): ?TermFallback {
Please login to merge, or discard this patch.
unit/includes/DataAccess/Scribunto/CachingFallbackBasedTermLookupTest.php 1 patch
Doc Comments   +21 added lines patch added patch discarded remove patch
@@ -110,6 +110,11 @@  discard block
 block discarded – undo
110 110
 		);
111 111
 	}
112 112
 
113
+	/**
114
+	 * @param string $term
115
+	 * @param string $requestLanguageCode
116
+	 * @param string $actualLanguageCode
117
+	 */
113 118
 	private function getTermFallback( $term, $requestLanguageCode, $actualLanguageCode = null ): ?TermFallback {
114 119
 		if ( $term === null ) {
115 120
 			return null;
@@ -264,6 +269,9 @@  discard block
 block discarded – undo
264 269
 		);
265 270
 	}
266 271
 
272
+	/**
273
+	 * @param ItemId $itemId
274
+	 */
267 275
 	private function mockCacheWithContent( string $term, $itemId ): void {
268 276
 		$termFallback = new TermFallback( 'en', $term, 'en', 'en' );
269 277
 		$this->revisionLookup->method( 'lookupLatestRevisionResolvingRedirect' )->willReturn( [ 1, $itemId ] );
@@ -284,6 +292,9 @@  discard block
 block discarded – undo
284 292
 			->willReturn( [ self::ITEM_Q1_REVISION, $itemId ] );
285 293
 	}
286 294
 
295
+	/**
296
+	 * @param ItemId $itemId
297
+	 */
287 298
 	private function mockCacheEmpty( $itemId ): void {
288 299
 		$this->mockRevisionLookup( $itemId );
289 300
 
@@ -293,6 +304,12 @@  discard block
 block discarded – undo
293 304
 			->willReturn( TermFallbackCacheFacade::NO_VALUE );
294 305
 	}
295 306
 
307
+	/**
308
+	 * @param TermFallback|null $termFallback
309
+	 * @param ItemId $targetEntityId
310
+	 * @param integer $revisionId
311
+	 * @param string $termType
312
+	 */
296 313
 	private function mockCacheSetExpectation(
297 314
 		$termFallback,
298 315
 		$targetEntityId,
@@ -307,6 +324,10 @@  discard block
 block discarded – undo
307 324
 			->with( $termFallback, $targetEntityId, $revisionId, $languageCode, $termType );
308 325
 	}
309 326
 
327
+	/**
328
+	 * @param ItemId $itemId
329
+	 * @param TermFallback|null $termFallback
330
+	 */
310 331
 	private function mockInternalLookupWithContent(
311 332
 		$itemId,
312 333
 		?TermFallback $termFallback,
Please login to merge, or discard this patch.