[theme-reviewers] THEME: Auto Show - 1.0.6

[Chip Bennett]

I'd say both AuthorURI *and* ThemeURI are invalid. AuthorURI is an auto
dealer site, and ThemeURI is simply a resource on that same site, with
essentially nothing but a screenshot of the Theme.

This Theme also needs to address appropriate backwards compatibility (e.g.
function_exists() calls around dynamic_sidebar(), etc.).

Function, class, option etc. names are not properly prefixed.

Theme is dealing with raw $_POST data, without sanitizing.

Theme uses unique DB entries for each option, rather than using a single
options array.

Theme is running raw DB queries, rather than using analogous core functions
(see e.g. function get_page_id() )

Theme deregisters core jquery globally, and not just for if ( ! is_admin()
).

Theme uses "home.php" for the Front Page template, rather than using
"front-page.php". (This will cause problems for users who use a static Page
as the Front Page, and display the blog posts index on a separate Page.)

...just a few observations from a quick scan of the Theme.

Chip

On Mon, Apr 11, 2011 at 10:24 AM, Daniel Tara <contact at onedesigns.com>wrote:

> Nice work Ian, but don't bother with him, he's a spammer. Trust me on this
> one.
>
> Actually because of the Author URI the theme could be subject for immediate
> rejection.
>
> Daniel
>
> --- On Mon, 4/11/11, Ian Stewart <ian at iandanielstewart.com> wrote:
>
>  From: Ian Stewart <ian at iandanielstewart.com>
>> Subject: [theme-reviewers] THEME: Auto Show - 1.0.6
>> To: theme-reviewers at lists.wordpress.org
>> Date: Monday, April 11, 2011, 6:12 PM
>>
>> I reviewed and ultimately didn't
>> approve Auto Show this morning.
>>
>> https://themes.trac.wordpress.org/ticket/3230
>>
>> Comments on the review are more than welcome. :)
>>
>> Ian
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
> _______________________________________________
> 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/20110411/5f13340e/attachment.htm>