WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#22905 closed defect (bug) (fixed)

PHPDoc refresh for register_sidebar()

Reported by: jeremyfelt Owned by: helen
Milestone: 3.6 Priority: normal
Severity: normal Version:
Component: Inline Docs Keywords: has-patch needs-codex
Focuses: Cc:
PR Number:

Description

Happened to notice that the docs for register_sidebar() include an example where the string argument had parameters separated by semi-colons, which parse_str() doesn't do.

Refreshed block to be a bit clearer/cleaner while changing that. Patch attached.

Attachments (3)

22905.diff (3.5 KB) - added by jeremyfelt 7 years ago.
Refresh of PHPDoc for register_sidebar
22905.2.diff (3.5 KB) - added by SergeyBiryukov 7 years ago.
A typo fix in line 526 and a couple of minor edits
22905.3.diff (697 bytes) - added by SergeyBiryukov 7 years ago.

Download all attachments as: .zip

Change History (8)

@jeremyfelt
7 years ago

Refresh of PHPDoc for register_sidebar

@SergeyBiryukov
7 years ago

A typo fix in line 526 and a couple of minor edits

#1 @SergeyBiryukov
7 years ago

  • Milestone changed from Awaiting Review to 3.6

#2 @helen
7 years ago

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

In 23196:

Refresh phpdoc for register_sidebar(). props jeremyfelt, SergeyBiryukov. fixes #22905.

#3 @SergeyBiryukov
7 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Two more minor fixes in 22905.3.diff.

#4 @helen
7 years ago

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

In 23199:

Minor phpdoc fixes for register_sidebar(). props SergeyBiryukov. fixes #22905.

#5 @DrewAPicture
7 years ago

  • Keywords needs-codex added

See phpdoc fixes/clarifications for register_sidebar() in [23196] and [23199]

Note: See TracTickets for help on using tickets.