<div dir="ltr"><div>Thanks Cais for the clarification.<br><br>To be honest, this is something bother me for a while. I believe esc_url is not required for home_url(). However, many reviewers make it required issue. <br><br>
</div></div><div class="gmail_extra"><br clear="all"><div>Zhijian (Stephen) Cui<br>Cell: (609)977-5089<br>eMail: <a href="mailto:scui2005@gmail.com">scui2005@gmail.com</a></div>
<br><br><div class="gmail_quote">On Wed, Jun 19, 2013 at 2:13 PM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><div><div>I don't consider that a manipulation of the $path parameter; but, with something like the following, I would say yes ...<br></div><br></div><div>home_url( theme_new_path() );<br></div>function theme_new_path() {<br>


</div><div>  $path = '/';<br></div><div></div>  return apply_filters( `theme_new_path_filter`, $path );<br><div>}<br><br></div><div>Something like this would be a much better candidate to be escaped although I really am not seeing this in a use case scenario just some random (pseudo) code to clarify what I see as manipulating the $path parameter of the function.<span class="HOEnZb"><font color="#888888"><br>


</font></span></div></div><div class="gmail_extra"><span class="HOEnZb"><font color="#888888"><br clear="all"><div>Edward Caissie<br>aka Cais.</div></font></span><div><div class="h5">
<br><br><div class="gmail_quote">On Wed, Jun 19, 2013 at 2:55 PM, Stephen Cui <span dir="ltr"><<a href="mailto:scui2005@gmail.com" target="_blank">scui2005@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


How about home_url( '/' ) ?<div><div><br clear="all"><div><br></div>
<br><div class="gmail_quote">On Wed, Jun 19, 2013 at 1:52 PM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div dir="ltr">If the theme is manipulating the $path parameter of the function, or attaching something to the `home_url` hook then I would say yes, otherwise `home_url()` on its own should be fine as by default $path is empty.<br>





</div><div class="gmail_extra"><br clear="all"><div>Edward Caissie<br>aka Cais.</div>
<br><br><div class="gmail_quote"><div><div>On Wed, Jun 19, 2013 at 2:39 PM, Stephen Cui <span dir="ltr"><<a href="mailto:scui2005@gmail.com" target="_blank">scui2005@gmail.com</a>></span> wrote:<br></div>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>

According to in example in <a href="http://codex.wordpress.org/Theme_Review#Site_Information" target="_blank">http://codex.wordpress.org/Theme_Review#Site_Information</a>, echo home_url() is acceptable (or not?)<br><br>But many reviewer ask the author to use echo esc_url( home_url() ).<br>






<br>Question: Is esc_url required in home_url() case?<br><br>Regards<span><font color="#888888"><br><br>Stephen<br><br>
</font></span><br></div></div>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
<br></blockquote></div><br></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
<br></blockquote></div><br>
</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
<br></blockquote></div><br></div></div></div>
<br>_______________________________________________<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>
<br></blockquote></div><br></div>