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

 What were all those `.gitignore` changes? I don't remember adding them.

 Replying to [comment:78 whyisjake]:
 > After chatting with @peterwilsoncc and @afragen, I think we need to be a
 little more explicit in the fallback handling here. In
 [attachment:53323.2.diff 53323.diff] I added a filter to the
 `wp_get_active_and_valid_plugins()` function to ensure that we get the
 correct version if active.

 I have mixed feelings with this update specially because its a single-use
 hook that is going to added to the hook stack forever.

 Also, @peterwilsoncc can you review the tests? I feel that some setups or
 tear-downs could be useful here, what do you think?

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