Opened 15 years ago
Closed 14 years ago
#2354 closed enhancement (fixed)
[patch] Add ordering to forum post replies
Reported by: | DJPaul | Owned by: | |
---|---|---|---|
Milestone: | 1.2.6 | Priority: | normal |
Severity: | Version: | ||
Component: | Groups | Keywords: | has-patch, tested |
Cc: |
Description
This patch allows the bp_has_forum_topic_posts() templatetag to order the replies in ascending or descending order (on timestamp).
By default, the default behaviour is "ascending"; that is, oldest post first. I've just extended this through up into the templatetag function to allow child themes to pass a new 'order' parameter to bp_has_forum_topic_posts().
Milestone set to 1.2.4 as I've provided a patch which is a minor enhancement which doesn't change or break the default behaviour.
Attachments (1)
Change History (9)
#1
@
15 years ago
- Milestone changed from 1.2.4 to 1.3
Good patch, but I gotta bump it to 1.3 since this will add functionality that could also use an interface and template changes to match.
It has my blessing for 1.3 though.
#2
@
15 years ago
Just a note: bp_get_the_topic_post_admin_links() will need updating for such a setting.
#5
@
14 years ago
What is the term to make this "descending" by default? IE. newest post first? ASC- I assume is ascending... DESC?
Adding this change doesn't seem to break anything...
#6
@
14 years ago
Ok got this... DESC reverses the order... This is exactly what I was hoping for. This works for me.
v1