[wp-trac] [WordPress Trac] #14348: If it's a HEAD request, stop after the head!

WordPress Trac wp-trac at lists.automattic.com
Thu Aug 16 07:12:48 UTC 2012


#14348: If it's a HEAD request, stop after the head!
---------------------------------------------+------------------
 Reporter:  mitchoyoshitaka                  |       Owner:
     Type:  enhancement                      |      Status:  new
 Priority:  normal                           |   Milestone:  3.5
Component:  Performance                      |     Version:  3.0
 Severity:  normal                           |  Resolution:
 Keywords:  has-patch 3.5-early 2nd-opinion  |
---------------------------------------------+------------------

Comment (by mbijon):

 I just tested attachment:14348-7.diff with Fiddler & the test suite. Both
 ways it worked as expected.

 The {{{do_action()}}} is in the correct place, it's executed at the same
 point as in my previous attachment:14348-6.diff. The distribution of code
 makes this a bit spaghetti'ish but shouldn't have any negative effects.

 Putting spaghetti aside, the action does seem like it's unnecessary. Such
 an abstraction is a positive thing for code that might be run in multiple
 places, but I can't think of any reasons to call
 {{{handle_head_request()}}} from a plugin that wouldn't already be
 addressed by the {{{exit_on_http_head}}} filter & Kurt's added
 {{{wp_die_handler}}}. Anyone else?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/14348#comment:41>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list