Okay, I suggested the template_redirect() hook in the ticket.<div><br></div><div>But, what we really need is what the official best-practice suggestion will be, for the purposes of the Guidelines. We need to ensure that all reviewers will be making consistent suggestions with respect to wp_enqueue_script() within functions.php.</div>
<div><br></div><div>Chip</div><div><br><div class="gmail_quote">On Wed, Sep 15, 2010 at 10:09 AM, Austin Matzko <span dir="ltr"><<a href="mailto:austin@pressedcode.com">austin@pressedcode.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Wed, Sep 15, 2010 at 9:56 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>> wrote:<br>
> Hmm... I'm completely unfamiliar with the template_redirect hook - and, I<br>
> thought enqueueing scripts was one of the uses of wp_head()? (Learning<br>
> something new every day, I guess!)<br>
> So, should I recommend hooking into template_redirect(), or wp_footer() - or<br>
> either, or both?<br>
<br>
</div>template_redirect is the last action hook before the actual template<br>
file is included and executed.<br>
<br>
You could also use the wp_enqueue_scripts action hook to attach your<br>
enqueuing callbacks; wp_enqueue_scripts is called early in wp_head.<br>
However, I prefer template_redirect because it's late enough that WP<br>
has parsed the request (so is_singular() and the like are available)<br>
but it's prior to the actual template file's being included, so if the<br>
theme queries posts on its own prior to that point I don't have to<br>
worry about resetting $wp_query.<br>
<br>
Whatever works for you, but default wp_head without a high priority is too late.<br>
<div><div></div><div class="h5">_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</div></div></blockquote></div><br></div>