Skip to:
Content

bbPress.org

Opened 10 years ago

Closed 9 years ago

#2591 closed enhancement (fixed)

Send extra data to Akismet to better detect forum spam

Reported by: eoigal's profile eoigal Owned by: netweb's profile netweb
Milestone: 2.6 Priority: normal
Severity: normal Version: trunk
Component: Extend - Akismet Keywords: has-patch commit
Cc:

Description

Updated plugin to send Akismet the following extra parameters that will help us identify suspicious new posts on a forum

comment_total - total comments submitted by the comment author to date
comment_last_active_gmt - datetime stamp of last post submitted on this post/topic
comment_account_registered_gmt - comment author's account registration datetime stamp

Attachments (4)

patch.diff (3.4 KB) - added by eoigal 10 years ago.
2591.2.diff (3.4 KB) - added by netweb 10 years ago.
2591.3.diff (3.8 KB) - added by netweb 10 years ago.
2591.4.diff (3.8 KB) - added by netweb 9 years ago.
Refreshed against /trunk

Download all attachments as: .zip

Change History (10)

@eoigal
10 years ago

@netweb
10 years ago

#1 follow-up: @netweb
10 years ago

  • Milestone changed from Awaiting Review to 2.6

Nice, thanks for the patch :)

I tweaked the patch to use bbp_get_user_post_count rather than bbp_get_user_reply_count_raw.

This way the users total post count includes topics and replies and not just replies.

This is under the assumption that Akismet will add some weight to the spam/ham calculation based on the count submitted thus including the topic counts along with reply counts would be more beneficial?

#2 in reply to: ↑ 1 @eoigal
10 years ago

Replying to netweb:

This is under the assumption that Akismet will add some weight to the spam/ham calculation based on the count submitted thus including the topic counts along with reply counts would be more beneficial?

Sure, bbp_get_user_post_count will be more useful, thanks.

This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.


10 years ago

#4 @netweb
10 years ago

  • Owner set to netweb

#5 @netweb
10 years ago

  • Keywords commit added

@netweb
10 years ago

@netweb
9 years ago

Refreshed against /trunk

#6 @netweb
9 years ago

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

In 5654:

Akismet: Pass additional user and post information to Aksimet helping identify suspicious new posts

  • comment_total: Total user posts via bbp_get_user_post_count()
  • comment_last_active_gmt: Topic last active time via bbp_get_topic_last_reply_id()
  • comment_account_registered_gmt: User registered via $userdata->user_registered

Props eoigal. Fixes #2591

Note: See TracTickets for help on using tickets.