[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 peterwilsoncc):

 Replying to [comment:80 johnbillion]:
 > * I would rather we didn't introduce logic that runs on every request to
 handle the Hello Dolly migration.
 > * What's the purpose of the `wp_should_migrate_hello_dolly` filter?
 > * Can we get PRs instead of patches too please? So the test suite can
 run.

 Could it be handled in this [https://github.com/WordPress/wordpress-
 develop/blob/632a634ab415b4ac6a0b0cdd99ff9bbba8bbb376/src/wp-
 includes/load.php#L1028-L1037 block of code] so it's only caught if the
 file does not exist?

 Alternatively, if plugins that have been deactivated due to a missing file
 are stored in an option somewhere, the upgrade routine could check for
 that and enable the relocated plugin that way.

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