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.<div><br></div><div>This Theme also needs to address appropriate backwards compatibility (e.g. function_exists() calls around dynamic_sidebar(), etc.).</div>
<div><br></div><div>Function, class, option etc. names are not properly prefixed.</div><div><br></div><div>Theme is dealing with raw $_POST data, without sanitizing.</div><div><br></div><div>Theme uses unique DB entries for each option, rather than using a single options array.</div>
<div><br></div><div>Theme is running raw DB queries, rather than using analogous core functions (see e.g. function get_page_id() )</div><div><br></div><div>Theme deregisters core jquery globally, and not just for if ( ! is_admin() ).</div>
<div><br></div><div>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.)</div>
<div><br></div><div>...just a few observations from a quick scan of the Theme.</div><div><br></div><div>Chip<br><br><div class="gmail_quote">On Mon, Apr 11, 2011 at 10:24 AM, Daniel Tara <span dir="ltr"><<a href="mailto:contact@onedesigns.com">contact@onedesigns.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Nice work Ian, but don't bother with him, he's a spammer. Trust me on this one.<br>
<br>
Actually because of the Author URI the theme could be subject for immediate rejection.<br>
<br>
Daniel<br>
<br>
--- On Mon, 4/11/11, Ian Stewart <<a href="mailto:ian@iandanielstewart.com" target="_blank">ian@iandanielstewart.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
From: Ian Stewart <<a href="mailto:ian@iandanielstewart.com" target="_blank">ian@iandanielstewart.com</a>><br>
Subject: [theme-reviewers] THEME: Auto Show - 1.0.6<br>
To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
Date: Monday, April 11, 2011, 6:12 PM<div class="im"><br>
I reviewed and ultimately didn't<br>
approve Auto Show this morning.<br>
<br>
<a href="https://themes.trac.wordpress.org/ticket/3230" target="_blank">https://themes.trac.wordpress.org/ticket/3230</a><br>
<br>
Comments on the review are more than welcome. :)<br>
<br>
Ian<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
<br>
</div></blockquote><div><div></div><div class="h5">
<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
</div></div></blockquote></div><br></div>