[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