WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 10 years ago

#35147 closed defect (bug) (fixed)

unregister_sidebar function of the comment document

Reported by: tmatsuur Owned by: Soean
Priority: normal Milestone: 4.8
Component: Widgets Version: 4.4
Severity: normal Keywords: good-first-bug has-patch
Cc: Focuses:

Description

Parameter of 'is_registered_sidebar' functions that have been added in version 4.4 has become a string or int.

In contrast to this, the parameter of the 'unregister_sidebar' function has become a string.

Both parameters is the key of the $wp_registered_sidebars. I think it is better to be unified.

Attachments (1)

35147.diff​ (701 bytes ) - added by Soean 11 years ago.

Download all attachments as: .zip

Change History (8)

#2 in reply to: ↑ 1 @tmatsuur
11 years ago

Replying to welcher:
Thank you.

#4 @welcher
11 years ago

  • Keywords has-patch dev-feedback added

#5 @welcher
11 years ago

  • Keywords good-first-bug needs-patch added; has-patch dev-feedback removed

@Soean
11 years ago

#6 @Soean
11 years ago

  • Keywords has-patch added; needs-patch removed

I copied the parameter and comment from 'is_registered_sidebar' to 'unregister_sidebar'. So that it's now unified.
See: 35147.diff

Last edited 11 years ago by Soean (previous) (diff)

#7 @DrewAPicture
10 years ago

  • Owner set to Soean
  • Status new → assigned

Assigning to mark the good-first-bug as "claimed".

See 35147.diff​

#8 @SergeyBiryukov
10 years ago

  • Milestone Awaiting Review → 4.8

#9 @SergeyBiryukov
10 years ago

  • Resolution → fixed
  • Status assigned → closed

In 39892:

Widgets: In unregister_sidebar(), rename the $name parameter to $sidebar_id for consistency with is_registered_sidebar().

Also correct the parameter type in @param entry.

Props Soean, tmatsuur.
Fixes #35147.

Note: See TracTickets for help on using tickets.