Making WordPress.org

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#1140 closed defect (bug) (duplicate)

"Pending" forum reply in stats, but not in topic

Reported by: medariox's profile medariox Owned by: netweb's profile netweb
Milestone: Priority: high
Component: International Forums Keywords: needs-patch
Cc:

Description

A reply from a moderator got tagged as "pending", but shows in the forum stats anyway.
The short reply, two w.org internal links and a small amount of text, doesn't show up in the topic at all (see attached screenshots).

Attachments (3)

forum.png (27.3 KB) - added by medariox 9 years ago.
Forum overview with (incorrect) stats
pending.png (8.5 KB) - added by medariox 9 years ago.
"Pending" status of the reply
topic.png (34.8 KB) - added by medariox 9 years ago.
Topic showing 2 replies, but 3 partecipants

Download all attachments as: .zip

Change History (11)

@medariox
9 years ago

Forum overview with (incorrect) stats

@medariox
9 years ago

"Pending" status of the reply

@medariox
9 years ago

Topic showing 2 replies, but 3 partecipants

#1 @netweb
9 years ago

  • Owner set to netweb
  • Status changed from new to reviewing

#2 in reply to: ↑ description @netweb
9 years ago

Replying to medariox:

The short reply, two w.org internal links and a small amount of text, doesn't show up in the topic at all (see attached screenshots).

bbPress inherits WordPress' setting, the default is 2, @ocean90 can you bump that up a few, 6, maybe for starters

"/wp-admin/options-discussion.php

"Hold a comment in the queue if it contains 2 or more links. (A common characteristic of comment spam is a large number of hyperlinks.)"

#3 @netweb
9 years ago

In [bbPress5870]:

Tests: Add topic voice count test test_bbp_get_topic_voice_count_with_pending_reply().

This test tests the voice count of a topic to ensure replies with post status pending are not included in the voice count until approved.

#4 @netweb
9 years ago

In [bbPress5871]:

Tests: Add forum last active id test test_bbp_get_forum_last_active_id_with_pending_reply().

This test tests the forums last active id to ensure replies with post status pending are not used as the forums last active id until the reply is approved.

#5 @ocean90
9 years ago

@netweb So what's the issue here?

This ticket was mentioned in Slack in #meta-i18n by netweb. View the logs.


9 years ago

#7 @netweb
9 years ago

  • Resolution set to invalid
  • Status changed from reviewing to closed

Closing this here as I've migrated the issue to bbPress Trac in ticket #bbPress2838

In the future please submit bbPress bugs to bbPress Trac, and any forums theme issues here on Meta Trac

https://bbpress.trac.wordpress.org/

#8 @ocean90
9 years ago

  • Resolution changed from invalid to duplicate
Note: See TracTickets for help on using tickets.