<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"><<a href="mailto:kwight@kwight.ca" target="_blank">kwight@kwight.ca</a>></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'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"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@gmail.com</a>></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"><<a href="mailto:sabreuse@gmail.com" target="_blank">sabreuse@gmail.com</a>></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 "helpers" 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'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 <<a href="mailto:sabreuse@gmail.com" target="_blank">sabreuse@gmail.com</a>> wrote:<br>
> Yeah, I suspect that "helpers.php" is an exploit that someone is using<br>
> somewhere -- exactly because it's such a generic-looking filename --<br>
> but it's also used as a name for a perfectly innocent helper function<br>
> library by other themes.<br>
><br>
><br>
> On Wed, Apr 25, 2012 at 10:32 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>> wrote:<br>
>> I replied, and "ottolook" tagged the topic. (If code is to be removed from<br>
>> SVN, Otto is the one to do it.)<br>
>><br>
>> The OP definitely found some malicious code, but some of the referenced<br>
>> Themes don't have malicious code, as far as I can tell.<br>
>><br>
>> Thanks,<br>
>><br>
>> Chip<br>
>><br>
>><br>
>> On Wed, Apr 25, 2012 at 9:12 AM, esmi at quirm dot net <<a href="mailto:esmi@quirm.net" target="_blank">esmi@quirm.net</a>><br>
>> wrote:<br>
>>><br>
>>><br>
>>><br>
>>> <<a href="http://wordpress.org/support/topic/backdoored-templates-on-themessvnwordpressorg" target="_blank">http://wordpress.org/support/topic/backdoored-templates-on-themessvnwordpressorg</a>><br>
>>><br>
>>> Mel<br>
>>> --<br>
>>> <a href="http://quirm.net" target="_blank">http://quirm.net</a><br>
>>> <a href="http://blackwidows.co.uk" target="_blank">http://blackwidows.co.uk</a><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>
>><br>
>><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>
_______________________________________________<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>