<div dir="ltr">Ola,<div><br></div><div>Redirecting the search results page is definitely weird :( i wouldn't approve the code which has any suspicious behaviour.</div><div>Thanks</div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Feb 24, 2014 at 12:15 PM, Daniel <span dir="ltr"><<a href="mailto:danielx386@gmail.com" target="_blank">danielx386@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The redirect search is a no no IMO as that really a plug-in thing.<br>
<br>
On 2/24/14, Ola Łączek <<a href="mailto:ola@bodera.com">ola@bodera.com</a>> wrote:<br>
> Hello!<br>
><br>
> I'm reviewing a theme, which has a couple of "clean up" functions, and I'd<br>
> like to know if those are allowed:<br>
> - Clean up output of stylesheet <link> tags<br>
> - Redirects search results from /?s=query to /search/query/, converts %20<br>
> to +<br>
> - Remove unnecessary self-closing tags on get_avatar(),<br>
> post_thumbnail_html()<br>
> - Fix for empty search queries redirecting to home page<br>
> - Don't return the default description in the RSS feed if it hasn't been<br>
> changed<br>
><br>
> Best regards,<br>
> Ola Laczek<br>
><br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Regards,<br>
Daniel Fenn<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>
</font></span></blockquote></div><br></div></div>