Page MenuHomePhabricator

SpecialNuke: Call to undefined method Title::getId()
Closed, ResolvedPublicBUG REPORT

Description

List of steps to reproduce

  • Go to Special:Nuke (in German: Spezial:Massenlöschung)
    • Enter a SQL patter, e.g. Foo_12%
    • Namespace = all
    • max pages= 500
    • On the next page select one (or more) of the matches

What happens?:

  • After Submit MediaWiki responds with HTTP 500 and an error page, with this error log:
/mediawiki/index.php?title=Spezial:Massenl%C3%B6schung&action=delete 
Error from line 379 of .../mediawiki/extensions/Nuke/includes/SpecialNuke.php: 
Call to undefined method Title::getId()

Backtrace:

#0 .../mediawiki/extensions/Nuke/includes/SpecialNuke.php(64): SpecialNuke->doDelete(array, string)
#1 .../mediawiki/includes/specialpage/SpecialPage.php(600): SpecialNuke->execute(NULL)
#2 .../mediawiki/includes/specialpage/SpecialPageFactory.php(635): SpecialPage->run(NULL)
#3 .../mediawiki/includes/MediaWiki.php(307): MediaWikiSpecialPageSpecialPageFactory->executePath(Title, RequestContext)
#4 .../mediawiki/includes/MediaWiki.php(945): MediaWiki->performRequest()
#5 .../mediawiki/includes/MediaWiki.php(548): MediaWiki->main()
#6 .../mediawiki/index.php(53): MediaWiki->run()
#7 .../mediawiki/index.php(46): wfIndexMain()
#8 {main}

What should have happened instead?:
Instead the selected pages should be deleted and no error messages should be shown.

Software version (if not a Wikimedia wiki), browser information, screenshots, other information, etc.:

  • MW 1.35.5
  • SMW, DisplayTitle and many other extensions
  • Maria 10.4.22
  • PHP 7.3.33

The same call is still present in the current master branch:

Event Timeline

Title::getId() is documented to be @since 1.35, but it actually is @since 1.36.

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

[mediawiki/core@master] Fix @since of Title::getId()

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

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

[mediawiki/core@REL1_35] Add Title::getId() as an alias for ::getArticleId()

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

Change 771013 merged by jenkins-bot:

[mediawiki/core@REL1_35] Add Title::getId() as an alias for ::getArticleId()

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

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

[mediawiki/extensions/Nuke@master] Bump required MW version to 1.35.6

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

Change 771012 merged by jenkins-bot:

[mediawiki/core@master] Fix @since of Title::getId()

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

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

[mediawiki/core@REL1_38] Fix @since of Title::getId()

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

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

[mediawiki/core@REL1_37] Fix @since of Title::getId()

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

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

[mediawiki/core@REL1_36] Fix @since of Title::getId()

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

Change 771652 merged by jenkins-bot:

[mediawiki/core@REL1_36] Fix @since of Title::getId()

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

Change 771651 merged by jenkins-bot:

[mediawiki/core@REL1_37] Fix @since of Title::getId()

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

Change 771650 merged by jenkins-bot:

[mediawiki/core@REL1_38] Fix @since of Title::getId()

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

Zabe claimed this task.

Thanks for the report!

This should be fixed in MediaWiki 1.35.6, which is scheduled to be released end of March.

Change 771067 merged by jenkins-bot:

[mediawiki/extensions/Nuke@master] Bump required MW version to 1.35.6

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

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

[mediawiki/extensions/Nuke@REL1_38] Bump required MW version to 1.35.6

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

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

[mediawiki/extensions/Nuke@REL1_37] Bump required MW version to 1.35.6

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

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

[mediawiki/extensions/Nuke@REL1_36] Bump required MW version to 1.35.6

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

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

[mediawiki/extensions/Nuke@REL1_35] Bump required MW version to 1.35.6

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

Change 771708 merged by jenkins-bot:

[mediawiki/extensions/Nuke@REL1_38] Bump required MW version to 1.35.6

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

Change 771709 merged by jenkins-bot:

[mediawiki/extensions/Nuke@REL1_37] Bump required MW version to 1.35.6

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

Change 771710 merged by jenkins-bot:

[mediawiki/extensions/Nuke@REL1_36] Bump required MW version to 1.35.6

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

Change 771711 merged by jenkins-bot:

[mediawiki/extensions/Nuke@REL1_35] Bump required MW version to 1.35.6

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