Skip to:
Content

bbPress.org

Opened 3 weeks ago

Closed 3 weeks ago

Last modified 3 weeks ago

#3697 closed defect (bug) (fixed)

Forum form helpers omit moderators and mishandle dropdowns

Reported by: johnjamesjacoby Owned by: johnjamesjacoby
Priority: normal Milestone: 2.6.20
Component: Component - Forums Version: 2.4
Severity: normal Keywords:
Cc:

Description

Three defects in forum form template helpers are present in 2.6.19:

  1. bbp_get_form_forum_moderators() resolves assigned moderator IDs to nicename strings, then tries to pluck user_nicename from those strings. The edit field is blank even when moderators are assigned.
  2. The new-forum type dropdown selects the public post status (publish), which is not one of its forum or category options. No option is explicitly selected.
  3. The visibility dropdown applies bbp_get_form_forum_type_dropdown instead of a visibility-specific filter. Callbacks on bbp_get_form_forum_visibility_dropdown never run.

Show assigned moderator nicenames when editing, explicitly select forum for a new forum, and add a distinct visibility filter while preserving the existing type-filter call for compatibility. Cover these paths with PHPUnit regression tests. Fix on trunk and backport the narrow source changes and regressions to branches/2.6 for 2.6.20. The broader forum-template coverage remains tracked in #3695.

Change History (3)

#1 @johnjamesjacoby
3 weeks ago

In 7772:

Forums: Correct form moderator and dropdown values.

Restore assigned moderator nicenames in forum edit forms, explicitly select the forum type for new forums, and add a visibility dropdown filter. Preserve the existing type dropdown filter call for compatibility. Complete the forum-form template tests and add regressions for these defects.

In trunk, for 2.7.

See #3697.
See #3695.

#2 @johnjamesjacoby
3 weeks ago

  • Owner set to johnjamesjacoby
  • Resolution → fixed
  • Status new → closed

In 7773:

Forums: Backport forum form helper corrections.

Restore assigned moderator nicenames, select the forum type for new forums, and add the visibility dropdown filter while preserving the existing type-filter call. Add targeted regression tests without backporting the broader 2.7 coverage.

In branches/2.6, for 2.6.20.

Fixes #3697.

#3 @johnjamesjacoby
3 weeks ago

  • Version 2.6.19 → 2.4
Note: See TracTickets for help on using tickets.