WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#25528 closed defect (bug) (fixed)

Hook docs: wp-admin/nav-menus.php

Reported by: pauldewouters Owned by: DrewAPicture
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: Inline Docs Keywords: has-patch commit
Focuses: Cc:

Description

Added hook documentation for the following actions/filters found in wp-includes/nav-menus.php:

  • File header
  • Required / Included files
  • wp_nav_menu_max_depth
  • after_menu_locations_table
  • wp_nav_locations_listed_per_menu

Attachments (3)

nav-menus-hook-docs.diff (2.9 KB) - added by pauldewouters 8 years ago.
nav-menus-hook-docs-diff-rev1.diff (1.2 KB) - added by pauldewouters 8 years ago.
25528.diff (1.3 KB) - added by DrewAPicture 8 years ago.
clean up

Download all attachments as: .zip

Change History (7)

#1 @DrewAPicture
8 years ago

  • Owner set to DrewAPicture
  • Status changed from new to reviewing
  • Type changed from enhancement to defect (bug)

#2 @kpdesign
8 years ago

  • Keywords needs-patch added; has-patch removed

@pauldewouters: Thanks for the patch. :)

Just a couple of things:

  • Please remove all functional doc changes (file header, functions, and requires/includes) - we're only documenting the hooks right now.
  • Version numbers should be 3-digit (i.e., 3.6.0).

Could you make these changes and resubmit the patch please?

@DrewAPicture
8 years ago

clean up

#3 @DrewAPicture
8 years ago

  • Keywords has-patch commit added; needs-patch removed
  • Milestone changed from Awaiting Review to 3.8

25528.diff should do it.

#4 @DrewAPicture
8 years ago

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

In 25993:

Inline documentation for hooks in wp-admin/nav-menus.php.

Props pauldewouters for the intial patch.
Fixes #25528.

Note: See TracTickets for help on using tickets.