[theme-reviewers] Header Redirect: Any Good Reason For This?

[Chip Bennett]

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?

Chip

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>
> wrote:
> > 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
> in
> > a WordPress Theme? (Note: I can't actually find anywhere that the Theme
> is
> > actually including the functions/index.php file in which the above code
> is
> > 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.
>
>
> --
> -Doug
> @zamoose
> http://literalbarrage.org/blog/
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20100927/ac02d9d8/attachment.htm>