[wp-trac] [WordPress Trac] #36335: Next generation: core autoloader proposal
WordPress Trac
noreply at wordpress.org
Mon Sep 19 17:15:59 UTC 2016
#36335: Next generation: core autoloader proposal
-----------------------------+------------------
Reporter: dnaber-de | Owner:
Type: feature request | Status: new
Priority: normal | Milestone: 4.7
Component: General | Version:
Severity: normal | Resolution:
Keywords: has-patch | Focuses:
-----------------------------+------------------
Comment (by TJNowell):
I'm not a fan of moving the files at this stage, that can be done once
their location is irrelevant and it might break existing code that
includes certain files directly
Generating the classmap should also be a build step, adding constants for
runtime seems counterproductive.
I'm also concerned that the generation code relies on global variables and
isn't very testable, it wouldn't be possible to write an easy unit test.
I'm also concerned by the new top level files, we should keep the root
clean and adding new top level files is going to garner some flack, I
would move those to wp-includes
Regarding performance, even crude benchmarks would be helpful. Without
optimisations an autoloader would make WP slower due to the larger number
of files, so we need to be clear about what performance improvements are
possible and which ones have been made, as well as some preliminary
benchmarks so that they can be replicated and expanded upon.
Thanks for taking the time Mike, I appreciate the effort and it's good to
see progress
--
Ticket URL: <https://core.trac.wordpress.org/ticket/36335#comment:202>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list