Page MenuHomePhabricator

TypeError: Argument 2 passed to ChangeTagsRevisionList::updateChangeTagsOnAll() must be of the type array, null given, called in /srv/mediawiki/php-1.40.0-wmf.1/includes/specials/SpecialEditTags.php on line 440
Closed, ResolvedPublicPRODUCTION ERROR

Description

Error
normalized_message
[{reqId}] {exception_url}   TypeError: Argument 2 passed to ChangeTagsRevisionList::updateChangeTagsOnAll() must be of the type array, null given, called in /srv/mediawiki/php-1.40.0-wmf.1/includes/specials/SpecialEditTags.php on line 440
exception.trace
from /srv/mediawiki/php-1.40.0-wmf.1/includes/changetags/ChangeTagsRevisionList.php(86)
#0 /srv/mediawiki/php-1.40.0-wmf.1/includes/specials/SpecialEditTags.php(440): ChangeTagsRevisionList->updateChangeTagsOnAll(array, NULL, NULL, string, User)
#1 /srv/mediawiki/php-1.40.0-wmf.1/includes/specials/SpecialEditTags.php(158): SpecialEditTags->submit()
#2 /srv/mediawiki/php-1.40.0-wmf.1/includes/specialpage/SpecialPage.php(701): SpecialEditTags->execute(NULL)
#3 /srv/mediawiki/php-1.40.0-wmf.1/includes/specialpage/SpecialPageFactory.php(1428): SpecialPage->run(NULL)
#4 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(316): MediaWiki\SpecialPage\SpecialPageFactory->executePath(string, RequestContext)
#5 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(904): MediaWiki->performRequest()
#6 /srv/mediawiki/php-1.40.0-wmf.1/includes/MediaWiki.php(562): MediaWiki->main()
#7 /srv/mediawiki/php-1.40.0-wmf.1/index.php(50): MediaWiki->run()
#8 /srv/mediawiki/php-1.40.0-wmf.1/index.php(46): wfIndexMain()
#9 /srv/mediawiki/w/index.php(3): require(string)
#10 {main}
Impact
Notes

Surfaced by T311567: ManualLogEntry::getComment() should always return string.

Reported in https://fi.wikipedia.org/w/index.php?title=Wikipedia:Kahvihuone_(tekniikka)&diff=20792152&oldid=20790507&diffmode=visual

Event Timeline

Change 832976 had a related patch set uploaded (by Func; author: Func):

[mediawiki/core@master] SpecialEditTags: Set default value of wpTagsToRemove to empty array

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

Change 832976 merged by jenkins-bot:

[mediawiki/core@master] SpecialEditTags: Set default value of wpTagsToRemove to empty array

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

Change 833029 had a related patch set uploaded (by Func; author: Func):

[mediawiki/core@REL1_37] SpecialEditTags: Set default value of wpTagsToRemove to empty array

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

Change 833029 merged by jenkins-bot:

[mediawiki/core@REL1_37] SpecialEditTags: Set default value of wpTagsToRemove to empty array

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

Jdforrester-WMF assigned this task to Func.
Jdforrester-WMF subscribed.

This should now be Resolved, given that the relevant code was fixed in master and the backports reverted per T318040: Decide if we should revert "ManualLogEntry::getComment() should always return string" out of REL1_37 before release.