Opened 12 years ago
Closed 7 years ago
#4240 closed enhancement (maybelater)
Make "get pagination" filters more conventionalized
Reported by: | cklosows | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | minor | Version: | 1.5.5 |
Component: | Core | Keywords: | needs-patch, trac-tidy-2018 |
Cc: | cklosowski@… |
Description
The filters for getting pagination for groups, forums, and topics are all named slightly differently, making adding filters more difficult to do without looking up the code in the core file:
bp_get_the_topic_pagination
bp_get_groups_pagination_links
bp_get_forum_pagination
I propose creating a naming convention for these since they all return the same type of data, full html of the pagination.
bp_get_[type]_pagination
Change History (8)
#2
follow-up:
↓ 3
@
12 years ago
Cool. A tip: We can't just rename filters for backward compatibility reasons, so the best practice is to add a new filter, and leave an inline comment to the effect of "use this one, not that one".
#3
in reply to:
↑ 2
@
12 years ago
Replying to boonebgorges:
Cool. A tip: We can't just rename filters for backward compatibility reasons, so the best practice is to add a new filter, and leave an inline comment to the effect of "use this one, not that one".
Noted. I'll remember that as I'm writing the patch. I had planned on doing something like that but thanks for the official stance. :)
#4
@
12 years ago
- Component changed from Core to All Components
- Milestone changed from Awaiting Review to Future Release
#6
@
8 years ago
While we definitely can't remove the old ones, we can soft deprecate them and put the newly consistent ones immediately after.
$foo = apply_filters( 'old_filter', $thing, $thing ); $foo = apply_filters( 'new_filter', $thing, $thing );
We can also move to conditionally assign the values from the old filters only if has_action/has_filter returns true. This would instead have us using the new hooks first, followed by the old ones.
$foo = apply_filters( 'new_filter', $thing, $thing ); if ( has_filter( 'old_filter' ) ) { $foo = apply_filters( 'old_filter', $thing, $thing ); }
#7
@
7 years ago
- Keywords trac-tidy-2018 added
We're closing this ticket because it has not received any contribution or comments for at least two years. We have decided that it is better to close tickets that are good ideas, which have not gotten (or are unlikely to get) contributions, rather than keep things open indefinitely. This will help us share a more realistic roadmap for BuddyPress with you.
Everyone very much appreciates the time and effort that you spent sharing your idea with us. On behalf of the entire BuddyPress team, thank you.
If you feel strongly that this enhancement should still be added to BuddyPress, and you are able to contribute effort towards it, we encourage you to re-open the ticket, or start a discussion about it in our Slack channel. Please consider that time has proven that good ideas without contributions do not get built.
For more information, see https://bpdevel.wordpress.com/2018/01/21/our-awaiting-contributions-milestone-contains/
or find us on Slack, in the #buddypress channel: https://make.wordpress.org/chat/
I will get a patch together for this in the next day or so.