<div class="gmail_extra">Hold that thought. There may be some...internal discussions being conducted along those lines. :)</div><div class="gmail_extra"><br></div><div class="gmail_extra">Chip<br><br><div class="gmail_quote">
On Wed, Apr 25, 2012 at 11:05 AM, Kirk Wight <span dir="ltr">&lt;<a href="mailto:kwight@kwight.ca" target="_blank">kwight@kwight.ca</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_extra">That&#39;s great! I agree with Cais, it really shows the value of these reviews.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Has there ever been a proposed pruning, similar to what happened with plugins last year (assuming we even have the mandate to do that)? Maybe setting a two year cutoff, and reviewing older stuff to see if some themes should actively be removed? It would seem a simple way to up the overall quality of the repo (although we would need to give proper warning, allowing affected developers time to update their themes if they so choose).</div>


<div class="gmail_extra"><br></div><div class="gmail_extra">Thoughts?<div><div class="h5"><br><br><div class="gmail_quote">On 25 April 2012 11:56, Edward Caissie <span dir="ltr">&lt;<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Good Work, Amy! ... and definitely yet another justification for the process and guidelines we use.<br><br><br clear="all">


Cais.<div><div><br>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 25, 2012 at 11:50 AM, Amy Hendrix <span dir="ltr">&lt;<a href="mailto:sabreuse@gmail.com" target="_blank">sabreuse@gmail.com</a>&gt;</span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I had a bit of time so I went through the list he posted --<br>
<br>
- Only one theme is available in the directory, and that was a false<br>
positive (same filename, but a completely different &quot;helpers&quot; file<br>
from the others on the list).<br>
- Two were themes that reviewers caught, rejected, and reported to<br>
<a href="http://wp.org" target="_blank">wp.org</a> at the time they were reviewed,<br>
- The rest were from before there was a review process, and none of<br>
them would pass review now.<br>
<br>
I think that&#39;s a nice indication that our process has made a big<br>
difference in the quality of what gets out there!<br>
<div><div><br>
On Wed, Apr 25, 2012 at 10:35 AM, Amy Hendrix &lt;<a href="mailto:sabreuse@gmail.com" target="_blank">sabreuse@gmail.com</a>&gt; wrote:<br>
&gt; Yeah, I suspect that &quot;helpers.php&quot; is an exploit that someone is using<br>
&gt; somewhere -- exactly because it&#39;s such a generic-looking filename --<br>
&gt; but it&#39;s also used as a name for a perfectly innocent helper function<br>
&gt; library by other themes.<br>
&gt;<br>
&gt;<br>
&gt; On Wed, Apr 25, 2012 at 10:32 AM, Chip Bennett &lt;<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>&gt; wrote:<br>
&gt;&gt; I replied, and &quot;ottolook&quot; tagged the topic. (If code is to be removed from<br>
&gt;&gt; SVN, Otto is the one to do it.)<br>
&gt;&gt;<br>
&gt;&gt; The OP definitely found some malicious code, but some of the referenced<br>
&gt;&gt; Themes don&#39;t have malicious code, as far as I can tell.<br>
&gt;&gt;<br>
&gt;&gt; Thanks,<br>
&gt;&gt;<br>
&gt;&gt; Chip<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Wed, Apr 25, 2012 at 9:12 AM, esmi at quirm dot net &lt;<a href="mailto:esmi@quirm.net" target="_blank">esmi@quirm.net</a>&gt;<br>
&gt;&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; &lt;<a href="http://wordpress.org/support/topic/backdoored-templates-on-themessvnwordpressorg" target="_blank">http://wordpress.org/support/topic/backdoored-templates-on-themessvnwordpressorg</a>&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Mel<br>
&gt;&gt;&gt; --<br>
&gt;&gt;&gt; <a href="http://quirm.net" target="_blank">http://quirm.net</a><br>
&gt;&gt;&gt; <a href="http://blackwidows.co.uk" target="_blank">http://blackwidows.co.uk</a><br>
&gt;&gt;&gt; _______________________________________________<br>
&gt;&gt;&gt; theme-reviewers mailing list<br>
&gt;&gt;&gt; <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
&gt;&gt;&gt; <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; theme-reviewers mailing list<br>
&gt;&gt; <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
&gt;&gt; <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
&gt;&gt;<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>
</div></div><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></blockquote></div><br></div></div></div>
<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>
<br></blockquote></div><br></div>