WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#9522 closed defect (bug) (fixed)

New check in admin-ajax.php prevents use of static sidebar IDs

Reported by: wupperpirat's profile Wupperpirat Owned by: azaozz's profile azaozz
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: Widgets Keywords: static, sidebar ID, redesign
Focuses: Cc:

Description

While testing revision 10915 with Thematic I recognized that static sidebar IDs are no longer working.

I introduced this feature to Thematic a while ago to protect the widgetized areas against the behaviour of other plugins. For example: Tabbed Widgets registers an 'Invisible Sidebar' .. as long as you're using dynamic sidebar IDs (sidebar-x) this will move all widgets registered to a sidebar to the next sidebar and so on .. using static sidebars (primary-aside, secondary-aside and so on) will prevent this behavior.

With the redesign of the widgets admin page the static sidebar IDs won't work at all .. line 1281 in admin-ajax.php prevents that the widgets (moved to a certain sidebar) are saved.

Cheers,

Chris

Change History (2)

#1 @ryan
15 years ago

  • Component changed from Administration to Widgets
  • Owner changed from anonymous to azaozz

#2 @azaozz
15 years ago

  • Milestone changed from Unassigned to 2.8
  • Resolution set to fixed
  • Status changed from new to closed

That regex is gone in [10922]

Note: See TracTickets for help on using tickets.