Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#15493 closed defect (bug) (fixed)

Twenty Ten: twentyten_remove_recent_comments_style() throws notice when Recent Comments not in use

Reported by: kawauso's profile kawauso Owned by: nacin's profile nacin
Milestone: 3.1 Priority: normal
Severity: trivial Version:
Component: Themes Keywords: has-patch
Focuses: Cc:


twentyten_remove_recent_comments_style() uses $wp_widget_factory->widgets['WP_Widget_Recent_Comments'], but this is not present when the Recent Comments widget is not in use and the user cannot modify widgets, resulting in an undefined index.

Attachments (1)

15493.diff (667 bytes) - added by kawauso 13 years ago.

Download all attachments as: .zip

Change History (6)

13 years ago

#1 @kawauso
13 years ago

  • Component changed from General to Themes
  • Severity changed from normal to trivial

#2 @nacin
13 years ago

  • Milestone changed from Awaiting Review to 3.1
  • Owner set to nacin
  • Status changed from new to accepted

This widget factory code makes my eyes glaze over. (And I wrote it.) I'm making this a filter.

#3 @nacin
13 years ago

Ran into #14876 while working on this.

#4 @nacin
13 years ago

(In [16521]) Revert [15994] for now, as the default widgets then go through an identity crisis. see #14876, #15493.

#5 @nacin
13 years ago

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

(In [16522]) Add a simple filter to allow removing the recent comments default widget styles. fixes #15493, see #14876.

Note: See TracTickets for help on using tickets.