[theme-reviewers] Cirumventing Theme Directory Preview in Ticket #3787

[Tony Jansen]

Hello all,

 

Earlier today during a review I encountered the following code in
functions.php:

/**

* The WordPress Theme Directory Preview may not be appropriate for this
theme, so redirecting to custom preview

*/

if( false !== strpos( site_url(), 'http://wp-themes.com'  )) {

                wp_redirect( 'http://demo.onedesigns.com/minimatica/' );

                exit();

}

which circumvents the regular theme preview to display something that may
have nothing to do with the theme at all or worse (not the case in this
case, mind you).

I have halted the review after conferring with greuben, who noted that it
was bad and unacceptable, and I recommended a not-approved. 

This technique may be old hat but I still like to know what you think about
it, was I right to handle it this way?

 

 <http://themes.trac.wordpress.org/ticket/3787#comment:3>
http://themes.trac.wordpress.org/ticket/3787#comment:3

 

Best Regards,

Tony Jansen

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110511/c2d00cf1/attachment.htm>