WordPress.org

Make WordPress Core

#21886 closed enhancement (fixed)

PHPDocs for options_reading_blog_charset()

Reported by: SergeyBiryukov Owned by: nacin
Milestone: 3.5 Priority: normal
Severity: trivial Version: 3.5
Component: Inline Docs Keywords: has-patch
Focuses: Cc:

Description

Background: #21507

Attachments (4)

21886.patch (1.7 KB) - added by SergeyBiryukov 20 months ago.
21886.2.patch (1.7 KB) - added by SergeyBiryukov 20 months ago.
Refreshed after [21856]
21886.3.patch (1.5 KB) - added by SergeyBiryukov 19 months ago.
21886.4.patch (1.2 KB) - added by SergeyBiryukov 19 months ago.

Download all attachments as: .zip

Change History (9)

SergeyBiryukov20 months ago

SergeyBiryukov20 months ago

Refreshed after [21856]

comment:1 nacin19 months ago

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

In [21872]:

Docs for options_reading_blog_charset(). props SergeyBiryukov, fixes #21886.

SergeyBiryukov19 months ago

comment:2 SergeyBiryukov19 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

add_js() was originally added in 3.1: [15946] (for #15205). Was the change to 3.5 in the commit intentional?

21886.3.patch adds @since for other *_add_js() functions.

Not sure if @package and @subpackage are relevant for those functions. The patch removes them for consistency with [21872].

comment:3 nacin19 months ago

@package and @subpackage are rarely relevant for functions at all. Since we renamed add_js(), I changed the @since to when it was actually introduced. Normally we'd also include some info about when it was originally introduced (before rename), but this is just a formality, as these are internal functions anyway.

SergeyBiryukov19 months ago

comment:4 SergeyBiryukov19 months ago

I see. 21886.4.patch adds @since 3.5.0 for other renamed functions.

comment:5 nacin19 months ago

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

In [21994]:

@since for some internal functions. props SergeyBiryukov, fixes #21886.

Note: See TracTickets for help on using tickets.