[wp-trac] [WordPress Trac] #36335: Next generation: core autoloader proposal
WordPress Trac
noreply at wordpress.org
Thu Aug 25 06:07:55 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 schlessera):
Ah, nice, @wonderboymusic , you made the effort to create a patch with the
code in my article. I never bothered because the feedback was not very
conclusive, but if there's indeed some interest to really evaluate this
approach, I'm all in!
I see you've already included all of the "easy-wins" in one patch already.
It probably makes sense to have one big patch with all the easily
converted classes in there, and then create additional issues for anything
that needs more complex changes.
Also, within the classes you're autoloading, there are some functions to
fetch a shared instance. You've moved these around, but I think the longer
term plan should be to deal with these with something along the lines of
#37699 (awesome work on that one as well, btw). Ideally, the
instantiation, and the decision whether to share the same instance or
create a new one for each request, should be centralized in some form and
taken out of the inner layers.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/36335#comment:60>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list