Page MenuHomePhabricator

Empty PHP entry points where JSON entry points exist
Open, HighPublic

Description

Although most PHP entry points have been emptied when they were converted to have an extension.json, I've come across a handful (at least), that the PHP entry points are still mostly as is... Sometimes with comments "these need to be kept in sync"

Kill them, kill them with fire.

Here you can find list of extensions where this have to be done.

Event Timeline

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

Change 460170 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Remove php entry point

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

Change 460128 merged by Jforrester:
[mediawiki/extensions/BounceHandler@master] Remove php entry point

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

Change 459239 abandoned by Zoranzoki21:
New version 1.0

Reason:
T205053

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

Change 460125 had a related patch set uploaded (by Jforrester; owner: Reedy):
[mediawiki/extensions/Babel@master] Remove PHP entry point

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

Change 460125 merged by jenkins-bot:
[mediawiki/extensions/Babel@master] Remove PHP entry point

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

Change 460137 merged by jenkins-bot:
[mediawiki/extensions/cldr@master] Remove PHP entry point

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

Change 490118 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/Translate@master] Remove PHP entry point

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

Change 500207 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AddMessages@master] Remove php entry point

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

Change 500208 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AddThis@master] Remove php entry point

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

Change 500209 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AdManager@master] Remove php entry point

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

Change 500210 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AkismetKlik@master] Remove php entry point

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

Change 500211 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AllTimeZones@master] Remove php entry point

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

Change 500212 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AnchorHandler@master] Remove php entry point

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

Change 500213 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/ApprovedRevs@master] Remove php entry point

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

Change 500214 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Arrays@master] Remove php entry point

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

Change 500215 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/ArticleIndex@master] Remove php entry point

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

Change 500216 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Blackout@master] Remove php entry point

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

Change 500217 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BlockAndNuke@master] Remove php entry point

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

Change 500218 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BreadCrumbs@master] Remove php entry point

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

Change 500219 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BreadCrumbs2@master] Remove php entry point

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

Change 500220 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Cargo@master] Remove php entry point

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

Change 500220 abandoned by Nikitrain:
Remove php entry point

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

Change 500221 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/CleanChanges@master] Remove php entry point

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

Change 500222 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/CloseWikis@master] Remove php entry point

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

@Krinkle Shouldn't this PHP entry points be removed after a previous and announced deprecation period? Also, shall we cherry-pick this removals to REL1_3(1,2)? Thanks.

@Krinkle Shouldn't this PHP entry points be removed after a previous and announced deprecation period? Also, shall we cherry-pick this removals to REL1_3(1,2)? Thanks.

Why would we cherry pick them? They’re not fixing any bugs

Also, cherry picking would essentially break back compact inside a branch, negating any deprecation period that might’ve happened

@Krinkle Shouldn't this PHP entry points be removed after a previous and announced deprecation period? Also, shall we cherry-pick this removals to REL1_3(1,2)? Thanks.

Why would we cherry pick them? They’re not fixing any bugs

I asked because I don't know the answer. Thanks for providing one.

Also, cherry picking would essentially break back compact inside a branch, negating any deprecation period that might’ve happened

Which we hadn't, unless we don't need to use @deprecated or wfDeprecated ( __METHOD__ , xx ); for this. I know that the PHP entry point does output a wfWarn message. If that is enough for these cases, then okay.

Thank you.

Change 500208 merged by jenkins-bot:
[mediawiki/extensions/AddThis@master] Remove php entry point

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

Change 500221 merged by jenkins-bot:
[mediawiki/extensions/CleanChanges@master] Remove php entry point

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

Change 500212 merged by jenkins-bot:
[mediawiki/extensions/AnchorHandler@master] Remove php entry point

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

Change 500211 merged by jenkins-bot:
[mediawiki/extensions/AllTimeZones@master] Remove php entry point

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

Change 500214 merged by jenkins-bot:
[mediawiki/extensions/Arrays@master] Remove php entry point

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

Change 500210 merged by jenkins-bot:
[mediawiki/extensions/AkismetKlik@master] Remove php entry point

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

Change 500219 merged by jenkins-bot:
[mediawiki/extensions/BreadCrumbs2@master] Remove php entry point

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

Change 500218 merged by jenkins-bot:
[mediawiki/extensions/BreadCrumbs@master] Remove php entry point

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

Change 500215 merged by jenkins-bot:
[mediawiki/extensions/ArticleIndex@master] Remove php entry point

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

Change 500216 merged by jenkins-bot:
[mediawiki/extensions/Blackout@master] Remove php entry point

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

Change 500207 merged by jenkins-bot:
[mediawiki/extensions/AddMessages@master] Remove php entry point

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

Krinkle removed a subscriber: Krinkle.Mon, Apr 1, 3:47 PM

Change 500220 restored by Nikitrain:
Remove php entry point

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

Change 500209 merged by jenkins-bot:
[mediawiki/extensions/AdManager@master] Remove php entry point

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

Change 501664 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/AutoProxyBlock@master] Remove php entry point

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

Change 501668 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BaseHooks@master] Remove php entry point

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

Change 501670 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BatchUserRights@master] Remove php entry point

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

Change 501671 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BiblioPlus@master] Remove php entry point

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

Change 501672 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/BibManager@master] Remove php entry point

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

Change 501675 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Carp@master] Remove php entry point

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

Change 501679 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/EditUser@master] Remove php entry point

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

Change 501680 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/EditNotify@master] Remove php entry point

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

Change 501682 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Editcount@master] Remove php entry point

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

Change 501683 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Description2@master] Remove php entry point

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

Change 501682 merged by jenkins-bot:
[mediawiki/extensions/Editcount@master] Remove php entry point

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

sbassett removed a subscriber: sbassett.Fri, Apr 5, 7:50 PM

Change 501684 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/DeleteBatch@master] Remove php entry point

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

Change 501690 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/DebugTemplates@master] Remove php entry point

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

Change 501691 had a related patch set uploaded (by Nikitrain; owner: Nikitrain):
[mediawiki/extensions/Checkpoint@master] Remove php entry point

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

Change 501680 merged by jenkins-bot:
[mediawiki/extensions/EditNotify@master] Remove php entry point

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

Change 501671 merged by jenkins-bot:
[mediawiki/extensions/BiblioPlus@master] Remove php entry point

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

Change 501675 abandoned by Nikitrain:
Remove php entry point

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

Change 501691 merged by jenkins-bot:
[mediawiki/extensions/Checkpoint@master] Remove php entry point

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

Change 501684 merged by jenkins-bot:
[mediawiki/extensions/DeleteBatch@master] Remove php entry point

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

Change 501668 merged by jenkins-bot:
[mediawiki/extensions/BaseHooks@master] Remove php entry point

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

Change 501672 merged by jenkins-bot:
[mediawiki/extensions/BibManager@master] Remove php entry point

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

Change 501664 merged by jenkins-bot:
[mediawiki/extensions/AutoProxyBlock@master] Remove php entry point

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

Change 501670 merged by jenkins-bot:
[mediawiki/extensions/BatchUserRights@master] Remove php entry point

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

Change 501683 merged by jenkins-bot:
[mediawiki/extensions/Description2@master] Remove php entry point

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

Change 501679 merged by jenkins-bot:
[mediawiki/extensions/EditUser@master] Remove php entry point

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

Change 500222 merged by jenkins-bot:
[mediawiki/extensions/CloseWikis@master] Remove php entry point

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

Change 501875 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/AutoCreateCategoryPages@master] Remove php entry point for extension AutoCreateCategoryPages

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

Change 501877 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MathSearch@master] Empty PHP entrypoint in extension MathSearch

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

Change 501878 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MetaMaster@master] Empty PHP entrypoint in extension MetaMaster

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

Change 501878 abandoned by Jeropbrendawm:
Empty PHP entrypoint in extension MetaMaster

Reason:
Wrong branch

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

Change 501879 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MetaMaster@master] Empty PHP entrypoint in extension MathSearch

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

Change 501880 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MinimumNameLength@master] Empty PHP entry point in extension MinimumNameLength

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

Change 501882 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MobileApp@master] Empty PHP entry point in extension MobileApp

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

Change 501883 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/Mpdf@master] Empty PHP entry point in extension Mpdf

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

Change 501884 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MsCalendar@master] Empty PHP entry point in extension MsCalendar

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

Change 501886 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MsLinks@master] Empty PHP entry point in extension MsLinks

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

Change 501887 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MsUpload@master] Empty PHP entry point in extension MsUpload

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

Change 501888 had a related patch set uploaded (by Jeropbrendawm; owner: Jeropbrendawm):
[mediawiki/extensions/MultiBoilerplate@master] Empty PHP entry point in extension MultiBoilerplate

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

Change 501933 had a related patch set uploaded (by Zoranzoki21; owner: Zoranzoki21):
[mediawiki/extensions/AdminLinks@master] Remove PHP entry point

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

Change 501690 merged by jenkins-bot:
[mediawiki/extensions/DebugTemplates@master] Remove php entry point

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

Change 500217 merged by jenkins-bot:
[mediawiki/extensions/BlockAndNuke@master] Remove php entry point

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

Change 490118 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/Translate@master] Remove PHP entry point

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

Change 503694 had a related patch set uploaded (by Zoranzoki21; owner: Zoranzoki21):
[mediawiki/extensions/WindowsAzureStorage@master] Convert WindowsAzureStorage to use extension registration

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