[wp-trac] [WordPress Trac] #41362: Make locate_template() filterable to change the template locations

WordPress Trac noreply at wordpress.org
Thu Aug 10 08:08:28 UTC 2017


#41362: Make locate_template() filterable to change the template locations
-------------------------------------------------+-------------------------
 Reporter:  kylejennings83                       |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
Component:  Themes                               |  Review
 Severity:  normal                               |     Version:  4.8
 Keywords:  has-patch needs-testing 2nd-opinion  |  Resolution:
                                                 |     Focuses:  template
-------------------------------------------------+-------------------------
Changes (by dingo_bastard):

 * keywords:   => has-patch needs-testing 2nd-opinion


Comment:

 I disagree with you that it is an MVC app. It is partly that, with REST
 API being in the core, so that you can decouple front end view part, but
 it doesn't follow MVC pattern.

 Not only that, but you cannot separate it in a subfolder - bedrock tries
 to do just that, but try to install a multisite and it just won't work (I
 know because I tried for several days to make it happen).

 Making it fully MVC compliant would mean rewriting almost everything, and
 that just won't happen (backward compatibility + just the sheer amount of
 work that would need to be done in order to make it happen).

 Just to be clear, I have nothing against this patch, if it's tested and if
 it helps with the development I'm all for it ;)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/41362#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list