Strangely, I didn't get the error in Theme-Check, but only in the uploader script.<div><br></div><div>I'll update Theme Check to trunk, and report back.</div><div><br></div><div>Chip<br><br><div class="gmail_quote">
On Wed, Feb 23, 2011 at 11:17 AM, Simon Prosser <span dir="ltr"><<a href="mailto:pross@pross.org.uk">pross@pross.org.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On 02/23/11 16:56, Chip Bennett wrote:<br>
> I noticed a false-positive in the Theme Uploader script:<br>
><br>
> Results of Automated Theme Scanning: Fail<br>
><br>
> - WARNING: Found *eval* in the PHP files. This is not allowed.<br>
><br>
><br>
> I don't use eval() anywhere in Oenology. I do, however, use the term<br>
> "evaluate"/"evaluated" in PHP comments. I could find no other examples of<br>
> "eval" appearing anywhere in any of the Theme template files.<br>
</div></div>Looks like the eval() regex needs updating, Chip can you update<br>
themecheck to latest trunk and test your theme again? ive updated the regex.<br>
FYI it is now /(?<![_|a-z0-9])eval\s?\(/<br>
<font color="#888888"><br>
--<br>
My Blog: <a href="http://www.pross.org.uk/" target="_blank">http://www.pross.org.uk/</a><br>
Plugins: <a href="http://www.pross.org.uk/plugins/" target="_blank">http://www.pross.org.uk/plugins/</a><br>
Themes : <a href="http://wordpress.org/extend/themes/profile/pross" target="_blank">http://wordpress.org/extend/themes/profile/pross</a><br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">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>
</font></blockquote></div><br></div>