WordPress.org
Get WordPress

Make WordPress Core

Opened 16 months ago

Closed 16 months ago

Last modified 13 months ago

#63649 closed defect (bug) (duplicate)

Missing PHPDoc block for the twentythirteen_credits action hook in the Twenty Thirteen theme

Reported by: pmbaldha Owned by:
Priority: normal Milestone:
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description (last modified by sabernhardt)

The Twenty Fourteen theme lacks a PHPDoc block for the twentyfourteen_credits action hook. For more information, refer to ​https://github.com/WordPress/wordpress-develop/blob/6.8.1/src/wp-content/themes/twentythirteen/footer.php#L18.

Please add the PHPDoc block for it as described on ​https://developer.wordpress.org/coding-standards/inline-documentation-standards/php/#4-hooks-actions-and-filters.

Change History (4)

This ticket was mentioned in ​PR #9173 on ​WordPress/wordpress-develop by ​@pmbaldha.
16 months ago #1

  • Keywords has-patch added

Trac ticket: https://core.trac.wordpress.org/ticket/63649

#2 in reply to: ↑ description @pmbaldha
16 months ago

Sorry, my mistake.
twentythirteen_credits* in place of twentyfourteen_credits.

Replying to pmbaldha:

The Twenty Fourteen theme lacks a PHPDoc block for the twentyfourteen_credits action hook. For more information, refer to ​https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-content/themes/twentythirteen/footer.php#L18.

Please add the PHPDoc block for it as described on ​https://developer.wordpress.org/coding-standards/inline-documentation-standards/php/#4-hooks-actions-and-filters.

#3 @sabernhardt
16 months ago

  • Description modified (diff)
  • Focuses docs added; coding-standards removed
  • Milestone Awaiting Review
  • Resolution → duplicate
  • Status new → closed

Duplicate of #63648.

I'm combining these four tickets, but you can keep the pull requests separate and update the link in each to
https://core.trac.wordpress.org/ticket/63648

​@SergeyBiryukov commented on ​PR #9173:
13 months ago #4

Thanks for the PR! Merged in r60740.

Note: See TracTickets for help on using tickets.