WatchedItemStore: Remove deprecated Title param to getNextRevision()
authorRoan Kattouw <roan.kattouw@gmail.com>
Sat, 18 May 2019 14:38:48 +0000 (16:38 +0200)
committerRoan Kattouw <roan.kattouw@gmail.com>
Sun, 19 May 2019 13:09:13 +0000 (15:09 +0200)
This was deprecated in I76bc6fd6ee, but somehow this usage was missed,
maybe the commit that added it went through review in parallel with that
one.

Change-Id: I8131f360f4300f62d56be64fda117038f51e693b

includes/watcheditem/WatchedItemStore.php
tests/phpunit/includes/watcheditem/WatchedItemStoreUnitTest.php

index ec25002..8560624 100644 (file)
@@ -1120,7 +1120,7 @@ class WatchedItemStore implements WatchedItemStoreInterface, StatsdAwareInterfac
                }
 
                $oldRev = $this->revisionLookup->getRevisionById( $oldid );
-               $nextRev = $this->revisionLookup->getNextRevision( $oldRev, $title );
+               $nextRev = $this->revisionLookup->getNextRevision( $oldRev );
                if ( !$nextRev ) {
                        // Oldid given and is the latest revision for this title; clear the timestamp.
                        return null;
index 82308de..7ec2d37 100644 (file)
@@ -2094,9 +2094,8 @@ class WatchedItemStoreUnitTest extends MediaWikiTestCase {
                                return $mockRevisionRecord;
                        },
                        'getNextRevision' =>
-                       function ( $oldRev, $titleArg ) use ( $mockRevisionRecord, $title ) {
+                       function ( $oldRev ) use ( $mockRevisionRecord ) {
                                $this->assertSame( $mockRevisionRecord, $oldRev );
-                               $this->assertSame( $title, $titleArg );
                                return false;
                        },
                ], [