Closed (fixed)
Project:
SpamSpan filter
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
31 Jan 2012 at 09:10 UTC
Updated:
3 Jan 2017 at 14:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
escoles commentedSeeing this also. As far as I can tell, this happens only if there is BOTH a subject and body argument -- omitting either will allow the code to work.
So you can have subject OR body, but not both.
Comment #2
vali hutchison commentedI've just tried the solution in http://drupal.org/node/1040172#comment-5282018 and it works. One thing I noticed I had to do to see the changes was to clear the caches after modifying the spamspan.module file.
So to clarify - i changed line 95 of spamspan.module to:
"(?:\?[A-Za-z0-9_= %\.\-\~\_\&|(?:\&)]*)?)" . # an optional ? followedI also noticed that if you have an exclamation mark in the subject then this throws a similar error. So to avoid this use:
"(?:\?[A-Za-z0-9_= %\.\-\~\_\&|(?:\&:\!)]*)?)" . # an optional ? followedComment #3
vitalie commentedThanks @Desert Dog. I have put the changes into a patch. (I've just added ; and ! to regexp since 'amp' is already being matched, however had to take care about splitting the headers correctly).
Comment #6
vitalie commentedComment #7
jennypanighetti commentedI need this fix, but I don't see a February 2015 7.x-1.x-dev release available!! I only see 6.x and 8.x in the dev box, and the previous 7.x dev was in November 2014.
Comment #8
vitalie commentedSorry @jenstechs, it should be available now.
Comment #9
nerdoc commentedWe have a year later now, how are chances to get this into D7 as stable release?
Suffering from that bug a over year now...
Is there a new release planned in near future?
Thanks.
Comment #10
vitalie commented@nerdoc, thx. There's a new release now.