Yes I had another checkin for that, but as that affected everything I 
wanted to keep it separate.    From what I see modifying that query is 
dangerous as it affects all header checks across the entire product, so 
I felt it safer to wait on that, sorry if you would have preferred both 
at the same time.

The changed line looks like

From
if (!isset($msg['Subject']) || !preg_match('/(.+yEnc)(\.\s*|\s*by 
xMas\s*|_|\s*--\s*READ NFO!\s*|\s*| \[S\d+E\d+\] )\((\d+)\/(\d+)\)/', 
$msg['Subject'], $matches)) {
To
if (!isset($msg['Subject']) || !preg_match('/(.+yEnc)(\.\s*|\s*by 
xMas\s*|_|\s*--\s*READ NFO!\s*|\s*| \[S\d+E\d+\] |*/\"\S+\" 
/*)\((\d+)\/(\d+)\)/', $msg['Subject'], $matches)) {

So I added an extra or clause \"\S+\" , which allows for the matches and 
does not break anything, /as far as I could tell/, but again it seemed 
like something for a separate checkin as it likely needs further review, 
to ensure it is not detrimental, though I could not see or find any 
place where it was for my testing.

If you would like I'll add the binaries.php regex as well, however I 
think that should be reviewed further (by someone better at regex than 
my self), I have the change made on my install without issues for the 
last 20 hours but, I don't want to break anything in binaries.php, so 
I'd personally perfer to add it as a separate pull request, if you are 
ok with that.

As for the 250, if I understand that correctly if it has more than 250 
parts it skips creation of a nzb, but as this group is images and there 
are very often more than 250 files that number needs to be higher say 
500 to ensure all nzbs are created and not dropped due to the number of 
parts.


ERIC

On 01/23/2014 08:34 AM, jonnyboy wrote:
>
> In www/lib/ReleaseCleaning.php:
>
> > @@ -1563,6 +1563,17 @@ public function releaseCleaner($subject, $groupName)
> >  			else
> >  				return array("cleansubject" => $this->releaseCleanerHelper($subject), "properlynamed" => false);
> >  		}
> > +		else if ($groupName === "alt.binaries.pictures.erotica.anime") {
> > +			// [ABPEA - Original] Arima Zin - Tennen Koiiro Alcohol [BB, Boy] - 005 of 229 - yEnc "Tennen_Koiiro_Alcohol-005.jpg"
>
> I don't think this subject would be inserted into collections table. 
> It does not pass this regex.
>
> |/(.+yEnc)(\.\s*|\s*by xMas\s*|_|\s*--\s*READ NFO!\s*|\s*| \[S\d+E\d+\] )\((\d+)\/(\d+)\)/
> |
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/nZEDb/nZEDb/pull/1139/files#r9112231>.
>