Make WordPress Core

Opened 18 years ago

Closed 18 years ago

Last modified 4 months ago

#6023 closed defect (bug) (fixed)

Widgets registering multpile instances don't display in available widgets list.

Reported by: mdawaffe's profile mdawaffe Owned by: mdawaffe's profile mdawaffe
Milestone: 2.5 Priority: normal
Severity: major Version: 2.5
Component: Administration Keywords:
Focuses: Cc:

Description

When widgets use the new-fangled "multi-widget" code, corrupt or un-upgraded data can sometimes prevent any widgets from showing. This is due to a bad "are there any widgets of this type" check in WordPress core.

Attached fixes that problem.

When widgets use the old-fangled "how many of these widgets do you want to use: 1,2,..." UI and then register that many widgets, only the first instance is accessible.

Attached does not fix that problem. Debugging now.

Attachments (2)

6023.diff (5.2 KB) - added by mdawaffe 18 years ago.
fix new-fangled multi widgets
6023-old.diff (2.0 KB) - added by mdawaffe 18 years ago.
old-fangled

Download all attachments as: .zip

Change History (7)

@mdawaffe
18 years ago

fix new-fangled multi widgets

#1 @mdawaffe
18 years ago

  • Owner changed from anonymous to mdawaffe
  • Status changed from new to assigned

@mdawaffe
18 years ago

old-fangled

#3 @mdawaffe
18 years ago

6023-old.diff fixes the problem with the test widget I have. If you have a widget that doesn't work under this patch, please attach it to this ticket so I can debug some more.

#4 @ryan
18 years ago

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

(In [7080]) Fix widgets using old-style multi instance support. Props mdawaffe. fixes #6023

#5 @westonruter
4 months ago

In 61466:

Widgets: Ensure a widget is registered prior to checking its ID in is_active_widget().

This avoids a possible PHP warning due to an undefined array key.

Developed in https://github.com/WordPress/wordpress-develop/pull/10710

Follow-up to [11090], [7080].

Props sageth, josephscott, Ipstenu, mindctrl, westonruter.
See #8441, #6023.
Fixes #57518.

Note: See TracTickets for help on using tickets.