Page MenuHomePhabricator
Create Task
Maniphest T177809

Fix multiple PHP class declarations in one file
Open, Needs TriagePublic
Actions

Description

There should be one PHP class per file, not multiple. Any files where this isn't the case should be fixed up.

Find a PHP file that has more than one class/interface in it (in MediaWiki core, a WMF deployed extension, or any other extension), and split the classes into individual files, that are named the same as the class itself.

Update any autoloader references to the class in the old file to point in the new file. In core this is autoload.php, in extensions this may be in the PHP entry point, or in extension.json

Details

SubjectRepoBranchLines +/-
Fix multiple PHP class declarations in one filemediawiki/extensions/WikiLexicalDatamaster+199 -188
Fix multiple PHP class declarations in one filemediawiki/extensions/CirrusSearchmaster+1 K -1 K
Fix multiple PHP class declarations in one filemediawiki/extensions/CentralNoticewmf_deploy+243 -167
Fix multiple PHP class declarations in one filemediawiki/extensions/CentralNoticemaster+243 -167
Fix multiple PHP class declarations in one filemediawiki/extensions/BlueSpiceFoundationmaster+1 K -1 K
Fix multiple PHP class declarations in one filemediawiki/extensions/Wikispeechmaster+19 -13
Fix multiple PHP class declarations in one filemediawiki/extensions/UploadWizardmaster+25 -16
Fix multiple PHP class declarations in one file in testsmediawiki/extensions/Echomaster+31 -30
Fix multiple PHP class declarations in one filemediawiki/extensions/Echomaster+397 -371
Fix multiple PHP class declarations in one file in includes/htmlformmediawiki/coremaster+34 -32
Fix multiple PHP class declarations in one file in includes/diffmediawiki/coremaster+559 -379
Fix multiple PHP class declarations in one file in includes/specialsemediawiki/coremaster+337 -164
Fix multiple PHP class declarations in one file in includes/authmediawiki/coremaster+50 -31
Fix multiple PHP class declarations in one file in includes/utilsmediawiki/coremaster+182 -181
Fix multiple PHP class declarations in one file in includes/loggingmediawiki/coremaster+955 -811
Fix multiple PHP class declarations in one file in includes/parsermediawiki/coremaster+3 K -2 K
Fix multiple PHP class declarations in one file in includes/cachemediawiki/coremaster+374 -259
Show related patches Customize query in gerrit

Related Objects
Search...

Event Timeline

Reedy created this task.Oct 10 2017, 12:34 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 10 2017, 12:34 AM
Reedy updated the task description. (Show Details)Oct 10 2017, 1:01 AM
Aklapper added projects: MediaWiki-extensions-General, MediaWiki-General.Oct 10 2017, 11:45 AM
Aklapper moved this task from Proposed tasks to Information needed on the Google-Code-in-2017 board.Nov 20 2017, 11:01 PM

Find a PHP file that has more than one class/interface in it

@Reedy: How would I or someone else do that? What to grep for, basically?

Aklapper renamed this task from Multiple class declarations in one file to Fix multiple PHP class declarations in one file.Nov 20 2017, 11:01 PM
Reedy added a comment.Dec 5 2017, 4:11 PM

I suspect, it's possible to grep for this... But I imagine it would need some post-processing...

Grep for php files with a class in it, get the file name, group and order by file name.... and exclude if after grouping it's just 1?

Seems excessively complex unless we provided them the command to run

Much easier option is to just (install and) use phpcs, and disable the "MediaWiki.Files.OneClassPerFile.MultipleFound" rule, and fix some....

So... PhpStorm makes a "nice" HTML output of this. I've uploaded a copy for mediawiki (core) includes folder to https://people.wikimedia.org/~reedy/inspections/dupeclasses/includes

Then there's 27 more in maintenance.. 84 in tests (I don't think they're both quite so important... Maybe?)

Ideally, outside of core, we want to target WMF deployed extensions/skins first...

1288 in all extensions. 33 in skins

Various breakdowns/sub folders scanned in https://people.wikimedia.org/~reedy/inspections/dupeclasses/

Reedy updated the task description. (Show Details)Dec 5 2017, 4:11 PM
Pppery added a parent task: T177810: Fix up PHPCS violations that are excluded in .phpcs.xml.Dec 19 2017, 1:18 AM
Pppery subscribed.

<adding parent task because the presence of multiple classes in the same file is a phpcs violation>

Platonides subscribed.Dec 29 2017, 2:04 AM
In T177809#3813320, @Reedy wrote:

I suspect, it's possible to grep for this... But I imagine it would need some post-processing...
Grep for php files with a class in it, get the file name, group and order by file name.... and exclude if after grouping it's just 1?

It's much more simple:

grep "^[a-z]* *class " -rc includes/ | grep -v ':[01]$'

Aklapper added a comment.Dec 31 2017, 1:12 PM

Who would mentor this in Google-Code-in-2017?

In T177809#3863006, @Platonides wrote:

It's much more simple:
grep "^[a-z]* *class " -rc includes/ | grep -v ':[01]$'

Output for /includes subfolder in git master of MediaWiki extensions listed below.
Should I start creating tasks on the GCI website for these? Asking as I'm very clueless about these things.

P6508 T177809
1$:acko\> more ./T177809.sh
2for repos in $(ls); do
3 if [ -d $repos ]; then
4 cd $repos
5 if [ -d "includes" ]; then
6 grep "^[a-z]* *class " -rc includes/ | grep -v ':[01]$'
7 fi
8 cd ..
9 fi
10done
11$:acko\> ./T177809.sh
12==== BlueSpiceFoundation
13includes/ExtJSStoreParams.php:2
14includes/validator/Validator.class.php:9
15includes/html/htmlformfields/HTMLFormFieldOverrides.php:6
16includes/html/htmlformfields/HTMLMultiSelectEx.php:2
17includes/Common.php:9
18
19==== CentralNotice
20includes/Campaign.php:2
21includes/Banner.php:4
22includes/CNBannerPager.php:2
23includes/MixinController.php:2
24
25==== CirrusSearch
26includes/BuildDocument/Builder.php:2
27includes/BuildDocument/Completion/SuggestScoring.php:4
28includes/Sanity/Remediator.php:2
29includes/Search/RescoreBuilders.php:17
30includes/Search/CrossProjectBlockScorer.php:6
31includes/Search/ResultsType.php:6
32
33==== CommunityTwitter
34includes/OAuth.php:11
35
36==== DataTransfer
37includes/DT_XMLParser.php:3
38includes/DT_PageStructure.php:2
39
40==== Echo
41includes/DiffParser.php:2
42includes/ContainmentSet.php:4
43
44==== EditNotify
45includes/ENPageStructure.php:2
46
47==== EducationProgram
48includes/CourseUndeletionHelper.php:2
49includes/LogFormatter.php:3
50includes/OrgDeletionHelper.php:2
51includes/Events/TimelineGroup.php:4
52
53==== EventLogging
54includes/JsonSchema.php:5
55
56==== Flow
57includes/Import/LiquidThreadsApi/Objects.php:10
58includes/Import/LiquidThreadsApi/CachedData.php:4
59includes/Import/LiquidThreadsApi/Iterators.php:3
60includes/Import/LiquidThreadsApi/Source.php:4
61includes/Import/Importer.php:5
62includes/Model/UUID.php:2
63includes/Formatter/ChangesListQuery.php:2
64includes/Formatter/ContributionsQuery.php:3
65includes/Formatter/CheckUserQuery.php:2
66includes/Formatter/AbstractQuery.php:2
67includes/Formatter/RevisionViewQuery.php:4
68includes/Data/Listener/WatchTopicListener.php:2
69includes/Exception/ExceptionHandling.php:17
70
71==== FundraisingEmailUnsubscribe
72includes/MediaWikiTwig.php:2
73
74==== MarkAsHelpful
75includes/MarkAsHelpfulItem.php:3
76
77==== NSFileRepo
78includes/filerepo/file/NSLocalFile.php:2
79
80==== OOUIPlayground
81includes/argument-filters/GroupElementFilter.php:2
82includes/WidgetInfo.php:3
83includes/CodeRenderer.php:7
84includes/WidgetFactory.php:2
85
86==== PageTriage
87includes/PageTriage.php:2
88includes/ArticleMetadata.php:9
89includes/PageTriageUtil.php:2
90
91==== PhpTags
92includes/Renderer.php:2
93
94==== SecurePoll
95includes/main/Store.php:3
96includes/user/Auth.php:3
97includes/crypt/Crypt.php:2
98includes/pages/CreatePage.php:3
99includes/pages/DetailsPage.php:2
100includes/pages/ListPage.php:2
101includes/pages/EntryPage.php:2
102includes/ballots/Ballot.php:2
103
104==== SemanticExpressiveness
105includes/ShortQueryResult.php:2
106
107==== SemanticPageMaker
108includes/widgets/SPM_WidgetDesignPage2.php:2
109includes/widgets/SPM_WidgetDesignPage.php:2
110includes/widgets/SPM_WidgetPage.php:2
111includes/widgets/datatype/SPM_WT_File.php:2
112includes/widgets/datatype/SPM_WT_Widget.php:2
113includes/widgets/datatype/SPM_WT_Page.php:2
114includes/widgets/datatype/SPM_WT_Media.php:2
115
116==== SpellingDictionary
117includes/AdminRights.php:4
118
119==== UploadWizard
120includes/specials/SpecialUploadWizard.php:2
121
122==== WikiLexicalData
123includes/specials/SpecialOWAddFromExternalAPI.php:2
124includes/specials/ExternalWordnik.php:3
125
126==== Wikispeech
127includes/CleanedContent.php:2

nikitavbv claimed this task.Jan 20 2018, 5:25 PM
nikitavbv subscribed.

I will work on fixing this

nikitavbv added a comment.Jan 20 2018, 6:09 PM

There are quite a lot of such files in core so to make things simplier to manage, review and merge I will split it into multiple patches (by directories)

gerritbot subscribed.Jan 20 2018, 6:10 PM

Change 405536 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/cache

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

gerritbot added a project: Patch-For-Review.Jan 20 2018, 6:10 PM
gerritbot added a comment.Jan 20 2018, 9:16 PM

Change 405545 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/parser

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

gerritbot added a comment.Jan 20 2018, 9:34 PM

Change 405547 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/logging

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

gerritbot added a comment.Jan 20 2018, 9:56 PM

Change 405548 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/utils

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

gerritbot added a comment.Jan 20 2018, 10:19 PM

Change 405551 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/specialse

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

gerritbot added a comment.Jan 20 2018, 10:24 PM

Change 405552 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/auth

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

gerritbot added a comment.Jan 20 2018, 10:40 PM

Change 405554 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/diff

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

gerritbot added a comment.Jan 20 2018, 10:48 PM

Change 405555 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/core@master] Fix multiple PHP class declarations in one file in includes/htmlform

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

Anomie subscribed.Jan 20 2018, 11:08 PM

Considering this will already be done by T166010: The Great Namespaceization Effort, it seems somewhat redundant to split files now only to rename them later. See discussion on this topic on T173799: Implement a PSR-4 autoloader in MediaWiki core.

nikitavbv added a comment.EditedJan 20 2018, 11:24 PM

@Anomie Thank you for giving links to those tasks! You are right, there is no need to split files now if general reogranization will be done. So can we close this task as "Declined"?

Reedy added a comment.Jan 20 2018, 11:27 PM

There's still various extensions that need fixing up... I think those bugs are just for MW Core

nikitavbv added a comment.Jan 20 2018, 11:52 PM

@Reedy Thank you! Then I will abandon my changes to core and fix extensions instead.

gerritbot added a comment.Jan 21 2018, 12:20 AM

Change 405536 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/cache

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

gerritbot added a comment.Jan 21 2018, 12:20 AM

Change 405545 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/parser

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

gerritbot added a comment.Jan 21 2018, 12:20 AM

Change 405547 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/logging

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

gerritbot added a comment.Jan 21 2018, 12:21 AM

Change 405548 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/utils

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

gerritbot added a comment.Jan 21 2018, 12:21 AM

Change 405552 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/auth

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

gerritbot added a comment.Jan 21 2018, 12:21 AM

Change 405551 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/specialse

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

gerritbot added a comment.Jan 21 2018, 12:21 AM

Change 405554 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/diff

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

gerritbot added a comment.Jan 21 2018, 12:21 AM

Change 405555 abandoned by Phantom42:
Fix multiple PHP class declarations in one file in includes/htmlform

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

gerritbot added a comment.Jan 21 2018, 8:59 PM

Change 405608 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/Echo@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Jan 21 2018, 9:26 PM

Change 405608 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Fix multiple PHP class declarations in one file

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

ReleaseTaggerBot added a project: MW-1.31-release-notes (WMF-deploy-2018-02-06 (1.31.0-wmf.20)).Jan 21 2018, 10:00 PM
gerritbot added a comment.Jan 22 2018, 1:29 AM

Change 405663 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/Wikispeech@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Jan 22 2018, 1:48 AM

Change 405665 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/WikiLexicalData@master] Fix multiple PHP class declarations in one file

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

Pppery unsubscribed.Jan 22 2018, 1:49 AM
nikitavbv mentioned this in rEWIS472daaa2d7d9: Fix multiple PHP class declarations in one file.Jan 22 2018, 2:31 AM
nikitavbv mentioned this in rEWISc3bc53f76faa: Fix multiple PHP class declarations in one file.Jan 22 2018, 2:14 PM
gerritbot added a comment.Jan 22 2018, 2:27 PM

Change 405718 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/Echo@master] Fix multiple PHP class declarations in one file in tests

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

nikitavbv mentioned this in T185492: Add PHP_CodeSniffer for codestyle checks to WikiLexicalData extension.Jan 22 2018, 2:33 PM
Anomie unsubscribed.Jan 22 2018, 4:04 PM
gerritbot added a comment.Jan 23 2018, 1:26 AM

Change 405718 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Fix multiple PHP class declarations in one file in tests

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

nikitavbv added a comment.Jan 24 2018, 4:19 PM

While working on this, I have noticed that in some extensions (for example WikiLexicalData) some classes are outside includes directory (need to take this into account while grep-ing). Additionally, some files are not loaded with autoloader, but are required from other files with require_once. I am not sure what to do with those. Do we need to replace require_once with autoloading first?

gerritbot added a comment.Jan 25 2018, 1:29 AM

Change 406136 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/UploadWizard@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Jan 25 2018, 5:27 PM

Change 406136 merged by jenkins-bot:
[mediawiki/extensions/UploadWizard@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Feb 1 2018, 6:02 PM

Change 405663 merged by jenkins-bot:
[mediawiki/extensions/Wikispeech@master] Fix multiple PHP class declarations in one file

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

Stashbot mentioned this in T182861: Break out lexicon tool code to a separate branch while under development.Apr 5 2018, 1:23 PM
Stashbot subscribed.

Mentioned in SAL (#wikimedia-cloud) [2018-04-05T13:23:41Z] <Lokal_Profil> Deploy latest from Git master: f951d84, 98103e1 (T182861), 70cf552, c3bc53f (T177809), 1101d4b, b122d77, 9ae6b57, f961d2b, cf08625 (T183858), 958b2d1, c54aed9, 92e85fe, b4e54b7 (T133680)

Stashbot mentioned this in T183858: Update for SPDX License List 3.0.Apr 5 2018, 1:23 PM
Stashbot mentioned this in T133680: [Task] Functionality: Play selected text (Wikispeech).
Samwilson subscribed.Jun 7 2018, 5:18 AM
gerritbot added a comment.Jul 7 2018, 6:38 PM

Change 444371 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/BlueSpiceFoundation@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Jul 9 2018, 6:29 AM

Change 444371 merged by jenkins-bot:
[mediawiki/extensions/BlueSpiceFoundation@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Jul 14 2018, 2:45 PM

Change 445779 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/CentralNotice@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Aug 18 2018, 5:32 PM

Change 453603 had a related patch set uploaded (by Phantom42; owner: Phantom42):
[mediawiki/extensions/CirrusSearch@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Aug 23 2018, 9:48 PM

Change 445779 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@master] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Nov 17 2018, 1:30 AM

Change 474396 had a related patch set uploaded (by AndyRussG; owner: Phantom42):
[mediawiki/extensions/CentralNotice@wmf_deploy] Fix multiple PHP class declarations in one file

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

gerritbot added a comment.Nov 17 2018, 2:17 AM

Change 474396 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Fix multiple PHP class declarations in one file

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

Aklapper edited projects, added Patch-Needs-Improvement; removed Patch-For-Review.Feb 3 2022, 2:22 PM
nikitavbv removed nikitavbv as the assignee of this task.Feb 12 2022, 3:50 PM
gerritbot added a comment.Aug 14 2023, 6:35 PM

Change 453603 abandoned by Ebernhardson:

[mediawiki/extensions/CirrusSearch@master] Fix multiple PHP class declarations in one file

Reason:

from 2018, it's just too out of date to be used.

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

gerritbot added a comment.Oct 26 2023, 7:17 AM

Change 405665 abandoned by Thiemo Kreuz (WMDE):

[mediawiki/extensions/WikiLexicalData@master] Fix multiple PHP class declarations in one file

Reason:

Per T271379.

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

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits