[wp-trac] [WordPress Trac] #37646: Make wp-settings.php a series of do_actions()

[WordPress Trac]

#37646: Make wp-settings.php a series of do_actions()
--------------------------------------+------------------------------
 Reporter:  johnjamesjacoby           |       Owner:
     Type:  enhancement               |      Status:  new
 Priority:  normal                    |   Milestone:  Awaiting Review
Component:  Bootstrap/Load            |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  2nd-opinion dev-feedback  |     Focuses:
--------------------------------------+------------------------------
Changes (by jorbin):

 * keywords:  needs-patch 2nd-opinion => 2nd-opinion dev-feedback


Comment:

 Discussed this a bit in the linked bug scrub.  Some thoughts:

 This is both an exciting and scary proposal.

 My biggest worry with tying all includes to actions is that it would make
 all of WordPress pluggable and that feels like a great way to increase
 breakage on updates.  Image a plugin that decides to include it’s own
 version of `src/wp-includes/category-template.php`. Core could never add a
 new function to that file again since it can’t count on that file
 existing. The Autoloader being worked on in #36335 will make it so that we
 don't need to worry about this.  This seems like a dependency before I
 would want to include more actions in the bootstrap process.

 We need to consider a lot of "what if" situations with this change, let's
 try to brainstorm those out here while we continue to explore this issue.

 I'm removing `needs-patch` because this needs more discussion before we
 write code.

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