[wp-trac] [WordPress Trac] #54795: Bootstrap: Move navigation post type hooks to `admin-filters.php`

WordPress Trac noreply at wordpress.org
Wed Feb 22 23:08:35 UTC 2023


#54795: Bootstrap: Move navigation post type hooks to `admin-filters.php`
--------------------------------------------------+-----------------------
 Reporter:  dlh                                   |       Owner:  audrasjb
     Type:  defect (bug)                          |      Status:  closed
 Priority:  normal                                |   Milestone:  6.0
Component:  Bootstrap/Load                        |     Version:
 Severity:  normal                                |  Resolution:  fixed
 Keywords:  has-patch commit assigned-for-commit  |     Focuses:
--------------------------------------------------+-----------------------

Comment (by johnbillion):

 I think we need to be more careful with these changes in the future.
 [53304] looks fine on the surface but actually subtly changes the order in
 which actions are added. I just had to spend some time debugging a plugin
 of mine which broke in 6.0 (the breakage had gone unnoticed) because its
 `admin_print_footer_scripts` callback had switched from running after
 `_wp_footer_scripts()` to running before it. This is because `admin-
 filters.php` is called much later than `default-filters.php`.

 Not a huge problem in my particular case, but let's bear it in mind for
 any similar changes in the future.

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


More information about the wp-trac mailing list