#60414: Core PHP autoloader proposal
-------------------------------------+-------------------------------------
Reporter: aristath | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Future Release
Component: General | Version:
Severity: normal | Resolution:
Keywords: has-patch has-unit- | Focuses: performance,
tests early | sustainability
-------------------------------------+-------------------------------------
Comment (by aristath):
A quick update of what happens in this ticket and a recap of the concerns
that were brought up in the proposal - along with what was done to address
them:
* Composer vs non-composer: It's easier for us to start with the basic
classmap implementation that exists in the PR, and then discuss separately
if we want to automate the classmap generation using Composer.
* Added an extra check in the health-check screen, to check if any Core
classes are being overridden, and show a critical warning if something
like that occurs
* PHPUnit tests were added and all existing automated tests pass without
issues.
* Performance impact: Adding an autoloader seems to reduce the memory use,
but - at this stage - has no other significantly measurable impact.
Any original concerns that were mentioned in the original ticket (#36335)
have already been added to this
[https://core.trac.wordpress.org/ticket/60414 ticket's description], along
with what was done to address them
The PR is complete and it works. There are no backward-compatibility
issues, and I constantly update it so it's up to date with `trunk`.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/60414#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform