WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#56862 closed defect (bug) (fixed)

Docs: Default parameter value is missing for themes_api function

Reported by: rezakhan995 Owned by: audrasjb
Priority: normal Milestone: 6.2
Component: General Version: 2.8
Severity: normal Keywords: has-patch commit
Cc: Focuses: docs

Description

In wp-admin/includes/theme.php file, themes_api() function inline document does not mention default @param value. $args @param should have the default value mentioned.

Change History (9)

#1 @costdev
4 years ago

  • Version 6.1 → 2.8

This ticket was mentioned in ​PR #3500 on ​WordPress/wordpress-develop by ​rezakhan995.
4 years ago #2

  • Keywords has-patch added; needs-patch removed

In wp-admin/includes/theme.php file, themes_api() function inline document dose not mentioned default @param value. $args @param should have the default value mentioned.

Trac ticket:

#3 @costdev
4 years ago

  • Milestone Awaiting Review → 6.2

The PR looks good to me! Adding for commit consideration.

#4 @costdev
4 years ago

  • Keywords commit added

#5 @audrasjb
4 years ago

  • Owner set to audrasjb
  • Status new → accepted

Self assigning for commit.

#6 @audrasjb
4 years ago

  • Resolution → fixed
  • Status accepted → closed

In 54655:

Docs: Add missing default parameter value in themes_api() docblock.

Props rezakhan995, costdev.
Fixes #56862.
See #56792.

#8 @jorbin
4 years ago

In 54657:

Revert accidental 5.9 branch commits [54655] and [54656]

Unprops audrasjb.
See #56867, #56792, #56862, #56792.

#9 @jorbin
4 years ago

In 54659:

Docs: Add missing default parameter value in themes_api() docblock.

Previously: [54655] [54657]

Props rezakhan995, costdev.
Fixes #56862.
See #56792.

Note: See TracTickets for help on using tickets.