[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