Opened 12 years ago
Closed 12 years ago
#2250 closed defect (bug) (fixed)
bbPress Widgets Bugs
Reported by: | alex-ye | Owned by: | |
---|---|---|---|
Milestone: | 2.3 | Priority: | normal |
Severity: | normal | Version: | 2.2.3 |
Component: | API - Widgets | Keywords: | has-patch |
Cc: | nashwan.doaqan@… |
Description
Hi , When I try to filter some bbPress widgets I found that they have many bugs !
All Widgets Bugs :
1 - Filter widget_title doesn't do anything .
2 - Hard to change the default widget settings .
3 - The empty widget control form should show the defaults .
Some widgets Bugs :
1 - Can't avoid printing before_title and after_title even if the title is empty !
BBP_Topics_Widget :
1 - Typo show_stickes instead of show_stickies in query args .
2 - Unnecessary use of bbp_exclude_forum_ids() function .
3 - Unnecessary convert post_status array to string .
4 - I found parent_forum functionality but no option for the user .
BBP_Replies_Widget :
1 - Unnecessary use of bbp_exclude_forum_ids() function .
2 - Unnecessary convert post_status array to string .
...etc
Attachments (1)
Change History (6)
#3
in reply to:
↑ 2
@
12 years ago
Replying to johnjamesjacoby:
Thanks for the patch. It will need some cleaning up, but it's a good start. In the future, it's best to break these up into smaller tickets and patches, so we can address them individually.
Moving to 2.3
Thanks :) , I will follow your advise in the future .
#4
@
12 years ago
When testing this patch can we double check that any topics from Private or Hidden Forums do not show in the widget when the user does not posses the required Role / Capability.
(This was was recently reported on bbpress.org forums though I could not reproduce the issue.)
- User not logged in.
- User logged in with role 'Participant'
- User logged in with role 'Moderator'
- User logged in as 'Keymaster/SuperAdmin'
Thanks for the patch. It will need some cleaning up, but it's a good start. In the future, it's best to break these up into smaller tickets and patches, so we can address them individually.
Moving to 2.3