WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#6949 closed defect (bug) (fixed)

Widget IDs need to be made lowercase

Reported by: filosofo Owned by:
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch wp_get_sidebars_widgets wp_register_sidebar_widget wp_register_widget_control widgets
Focuses: Cc:

Description

I've been beating my head against a wall trying to figure out why my multi-widget refused to save. It turns out that it was because I was using !camelCase in the id argument of the wp_register_sidebar_widget function call. wp_get_sidebars_widgets checks whether the name matches the strtolower version of itself, so without my patch, an id of varied case will fail the check every time.

Attachments (1)

widget_id.diff (709 bytes) - added by filosofo 6 years ago.

Download all attachments as: .zip

Change History (2)

filosofo6 years ago

comment:1 ryan6 years ago

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

(In [7915]) strtolower widget id. Props filosofo. fixes #6949

Note: See TracTickets for help on using tickets.