Skip to:

Opened 14 years ago

Closed 14 years ago

#2684 closed defect (bug) (fixed)

First Comment Doesn't Register Activity

Reported by: nahummadrid's profile nahummadrid Owned by:
Milestone: 1.2.7 Priority: normal
Severity: Version:
Component: Core Keywords: needs-testing


If WP Discussion settings are set to "Comment Author must have a previously approved comment" and NOT "An admin must always approve the comment", upon a new user's first comment being approved it does not get an activity item registered for that comment.

When that same user makes second comment and it goes through only then do the bp activity items begin to fire.

Can anyone confirm? This wasn't an issue having both settings checked.

looking at all the settings:
Allow people to post comment on new article (checked)
Comment author must fill out name and email (checked)
Users must be register and logged in to comment (unchecked)
An administrator must always approve the comment (unchecked)
Comment author must have a previously approved comment (checked)

using Multisite WP3/BP 1.26

Change History (7)

#1 @johnjamesjacoby
14 years ago

  • Keywords needs-testing added
  • Milestone changed from 1.2.6 to 1.2.7

#2 @DJPaul
14 years ago

Not tested but I believe we'll need to use the 'comment_hold_to_approve' action to capture these approved comments. See wp_transition_comment_status().

#3 @johnjamesjacoby
14 years ago

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

(In [3420]) Fixes #2684 (branch)

#4 @johnjamesjacoby
14 years ago

(In [3421]) Fixes #2684 (trunk)

#5 @DJPaul
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

I want to check if an comment marked as spam (i.e. by Akismet) would get through this; I'll report back :)

#6 @pisanojm
14 years ago

@djpaul @buddypressoverlords

Hey this ticket has been sitting here for two weeks without any further feedback and is holding up the release...are there any further patches coming to test?

#7 @DJPaul
14 years ago

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

It's not holding up release, but I think we're good to close this ticket.

Note: See TracTickets for help on using tickets.