I find that we wanted to be notified about reported spam more often than once a day. The code has a hard-set number for the interval at 86,400 seconds, or one day, so that the report will only be generated if it's been 86,400 seconds since the last report.

The attached patch will change that into a configurable option in the spam module's options page.

CommentFileSizeAuthor
#3 spam-reportinterval_0.patch1.65 KBnschelly
spam-reportinterval.patch1.59 KBnschelly

Comments

jeremy’s picture

jeremy commented
Status: Reviewed & tested by the community » Needs work

When you submit patches, make them "code needs review", not "ready to be committed" unless you are one of the module maintainers.

Please resync your patch, the administrative pages were completely restructured tonight.

Thanks.

nschelly’s picture

nschelly commented

Sorry about that - I picked the status by accident and couldn't change it. I'll re-patch the new code and send it up again soon.
-N

nschelly’s picture

nschelly commented
Status: Needs work » Needs review
StatusFileSize
new spam-reportinterval_0.patch1.65 KB

Here's a new patch against the latest version... I like the new admin pages by the way.
-N

jeremy’s picture

jeremy commented
Version: 5.x-1.0 » 5.x-3.x-dev
Status: Needs review » Needs work

Sorry, new features will only be added in the upcoming 5.x-3.x version of the module.

nschelly’s picture

nschelly commented

I'm having trouble following here...
-N

Do you not like the idea and not want to incorporate it?
or
Did I patch it against the wrong version?
or
Did my patch not work as expected?

jeremy’s picture

jeremy commented
Status: Needs work » Closed (duplicate)

Added to issue where spam reporting is being added to 5.x-3.x version of module. Marking this issue as now duplicate.