bbPress Widgets Bugs

[alex-ye]

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

[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

[alex-ye]

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 .

[netweb]

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'

[johnjamesjacoby]

(In [4806]) Audit up widget code:

• Add missing UI for some options.
• Introduce methods for parsing widget arguments.
• General code clean up.
• Props alex-ye.
• Fixes #2250.