Skip to:
Content

bbPress.org

Opened 14 years ago

Closed 14 years ago

#1233 closed enhancement (fixed)

Topic subscription and Akismet/spam?

Reported by: chrishajer's profile chrishajer Owned by:
Milestone: 1.1 Priority: normal
Severity: normal Version: 1.0.2
Component: Back-end Keywords: has-patch akismet
Cc: nightgunner5@…

Description

I am subscribed to this topic:
http://bbpress.org/forums/topic/wp-integration-coockie-does-not-work

I just received this email notification (see text attachment).

I am wondering if the subscriptions are sent out before or after Akismet catches it, or if this one happened to be manually moderated at bbPress.org, so when the notification was sent out it was NOT spam.

I would like to ensure that notifications of spam are not sent out to subscribers.

Attachments (2)

notification-email.txt (2.8 KB) - added by chrishajer 14 years ago.
copy of email notification I received
1233.patch (1.8 KB) - added by Nightgunner5 14 years ago.
Suggested fix - Untested, but should work.

Download all attachments as: .zip

Change History (7)

@chrishajer
14 years ago

copy of email notification I received

#1 @chrishajer
14 years ago

Linking relevant reports from the bbpress.org/forums

http://bbpress.org/forums/topic/try-out-email-subscriptions

@Nightgunner5
14 years ago

Suggested fix - Untested, but should work.

#2 follow-up: @matt
14 years ago

Your patch includes other changes that seem to have no relevance, and not sure what you mean by it's "untested. It either works or it doesn't.

#3 in reply to: ↑ 2 @Nightgunner5
14 years ago

  • Cc nightgunner5@… added
  • Keywords has-patch added

Replying to matt:

Your patch includes other changes that seem to have no relevance, and not sure what you mean by it's "untested. It either works or it doesn't.

I haven't been able to test it because my test forum is on my local computer and can't send mail.

The "unrelated" changes are removal of whitespace from the ends of lines and a logic fix for a related function.

#4 @GautamGupta
14 years ago

  • Keywords subscription notification removed
  • Priority changed from low to normal

#5 @chrishajer
14 years ago

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

(In [2419]) Check bb_ksd_pre_post_status before sending out "subscribe to topic" notifications". Fixes #1233, props Nightgunner5

Note: See TracTickets for help on using tickets.