Error
MediaWiki version: 1.35.0-wmf.39
message
Call to a member function getRevertTags() on null
Impact
production logspam
Description
Description
Details
Details
- Request ID
- 8683ed53-7d10-4e7e-bf0e-34f4e4fb8303
- Request URL
- https://www.wikidata.org/w/index.php?title=Q18479578&action=rollback&from=2804%3A14C%3A5BB3%3AA319%3A199D%3ABE00%3A6E42%3AA58D&token=73608e50a1dadf28653562de0046dfa85f05e30e%2B%5C
Stack Trace
exception.trace
#0 /srv/mediawiki/php-1.35.0-wmf.39/includes/page/WikiPage.php(3231): WikiPage->commitRollback(string, string, boolean, NULL, User, NULL)
#1 /srv/mediawiki/php-1.35.0-wmf.39/includes/actions/RollbackAction.php(124): WikiPage->doRollback(string, string, string, boolean, NULL, User)
#2 /srv/mediawiki/php-1.35.0-wmf.39/includes/actions/RollbackAction.php(87): RollbackAction->handleRollbackRequest()
#3 /srv/mediawiki/php-1.35.0-wmf.39/includes/MediaWiki.php(527): RollbackAction->show()
#4 /srv/mediawiki/php-1.35.0-wmf.39/includes/MediaWiki.php(313): MediaWiki->performAction(Article, Title)
#5 /srv/mediawiki/php-1.35.0-wmf.39/includes/MediaWiki.php(986): MediaWiki->performRequest()
#6 /srv/mediawiki/php-1.35.0-wmf.39/includes/MediaWiki.php(543): MediaWiki->main()
#7 /srv/mediawiki/php-1.35.0-wmf.39/index.php(53): MediaWiki->run()
#8 /srv/mediawiki/php-1.35.0-wmf.39/index.php(46): wfIndexMain()
#9 /srv/mediawiki/w/index.php(3): require(string)
#10 {main}| Project | Branch | Lines +/- | Subject | |
|---|---|---|---|---|
| mediawiki/core | master | +6 -5 | Make EditResult::getRevertTags() call to be only when the update was successful |
Event Timeline
Comment Actions
PageUpdater::getEditResult() is clearly documented to return null. But in this case, apart from the unchecked call it's also being done too early.
Comment Actions
Change 610353 had a related patch set uploaded (by Ammarpad; owner: Ammarpad):
[mediawiki/core@master] Make EditResult::getRevertTags() call to be only when the update was successful
Comment Actions
Change 610353 merged by jenkins-bot:
[mediawiki/core@master] Make EditResult::getRevertTags() call to be only when the update was successful