Skip to:
Content

Opened 3 years ago

Closed 18 months ago

#2758 closed enhancement (fixed)

Topics widget: Wrong way to output the field ID

Reported by: alex-ye Owned by: netweb
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.1
Component: API - Widgets Keywords: has-patch commit
Cc:

Description

In the Topics Widget, We used $this->get_field_name() instead of $this->get_field_id() to output the 'order_by' field ID.

Attachments (1)

widgets.patch (1.0 KB) - added by alex-ye 3 years ago.

Download all attachments as: .zip

Change History (3)

@alex-ye
3 years ago

#1 @thebrandonallen
18 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 2.6
  • Version set to 2.1

Good catch @alex-ye!

#2 @netweb
18 months ago

  • Owner set to netweb
  • Resolution set to fixed
  • Status changed from new to closed

In 6038:

Widgets: Use $this->get_field_id() in BBP_Topics_Widget::form() method to correctly create the widget id form attribute

Props alex-ye
Fixes #2758

Note: See TracTickets for help on using tickets.