[go: nahoru, domu]

Page MenuHomePhabricator

Remove unnecessary targets definitions
Closed, ResolvedPublic

Description

NOTE: Feel free to untag your project and create subtasks if needed.

In https://gerrit.wikimedia.org/r/c/mediawiki/core/+/865811 we updated the default for targets to be ["desktop", "mobile"]. This means that any extensions/skins defining this are now redundant and can remove these definitions,

The following query marks some of the offenders (regex to be improved later):
https://codesearch.wmcloud.org/search/?q=%22targets%22&i=nope&files=(extension%7Cskin).json&excludeFiles=&repos=

TODO

  • Remove the lines
  • Bump the MediaWiki required version to 1.40

Details

SubjectRepoBranchLines +/-
mediawiki/extensions/CentralNoticewmf_deploy+0 -0
mediawiki/extensions/Gadgetsmaster+2 -61
mediawiki/extensions/UniversalLanguageSelectormaster+0 -48
mediawiki/extensions/WikimediaEventsmaster+0 -1
mediawiki/extensions/Translatemaster+11 -151
mediawiki/extensions/ConfirmEditmaster+1 -5
mediawiki/extensions/VisualEditormaster+0 -4
mediawiki/extensions/TemplateDatamaster+0 -1
mediawiki/extensions/ContentTranslationmaster+0 -1
mediawiki/extensions/WikibaseLexememaster+0 -16
mediawiki/extensions/AdvancedSearchmaster+0 -5
mediawiki/extensions/Citoidmaster+0 -3
mediawiki/extensions/CentralNoticemaster+0 -15
mediawiki/extensions/QuickSurveysmaster+0 -1
mediawiki/extensions/ConfirmEditmaster+0 -1
mediawiki/extensions/GlobalCssJsmaster+0 -7
mediawiki/extensions/Babelmaster+2 -6
mediawiki/extensions/CentralNoticemaster+0 -44
mediawiki/extensions/ConfirmEditmaster+4 -28
mediawiki/extensions/ORESmaster+1 -9
mediawiki/extensions/NavigationTimingmaster+0 -4
mediawiki/extensions/WebAuthnmaster+0 -20
mediawiki/extensions/DismissableSiteNoticemaster+1 -9
mediawiki/extensions/DiscussionToolsmaster+2 -18
mediawiki/extensions/CodeEditormaster+0 -20
mediawiki/extensions/MachineVisionmaster+1 -9
mediawiki/extensions/MediaSearchmaster+1 -9
mediawiki/extensions/WikibaseMediaInfomaster+0 -36
mediawiki/extensions/GlobalWatchlistmaster+0 -12
mediawiki/extensions/CharInsertmaster+0 -8
mediawiki/extensions/AbuseFiltermaster+1 -5
mediawiki/extensions/wikihieromaster+0 -4
mediawiki/extensions/timelinemaster+0 -4
mediawiki/extensions/VueTestmaster+4 -20
mediawiki/extensions/Scoremaster+0 -4
mediawiki/extensions/CentralAuthmaster+1 -17
mediawiki/extensions/SyntaxHighlight_GeSHimaster+0 -12
mediawiki/extensions/ArticlePlaceholdermaster+1 -5
mediawiki/extensions/FlaggedRevsmaster+1 -13
mediawiki/extensions/WikiEditormaster+0 -8
mediawiki/extensions/UploadWizardmaster+1 -17
mediawiki/extensions/WikimediaBadgesmaster+0 -4
mediawiki/extensions/Wikidata.orgmaster+0 -4
mediawiki/extensions/VisualEditormaster+0 -4
mediawiki/extensions/TitleBlacklistmaster+1 -9
mediawiki/extensions/OATHAuthmaster+0 -4
mediawiki/extensions/CategoryTreemaster+0 -8
mediawiki/extensions/ImageSuggestionsmaster+0 -4
mediawiki/extensions/WikimediaEventsmaster+0 -12
mediawiki/extensions/Quizmaster+2 -10
mediawiki/extensions/BetaFeaturesmaster+0 -8
mediawiki/extensions/InputBoxmaster+1 -5
mediawiki/extensions/SpamBlacklistmaster+1 -5
mediawiki/extensions/WikibaseQualityConstraintsmaster+0 -4
mediawiki/extensions/Thanksmaster+0 -16
mediawiki/extensions/GrowthExperimentsmaster+0 -8
mediawiki/extensions/Wikisourcemaster+0 -4
mediawiki/extensions/CampaignEventsmaster+0 -32
mediawiki/extensions/CodeEditormaster+0 -20
mediawiki/extensions/TimedMediaHandlermaster+3 -43
mediawiki/extensions/ChessBrowsermaster+0 -4
mediawiki/extensions/Citemaster+1 -1
mediawiki/extensions/Echomaster+1 -49
mediawiki/extensions/Citemaster+3 -31
mediawiki/extensions/TwoColConflictmaster+0 -4
mediawiki/extensions/EventLoggingmaster+0 -8
mediawiki/extensions/Citoidmaster+1 -9
mediawiki/extensions/SearchVuemaster+0 -4
mediawiki/extensions/AdvancedSearchmaster+0 -20
mediawiki/coremaster+0 -14
mediawiki/extensions/ProofreadPagemaster+3 -35
mediawiki/extensions/ContentTranslationmaster+14 -118
mediawiki/extensions/VisualEditormaster+11 -247
mediawiki/extensions/MobileFrontendmaster+26 -66
mediawiki/extensions/Mathmaster+2 -26
mediawiki/extensions/WikiLambdamaster+1 -17
mediawiki/extensions/WikidataPageBannermaster+4 -20
mediawiki/extensions/QuickSurveysmaster+1 -15
mediawiki/extensions/RelatedArticlesmaster+2 -14
mediawiki/extensions/NearbyPagesmaster+1 -3
mediawiki/skins/MinervaNeuemaster+0 -41
mediawiki/skins/Vectormaster+0 -8
mediawiki/extensions/Wikistoriesmaster+0 -4
mediawiki/extensions/Popupsmaster+3 -3
mediawiki/skins/MonoBookmaster+0 -1
mediawiki/extensions/Scribuntomaster+1 -3
mediawiki/coremaster+0 -180
mediawiki/extensions/Kartographermaster+2 -90
mediawiki/extensions/ReadingListsmaster+1 -3
mediawiki/coremaster+1 -4
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 990061 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990114 merged by jenkins-bot:

[mediawiki/extensions/Wikisource@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990090 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990102 merged by jenkins-bot:

[mediawiki/extensions/Thanks@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990110 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990100 merged by jenkins-bot:

[mediawiki/extensions/SpamBlacklist@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990092 merged by jenkins-bot:

[mediawiki/extensions/InputBox@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990091 merged by jenkins-bot:

[mediawiki/extensions/ImageSuggestions@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990113 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990096 merged by jenkins-bot:

[mediawiki/extensions/OATHAuth@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990062 merged by jenkins-bot:

[mediawiki/extensions/CategoryTree@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990059 merged by jenkins-bot:

[mediawiki/extensions/BetaFeatures@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990103 merged by jenkins-bot:

[mediawiki/extensions/TitleBlacklist@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990098 merged by jenkins-bot:

[mediawiki/extensions/Quiz@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990105 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990111 merged by jenkins-bot:

[mediawiki/extensions/Wikidata.org@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990112 merged by jenkins-bot:

[mediawiki/extensions/WikimediaBadges@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990106 merged by jenkins-bot:

[mediawiki/extensions/WikiEditor@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990104 merged by jenkins-bot:

[mediawiki/extensions/UploadWizard@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990101 merged by jenkins-bot:

[mediawiki/extensions/SyntaxHighlight_GeSHi@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990058 merged by jenkins-bot:

[mediawiki/extensions/ArticlePlaceholder@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990088 merged by jenkins-bot:

[mediawiki/extensions/FlaggedRevs@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990063 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990099 merged by jenkins-bot:

[mediawiki/extensions/Score@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990116 merged by jenkins-bot:

[mediawiki/extensions/wikihiero@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990107 merged by jenkins-bot:

[mediawiki/extensions/VueTest@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990115 merged by jenkins-bot:

[mediawiki/extensions/timeline@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990065 merged by jenkins-bot:

[mediawiki/extensions/CharInsert@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990057 merged by jenkins-bot:

[mediawiki/extensions/AbuseFilter@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990089 merged by jenkins-bot:

[mediawiki/extensions/GlobalWatchlist@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990109 merged by jenkins-bot:

[mediawiki/extensions/WikibaseMediaInfo@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990093 merged by jenkins-bot:

[mediawiki/extensions/MachineVision@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990094 merged by jenkins-bot:

[mediawiki/extensions/MediaSearch@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990086 merged by jenkins-bot:

[mediawiki/extensions/CodeEditor@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990087 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990156 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/ConfirmEdit@master] */extension.json: Drop RL targets definitions, no longer honoured

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

Change 990159 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/DismissableSiteNotice@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990159 merged by jenkins-bot:

[mediawiki/extensions/DismissableSiteNotice@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990108 merged by jenkins-bot:

[mediawiki/extensions/WebAuthn@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990095 merged by jenkins-bot:

[mediawiki/extensions/NavigationTiming@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990097 merged by jenkins-bot:

[mediawiki/extensions/ORES@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 990156 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] */extension.json: Drop RL targets definitions, no longer honoured

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

Change 990064 merged by jenkins-bot:

[mediawiki/extensions/CentralNotice@master] extension.json: Drop RL targets definitions, no longer honoured

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

Change 906607 merged by jenkins-bot:

[mediawiki/extensions/Babel@master] Remove unnecessary targets definitions and bump required version to MW 1.40

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

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

[mediawiki/extensions/VisualEditor@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/WikibaseLexeme@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/GlobalCssJs@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/AdvancedSearch@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/CentralNotice@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/Citoid@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/ConfirmEdit@master] Drop RL targets definitions from module definition, no longer honoured

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

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

[mediawiki/extensions/ContentTranslation@master] Drop RL targets definitions from module definition, no longer honoured

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

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

[mediawiki/extensions/TemplateData@master] Drop RL targets definitions from module definition, no longer honoured

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

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

[mediawiki/extensions/UniversalLanguageSelector@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/QuickSurveys@master] tests: Remove call to Module::getTargets

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

Change 992505 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] Drop RL targets definitions from module definition, no longer honoured

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

Change 992501 merged by jenkins-bot:

[mediawiki/extensions/GlobalCssJs@master] Drop RL targets definitions from module class, no longer honoured

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

Change 992532 merged by jenkins-bot:

[mediawiki/extensions/QuickSurveys@master] tests: Remove call to Module::getTargets

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

Change 992503 merged by jenkins-bot:

[mediawiki/extensions/CentralNotice@master] Drop RL targets definitions from module class, no longer honoured

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

Change 992502 merged by jenkins-bot:

[mediawiki/extensions/AdvancedSearch@master] Drop RL targets definitions from module class, no longer honoured

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

Change 992504 merged by jenkins-bot:

[mediawiki/extensions/Citoid@master] Drop RL targets definitions from module class, no longer honoured

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

Change 992500 merged by jenkins-bot:

[mediawiki/extensions/WikibaseLexeme@master] Drop RL targets definitions from module class, no longer honoured

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

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

[mediawiki/extensions/ConfirmEdit@master] Turnstile: Drop RL targets definitions, no longer honoured

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

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

[mediawiki/extensions/WikimediaEvents@master] Drop RL targets definitions from module definition, no longer honoured

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

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

[mediawiki/extensions/Translate@master] Drop RL targets definitions from module class

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

Change 992527 merged by jenkins-bot:

[mediawiki/extensions/ContentTranslation@master] Drop RL targets definitions from module definition, no longer honoured

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

Change 992528 merged by jenkins-bot:

[mediawiki/extensions/TemplateData@master] Drop RL targets definitions from module definition, no longer honoured

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

Change 992497 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Drop RL targets definitions from module class, no longer honoured

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

Change 992546 merged by jenkins-bot:

[mediawiki/extensions/Translate@master] Drop RL targets definitions from module class

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

Change 992544 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] Turnstile: Drop RL targets definitions, no longer honoured

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

Change 992545 merged by jenkins-bot:

[mediawiki/extensions/WikimediaEvents@master] Drop RL targets definitions from module definition, no longer honoured

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

Change 992529 merged by jenkins-bot:

[mediawiki/extensions/UniversalLanguageSelector@master] Drop RL targets definitions from module class

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

Change 992761 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/Gadgets@master] Drop RL targets definitions from module class, and validation / error code

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

Change 992761 merged by jenkins-bot:

[mediawiki/extensions/Gadgets@master] Drop RL targets definitions from module class, and validation / error code

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

Per extension search and PHP array search this appears to now be done for all Wikimedia production code (just waiting for the irregular production branching of CentralNotice). Great work, all!

For non-Wikimedia-production code, there are 86 and 51 hits, respectively (with some false positives).

Jdlrobson claimed this task.

Thanks James! I think we should close out this ticket as some of the remaining non-production code may not be maintained.
I've updated https://www.mediawiki.org/wiki/ResourceLoader/Writing_a_mobile_friendly_ResourceLoader_module with improved guidelines.

Change #1052668 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy

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