[theme-reviewers] Header Redirect: Any Good Reason For This?
chip at chipbennett.net
Mon Sep 27 17:05:23 UTC 2010
Okay, so at least I was on the right track!
So, is this simply a "remove this function" issue, or is it elevated to a
"explain yourself" matter?
On Mon, Sep 27, 2010 at 12:00 PM, Doug Stewart <zamoose at gmail.com> wrote:
> On Mon, Sep 27, 2010 at 12:55 PM, Chip Bennett <chip at chipbennett.net>
> > Checking the diff for an update to a Theme that I had previously
> approved, I
> > found this:
> > <?php ( header('Location: http://allfreecms.com') );?>
> > Any good reason for this to exist? My limited PHP knowledge tells me that
> > it's some sort of header redirect. Does it have any business being called
> > a WordPress Theme? (Note: I can't actually find anywhere that the Theme
> > actually including the functions/index.php file in which the above code
> > found..)
> > Chip
> It is indeed a redirect. If it's called as the first element in a
> page, it will redirect. If it's called further down, you're likely to
> get the "Headers already sent..." PHP error.
> In any event, there's precious little reason that I can think of to
> include it in a theme.
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the theme-reviewers