Page MenuHomePhabricator

TypeError: Argument 1 passed to Flow\Hooks::onSpecialCheckUserGetLinksFromRow() must be SpecialPage, CheckUserGetEditsPager given
Closed, ResolvedPublicPRODUCTION ERROR

Description

Error
normalized_message
[{reqId}] {exception_url}   TypeError: Argument 1 passed to Flow\Hooks::onSpecialCheckUserGetLinksFromRow() must be an instance of SpecialPage, instance of MediaWiki\CheckUser\CheckUserPagers\CheckUserGetEditsPager given, called in /srv/mediawiki/php-1.39
exception.trace
from /srv/mediawiki/php-1.39.0-wmf.22/extensions/Flow/includes/Hooks.php(517)
#0 /srv/mediawiki/php-1.39.0-wmf.22/includes/HookContainer/HookContainer.php(338): Flow\Hooks::onSpecialCheckUserGetLinksFromRow(MediaWiki\CheckUser\CheckUserPagers\CheckUserGetEditsPager, stdClass, array)
#1 /srv/mediawiki/php-1.39.0-wmf.22/includes/HookContainer/HookContainer.php(137): MediaWiki\HookContainer\HookContainer->callLegacyHook(string, array, array, array)
#2 /srv/mediawiki/php-1.39.0-wmf.22/includes/Hooks.php(137): MediaWiki\HookContainer\HookContainer->run(string, array, array)
#3 /srv/mediawiki/php-1.39.0-wmf.22/extensions/CheckUser/src/CheckUserPagers/CheckUserGetEditsPager.php(347): Hooks::run(string, array)
#4 /srv/mediawiki/php-1.39.0-wmf.22/extensions/CheckUser/src/CheckUserPagers/CheckUserGetEditsPager.php(136): MediaWiki\CheckUser\CheckUserPagers\CheckUserGetEditsPager->getLinksFromRow(stdClass)
#5 /srv/mediawiki/php-1.39.0-wmf.22/includes/pager/ReverseChronologicalPager.php(115): MediaWiki\CheckUser\CheckUserPagers\CheckUserGetEditsPager->formatRow(stdClass)
#6 /srv/mediawiki/php-1.39.0-wmf.22/includes/pager/IndexPager.php(634): ReverseChronologicalPager->getRow(stdClass)
#7 /srv/mediawiki/php-1.39.0-wmf.22/extensions/CheckUser/src/Specials/SpecialCheckUser.php(681): IndexPager->getBody()
#8 /srv/mediawiki/php-1.39.0-wmf.22/extensions/CheckUser/src/Specials/SpecialCheckUser.php(285): MediaWiki\CheckUser\Specials\SpecialCheckUser->doIPEditsRequest(string, boolean)
#9 /srv/mediawiki/php-1.39.0-wmf.22/includes/specialpage/SpecialPage.php(690): MediaWiki\CheckUser\Specials\SpecialCheckUser->execute(NULL)
#10 /srv/mediawiki/php-1.39.0-wmf.22/includes/specialpage/SpecialPageFactory.php(1423): SpecialPage->run(NULL)
#11 /srv/mediawiki/php-1.39.0-wmf.22/includes/MediaWiki.php(316): MediaWiki\SpecialPage\SpecialPageFactory->executePath(string, RequestContext)
#12 /srv/mediawiki/php-1.39.0-wmf.22/includes/MediaWiki.php(911): MediaWiki->performRequest()
#13 /srv/mediawiki/php-1.39.0-wmf.22/includes/MediaWiki.php(568): MediaWiki->main()
#14 /srv/mediawiki/php-1.39.0-wmf.22/index.php(50): MediaWiki->run()
#15 /srv/mediawiki/php-1.39.0-wmf.22/index.php(46): wfIndexMain()
#16 /srv/mediawiki/w/index.php(3): require(string)
#17 {main}
Notes
  • Seems new in 1.39.0-wmf.22
  • Only 3 instances since deploying to group1 yesterday
  • All on commonswiki

Event Timeline

Change 818181 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/Flow@master] Update CheckUser hook for pagination

https://gerrit.wikimedia.org/r/818181

@Zabe - pretty low rate on this one, is there any reason to block on above?

@Zabe - pretty low rate on this one, is there any reason to block on above?

The impact of this is that every time a check user does a 'get edits' on a user or an ip on a wiki with Flow enabled it will fatal. Most wikis where this is happens are in group2, so I would prefer to get this fix deployed before the train rolls forward, in order to prevent some CU's not being capable of doing their work on some wikis.

Zabe triaged this task as Unbreak Now! priority.Jul 28 2022, 6:07 PM

Change 818181 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] Update CheckUser hook for pagination

https://gerrit.wikimedia.org/r/818181

Change 818154 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/Flow@wmf/1.39.0-wmf.22] Update CheckUser hook for pagination

https://gerrit.wikimedia.org/r/818154

Change 818154 merged by jenkins-bot:

[mediawiki/extensions/Flow@wmf/1.39.0-wmf.22] Update CheckUser hook for pagination

https://gerrit.wikimedia.org/r/818154

Mentioned in SAL (#wikimedia-operations) [2022-07-28T19:34:33Z] <brennen@deploy1002> Synchronized php-1.39.0-wmf.22/extensions/Flow: Backport: [[gerrit:818154|Update CheckUser hook for pagination (T314058 T314069)]] (duration: 03m 16s)

brennen claimed this task.