Ticket #4639 (closed defect (bug): fixed)

Opened 5 years ago

Last modified 5 years ago

action admin_print_scripts- isnt fired for plugins which reside in a subdir.

Reported by: DD32 Owned by: anonymous
Priority: normal Milestone: 2.3
Component: Administration Version: 2.3
Severity: normal Keywords: admin_print_scripts get_plugin_page_hook has-patch
Cc:

Description

admin-header.php:

if ( isset($page_hook) )
	do_action('admin_print_scripts-' . $page_hook);
else if ( isset($plugin_page) )
	do_action('admin_print_scripts-' . $plugin_page);

For plugins which reside in a subdirectory (ie. /wp-content/plugins/plugin_name/plugin.php), the admin_print_scripts- action doesnt fire, The problem is that $page_hook is a 0 character string, however IS set.

Theres 2 options i can see, correcting the function which assigns to $page_hook, or testing for isset() && ! empty()

Attached is a patch to modify get_plugin_page_hook() to return null insead of , this causes $page_hook to "not be set" which in turn, causes the print scripts hook to correctly run.

Attachments

4639.get_plugin_page_hook.patch Download (466 bytes) - added by DD32 5 years ago.
get_plugin_page_hook fix
4639.get_plugin_page_hook.diff Download (466 bytes) - added by DD32 5 years ago.
get_plugin_page_hook fix as diff instead

Change History

DD325 years ago

get_plugin_page_hook fix

DD325 years ago

get_plugin_page_hook fix as diff instead

comment:1   DD325 years ago

  • Keywords admin_print_scripts get_plugin_page_hook added

Apologies for the 2 different format patches/diffs, the HTML viewer didnt work for the .patch file, so i thought it only accepted .diff, but that isnt working either.

  • Keywords has-patch added
  • Milestone set to 2.3 (trunk)

comment:3   ryan5 years ago

  • Status changed from new to closed
  • Resolution set to fixed

(In [5803]) Return hook as null instead of empty string. Props DD32. fixes #4639

Note: See TracTickets for help on using tickets.