Page MenuHomePhabricator

Hard deprecate class Hooks with all deprecated functions (and remove in 1.42)
Closed, ResolvedPublic

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
mediawiki/extensions/RegexBlockREL1_39+5 -3
mediawiki/coremaster+10 -528
mediawiki/extensions/BetaFeaturesREL1_40+7 -7
mediawiki/extensions/MathREL1_40+12 -5
mediawiki/extensions/InterwikiREL1_40+1 -2
mediawiki/extensions/WSOAuthREL1_40+31 -10
mediawiki/extensions/WSOAuthREL1_39+31 -10
mediawiki/extensions/CentralNoticewmf/1.41.0-wmf.10+5 -2
mediawiki/coremaster+25 -5
mediawiki/extensions/Favoritesmaster+4 -2
mediawiki/extensions/RegexBlockmaster+5 -3
mediawiki/extensions/TinyMCEmaster+2 -2
mediawiki/extensions/WikEdDiffmaster+2 -29
mediawiki/extensions/SemanticPageSeriesmaster+3 -2
mediawiki/extensions/OAuthAuthenticationmaster+3 -3
mediawiki/extensions/OpenIDmaster+3 -3
mediawiki/extensions/NetworkAuthmaster+2 -2
mediawiki/coremaster+321 -192
mediawiki/extensions/NSFileRepomaster+5 -4
mediawiki/extensions/NSFileRepoREL1_39+5 -4
mediawiki/extensions/Echomaster+2 -3
mediawiki/extensions/Mathmaster+12 -5
mediawiki/extensions/Interwikimaster+1 -2
mediawiki/extensions/Quizmaster+3 -3
mediawiki/extensions/TemplateStylesmaster+3 -3
mediawiki/extensions/GlobalUserPagemaster+1 -2
mediawiki/extensions/Translatemaster+2 -2
mediawiki/extensions/Scribuntomaster+5 -3
mediawiki/extensions/BetaFeaturesmaster+7 -7
mediawiki/extensions/PhpTagsmaster+2 -2
mediawiki/extensions/WikimediaEventsmaster+0 -1
mediawiki/skins/MinervaNeuemaster+1 -2
mediawiki/coremaster+6 -2
mediawiki/coremaster+8 -5
mediawiki/extensions/Echomaster+25 -20
mediawiki/extensions/ORESmaster+5 -5
mediawiki/extensions/Flowmaster+4 -6
mediawiki/extensions/CentralNoticemaster+5 -2
mediawiki/extensions/EmailAuthmaster+4 -4
mediawiki/extensions/PageFormsmaster+37 -29
mediawiki/extensions/FeaturedFeedsmaster+1 -2
mediawiki/extensions/MathSearchmaster+2 -2
mediawiki/extensions/BlueSpicePageVersionmaster+8 -4
mediawiki/extensions/NewUserNotifmaster+4 -2
mediawiki/extensions/SemanticInternalObjectsmaster+3 -2
mediawiki/extensions/Auth_remoteusermaster+15 -14
mediawiki/skins/HasSomeColoursmaster+0 -22
mediawiki/extensions/SocialProfilemaster+15 -15
mediawiki/extensions/PagePropertiesmaster+7 -7
mediawiki/extensions/PhpTagsStoragemaster+1 -1
mediawiki/extensions/PhpTagsFunctionsmaster+2 -2
mediawiki/extensions/PhpTagsWidgetsmaster+3 -3
mediawiki/extensions/Formmaster+2 -2
mediawiki/extensions/GoogleSiteSearchmaster+4 -2
mediawiki/extensions/ImageComparemaster+4 -2
mediawiki/extensions/Farmermaster+5 -4
mediawiki/extensions/OpenGraphMetamaster+2 -2
mediawiki/extensions/PageSchemasmaster+4 -2
mediawiki/extensions/RandomGameUnitmaster+2 -2
mediawiki/extensions/SocialLoginmaster+5 -5
mediawiki/extensions/SphinxSearchmaster+4 -3
mediawiki/extensions/ContentTransfermaster+5 -5
mediawiki/extensions/MarkAsHelpfulmaster+7 -3
mediawiki/extensions/SiteSettingsmaster+36 -27
mediawiki/extensions/WSOAuthmaster+31 -10
mediawiki/skins/DeskMessMirroredmaster+3 -1
mediawiki/skins/Bouquetmaster+1 -1
mediawiki/skins/apexmaster+1 -1
mediawiki/skins/Duskmaster+4 -2
mediawiki/skins/DuskToDawnmaster+1 -1
mediawiki/skins/eruditemaster+3 -1
mediawiki/skins/Gamepressmaster+3 -1
mediawiki/skins/Foregroundmaster+3 -1
mediawiki/skins/Metrolookmaster+6 -4
mediawiki/skins/Nimbusmaster+1 -1
mediawiki/skins/p2wikimaster+3 -1
mediawiki/skins/Pivotmaster+3 -1
mediawiki/skins/Truglassmaster+4 -2
mediawiki/skins/Refreshedmaster+2 -2
mediawiki/skins/webplatformmaster+0 -3
mediawiki/extensions/EmailDiffmaster+2 -2
mediawiki/extensions/FanBoxesmaster+0 -1
mediawiki/extensions/UserMergemaster+6 -4
mediawiki/extensions/PhpTagsWikimaster+1 -1
mediawiki/extensions/AdminLinksmaster+1 -1
mediawiki/extensions/BatchUserRightsmaster+3 -2
mediawiki/extensions/ConfirmAccountmaster+1 -1
mediawiki/extensions/ApprovedRevsmaster+12 -6
mediawiki/extensions/Mpdfmaster+3 -1
mediawiki/extensions/MultiLanguageManagermaster+4 -6
mediawiki/extensions/Formsmaster+2 -1
mediawiki/extensions/PhpTagsmaster+4 -2
mediawiki/extensions/SearchParserFunctionmaster+5 -3
mediawiki/extensions/SharedHelpPagesmaster+1 -1
mediawiki/extensions/StaffPowersmaster+1 -1
mediawiki/extensions/UserGroupsmaster+4 -2
mediawiki/extensions/TwnMainPagemaster+1 -1
mediawiki/extensions/WatchAnalyticsmaster+2 -2
mediawiki/extensions/VEForAllmaster+2 -2
mediawiki/extensions/WikiForummaster+1 -1
mediawiki/extensions/Commentsmaster+4 -3
mediawiki/extensions/GlobalContribsmaster+4 -4
mediawiki/extensions/Pushmaster+8 -4
mediawiki/extensions/Cargomaster+7 -5
mediawiki/extensions/LiveChatmaster+6 -6
mediawiki/extensions/BibManagermaster+13 -9
mediawiki/extensions/Translatemaster+19 -17
mediawiki/extensions/TorBlockmaster+13 -3
mediawiki/extensions/XAnalyticsmaster+2 -2
mediawiki/extensions/LoginNotifymaster+4 -3
mediawiki/extensions/LiquidThreadsmaster+61 -48
mediawiki/extensions/Mathmaster+3 -4
mediawiki/extensions/ContactPagemaster+5 -5
mediawiki/extensions/WikimediaEventsmaster+1 -1
mediawiki/extensions/FileImportermaster+7 -6
mediawiki/extensions/CodeEditormaster+13 -5
mediawiki/extensions/GrowthExperimentsmaster+1 -2
mediawiki/extensions/Wikibasemaster+19 -6
mediawiki/extensions/Wikibasemaster+109 -31
mediawiki/extensions/WikibaseMediaInfomaster+5 -4
mediawiki/coremaster+0 -11
mediawiki/extensions/LdapAuthenticationmaster+10 -7
mediawiki/extensions/BetaFeaturesmaster+6 -6
mediawiki/extensions/IPInfomaster+1 -2
mediawiki/extensions/WikimediaMaintenancemaster+1 -1
Show related patches Customize query in gerrit

Related Objects

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

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

[mediawiki/skins/MinervaNeue@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/BetaFeatures@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/GlobalUserPage@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/Interwiki@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/PhpTags@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/Scribunto@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/TemplateStyles@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/Quiz@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/Translate@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/Math@master] Replace deprecated Hooks::run

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

Change 916799 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/WikimediaEvents@master] Removed unused Hooks class import

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

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

[mediawiki/core@master] Hard-deprecate Hooks::isRegistered/getHandlers/run/runner

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

Change 916800 merged by jenkins-bot:

[mediawiki/extensions/BetaFeatures@master] Replace deprecated Hooks::run

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

Change 916809 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] Removed unused Hooks class import

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

Change 916807 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Replace deprecated Hooks::run

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

Change 916804 merged by jenkins-bot:

[mediawiki/extensions/Scribunto@master] Replace deprecated Hooks::run

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

Change 916805 merged by jenkins-bot:

[mediawiki/extensions/TemplateStyles@master] Replace deprecated Hooks::run

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

Change 916803 merged by jenkins-bot:

[mediawiki/extensions/PhpTags@master] Replace deprecated Hooks::run

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

Change 916801 merged by jenkins-bot:

[mediawiki/extensions/GlobalUserPage@master] Replace deprecated Hooks::run

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

Change 916806 merged by jenkins-bot:

[mediawiki/extensions/Quiz@master] Replace deprecated Hooks::run

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

Change 916802 merged by jenkins-bot:

[mediawiki/extensions/Interwiki@master] Replace deprecated Hooks::register

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

Change 916808 merged by jenkins-bot:

[mediawiki/extensions/Math@master] Replace deprecated Hooks::run

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

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

[mediawiki/extensions/Echo@master] Replace deprecated Hooks::run

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

Change 916822 merged by jenkins-bot:

[mediawiki/extensions/Echo@master] Replace deprecated Hooks::run

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

Change 916747 merged by Robert Vogel:

[mediawiki/extensions/NSFileRepo@master] Replace deprecated Hooks::run

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

Change 917726 had a related patch set uploaded (by Robert Vogel; author: Umherirrender):

[mediawiki/extensions/NSFileRepo@REL1_39] Replace deprecated Hooks::run

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

Change 917726 merged by Robert Vogel:

[mediawiki/extensions/NSFileRepo@REL1_39] Replace deprecated Hooks::run

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

Change 916796 merged by jenkins-bot:

[mediawiki/core@master] Replace internal Hooks::runner

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

Change 916745 merged by Umherirrender:

[mediawiki/extensions/NetworkAuth@master] Replace deprecated Hooks::run

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

Change 916750 merged by Umherirrender:

[mediawiki/extensions/OpenID@master] Replace deprecated Hooks::run

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

Change 916751 merged by Umherirrender:

[mediawiki/extensions/OAuthAuthentication@master] Replace deprecated Hooks::run

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

Change 916760 merged by Umherirrender:

[mediawiki/extensions/SemanticPageSeries@master] Replace deprecated Hooks::run

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

Change 916772 merged by Umherirrender:

[mediawiki/extensions/WikEdDiff@master] Remove comment out code about deprecated Hooks::run

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

Change 916765 merged by Umherirrender:

[mediawiki/extensions/TinyMCE@master] Replace deprecated Hooks::run

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

Change 916756 merged by Umherirrender:

[mediawiki/extensions/RegexBlock@master] Replace deprecated Hooks::run

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

Change 916776 merged by Umherirrender:

[mediawiki/extensions/Favorites@master] Replace deprecated Hooks::run

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

Change 916810 merged by jenkins-bot:

[mediawiki/core@master] Hard-deprecate Hooks::isRegistered/getHandlers/run/runWithoutAbort

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

Umherirrender renamed this task from Hard deprecate class Hooks with all deprecated functions (and remove class a release later) to Hard deprecate class Hooks with all deprecated functions (and remove in 1.42).May 19 2023, 3:16 PM
Umherirrender changed the task status from Open to Stalled.
Umherirrender removed a project: Patch-For-Review.

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

[mediawiki/extensions/CentralNotice@wmf/1.41.0-wmf.10] Replace deprecated Hooks::runWithoutAbort

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

Change 923283 merged by jenkins-bot:

[mediawiki/extensions/CentralNotice@wmf/1.41.0-wmf.10] Replace deprecated Hooks::runWithoutAbort

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

Mentioned in SAL (#wikimedia-operations) [2023-05-25T22:05:28Z] <zabe@deploy1002> Started scap: Backport for [[gerrit:923283|Replace deprecated Hooks::runWithoutAbort (T335536)]], [[gerrit:923276|BannerRenderer: Make sure the language variant is valid (T337427)]]

Mentioned in SAL (#wikimedia-operations) [2023-05-25T22:06:59Z] <zabe@deploy1002> zabe and ladsgroup: Backport for [[gerrit:923283|Replace deprecated Hooks::runWithoutAbort (T335536)]], [[gerrit:923276|BannerRenderer: Make sure the language variant is valid (T337427)]] synced to the testservers: mwdebug1001.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-05-25T22:14:42Z] <zabe@deploy1002> Finished scap: Backport for [[gerrit:923283|Replace deprecated Hooks::runWithoutAbort (T335536)]], [[gerrit:923276|BannerRenderer: Make sure the language variant is valid (T337427)]] (duration: 09m 14s)

Change 926788 had a related patch set uploaded (by Cicalese; author: Umherirrender):

[mediawiki/extensions/WSOAuth@REL1_39] Replace deprecated Hooks::run

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

Change 926788 merged by Cicalese:

[mediawiki/extensions/WSOAuth@REL1_39] Replace deprecated Hooks::run

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

Change 926794 had a related patch set uploaded (by Cicalese; author: Umherirrender):

[mediawiki/extensions/WSOAuth@REL1_40] Replace deprecated Hooks::run

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

Change 926794 merged by Cicalese:

[mediawiki/extensions/WSOAuth@REL1_40] Replace deprecated Hooks::run

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

Change 948587 had a related patch set uploaded (by Paladox; author: Umherirrender):

[mediawiki/extensions/Interwiki@REL1_40] Replace deprecated Hooks::register

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

Change 948587 merged by jenkins-bot:

[mediawiki/extensions/Interwiki@REL1_40] Replace deprecated Hooks::register

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

Ladsgroup subscribed.

Let's make it a blocker so we can remove it soon

The title says “remove in 1.42”, why did you add MW-1.41-release rather than MW-1.42-release?

It needs to be hard-deprecated before the 1.41 release and then we can start removing anything in 1.42

I see. Maybe it could be mentioned in the task description that it should be hard-deprecated in 1.41?

All (static) functions of the class are hard-deprecated, this task is stalled until release of 1.42 to remove the class.

wow thanks. Last I checked it wasn't done. Thanks!

Change 952439 had a related patch set uploaded (by Paladox; author: Umherirrender):

[mediawiki/extensions/Math@REL1_40] Replace deprecated Hooks::run

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

Change 952439 merged by Umherirrender:

[mediawiki/extensions/Math@REL1_40] Replace deprecated Hooks::run

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

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

[mediawiki/core@master] Remove Hooks class

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

Umherirrender changed the task status from Stalled to Open.Oct 12 2023, 7:58 PM
Umherirrender claimed this task.

Change 967492 had a related patch set uploaded (by Paladox; author: Umherirrender):

[mediawiki/extensions/BetaFeatures@REL1_40] Replace deprecated Hooks::run

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

Change 967492 abandoned by Paladox:

[mediawiki/extensions/BetaFeatures@REL1_40] Replace deprecated Hooks::run

Reason:

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

Change 965587 merged by jenkins-bot:

[mediawiki/core@master] Remove Hooks class, all contents deprecated in 1.35 or 1.41

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

Change #1160974 had a related patch set uploaded (by Reedy; author: Umherirrender):

[mediawiki/extensions/RegexBlock@REL1_39] Replace deprecated Hooks::run

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

Change #1160974 merged by Umherirrender:

[mediawiki/extensions/RegexBlock@REL1_39] Replace deprecated Hooks::run

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