Okay, so at least I was on the right track!<div><br></div><div>So, is this simply a &quot;remove this function&quot; issue, or is it elevated to a &quot;explain yourself&quot; matter?</div><div><br></div><div>Chip<br><br><div class="gmail_quote">
On Mon, Sep 27, 2010 at 12:00 PM, Doug Stewart <span dir="ltr">&lt;<a href="mailto:zamoose@gmail.com">zamoose@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On Mon, Sep 27, 2010 at 12:55 PM, Chip Bennett &lt;<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>&gt; wrote:<br>
&gt; Checking the diff for an update to a Theme that I had previously approved, I<br>
&gt; found this:<br>
&gt;<br>
&gt; &lt;?php  ( header(&#39;Location: <a href="http://allfreecms.com" target="_blank">http://allfreecms.com</a>&#39;) );?&gt;<br>
&gt;<br>
&gt; Any good reason for this to exist? My limited PHP knowledge tells me that<br>
&gt; it&#39;s some sort of header redirect. Does it have any business being called in<br>
&gt; a WordPress Theme? (Note: I can&#39;t actually find anywhere that the Theme is<br>
&gt; actually including the functions/index.php file in which the above code is<br>
&gt; found..)<br>
&gt; Chip<br>
&gt;<br>
<br>
</div></div>It is indeed a redirect. If it&#39;s called as the first element in a<br>
page, it will redirect. If it&#39;s called further down, you&#39;re likely to<br>
get the &quot;Headers already sent...&quot; PHP error.<br>
<br>
In any event, there&#39;s precious little reason that I can think of to<br>
include it in a theme.<br>
<font color="#888888"><br>
<br>
--<br>
-Doug<br>
@zamoose<br>
<a href="http://literalbarrage.org/blog/" target="_blank">http://literalbarrage.org/blog/</a><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></blockquote></div><br></div>