[wp-trac] [WordPress Trac] #53323: Place Hello Dolly in containing folder

[WordPress Trac]

#53323: Place Hello Dolly in containing folder
-------------------------------------------------+-------------------------
 Reporter:  afragen                              |       Owner:  whyisjake
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  6.9
Component:  Upgrade/Install                      |     Version:  5.8
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-unit-tests reporter-   |     Focuses:
  feedback                                       |
-------------------------------------------------+-------------------------

Comment (by Otto42):

 We don't need to change core code for this. It can be handled entirely by
 the hello.php file that is already there.

 Basically, just include the Hello Dolly plugin as an external, much like
 akismet already is. That way, the new files are there after an update.

 Then, replace the contents of the hello PHP file with a small bit of code
 that does three things. First it checks if the Hello Dolly plugin is
 there. Second, it rewrites the active plugins option to stop including
 itself and to include the new hello plugin. Then finally, it simply
 includes the Hello Dolly plugin directly. This is basically a one-stop
 shop to the plugin deactivating itself, making itself irrelevant and
 changing it to including new plugins as part of the normal plugin load
 system.

 Nothing required in core. Nothing needed in updates logic, it's simply a
 plugin updating its location and then not being loaded anymore. Seems like
 a very simple process.

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