Skip to:
Content

bbPress.org

Opened 15 years ago

Closed 15 years ago

#1320 closed task (blessed) (fixed)

Post count functions are needed

Reported by: gautamgupta's profile GautamGupta Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.0 Priority: highest omg sweet tea
Severity: normal Version:
Component: Back-end Keywords:
Cc:

Description

Here's the proposed solution:

  • Have bbp_forum_id meta for replies
  • Have bbp_topic_count and bbp_topic_reply_count meta for forums
  • Have recount functionality

Attachments (1)

counting-123.diff​ (19.4 KB) - added by GautamGupta 15 years ago.
Works like a charm for me. TODO: Recount

Download all attachments as: .zip

Change History (11)

#1 @GautamGupta
15 years ago

  • Component changed from Administration to Back-end

#2 @GautamGupta
15 years ago

  • Summary changed from Forums need reply count functions to Post count functions are needed

#3 @johnjamesjacoby
15 years ago

  • Priority changed from high to highest

#4 @johnjamesjacoby
15 years ago

  • Owner set to johnjamesjacoby

#5 @johnjamesjacoby
15 years ago

Halfway there. Still some sub-forum logic to sort out (walking up the tree and rejigging all the counts)

#6 @johnjamesjacoby
15 years ago

Hope to revisit this over the weekend.

#7 @elfin
15 years ago

assuming this is related - I just moved some topic and the forum count for topics wasn't updated - thereby showing an incorrect count.

#8 @elfin
15 years ago

and the forum i moved them to shows 4 topics, but the freshness says No topics.

@GautamGupta
15 years ago

Works like a charm for me. TODO: Recount

#9 @GautamGupta
15 years ago

Attached a patch, with the solution as discussed in chat. Contains 2 unrelated changes: sticky css and revision id in the id attribute of the revision log item.

#10 @johnjamesjacoby
15 years ago

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

(In [2806]) Adjust topic and reply counts up the subforum tree. Fixes #1320. Props GautamGupta

Note: See TracTickets for help on using tickets.