Skip to:
Content

bbPress.org

Opened 10 years ago

Closed 10 years ago

#2858 closed task (blessed) (fixed)

Send post titles to Akismet

Reported by: cfinke's profile cfinke Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6 Priority: high
Severity: normal Version:
Component: Extend - Akismet Keywords: has-patch commit
Cc:

Description

Lots of forum spammers are now putting the spammiest part of their posts into the titles, but bbPress doesn't include the post title when checking the first post in a new thread. Fixing that will improve Akismet's performance for this type of spam.

Patch forthcoming.

Attachments (2)

2858.1.patch​ (802 bytes) - added by cfinke 10 years ago.
Includes the post title for the first post in a thread. Won't affect subsequent replies.
2858.2.patch​ (1.6 KB) - added by cfinke 10 years ago.
Supersedes the previous patch. This patch also includes the post title when submitting a post as ham or spam.

Download all attachments as: .zip

Change History (4)

@cfinke
10 years ago

Includes the post title for the first post in a thread. Won't affect subsequent replies.

@cfinke
10 years ago

Supersedes the previous patch. This patch also includes the post title when submitting a post as ham or spam.

#1 @johnjamesjacoby
10 years ago

  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 2.6
  • Priority changed from normal to high
  • Type changed from enhancement to task

Thanks for the patch; I'll make sure it goes into trunk and the 2.5 branch today.

#2 @johnjamesjacoby
10 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 5933:

Akismet: include title with content. Props cfinke. Fixes #2858.

Note: See TracTickets for help on using tickets.