Opened 12 years ago
Closed 12 years ago
#30540 closed defect (bug) (fixed)
Docblock for wp_get_nav_menu_to_edit in /wp-admin/includes/nav-menu.php incorrectly declares $menu_id param as string
| Reported by: | dustyf | Owned by: | DrewAPicture |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.1 |
| Component: | Menus | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description
The $menu_id param in wp_get_nav_menu_to_edit in file, /wp-admin/includes/nav-menu.php, should be declared as an int instead of a string. The parameter is declaring an ID of a specific menu that will be edited.
Attachments (2)
Change History (6)
#2
@
12 years ago
- Milestone Awaiting Review → 4.1
- Version trunk
Hi Jaime, thanks for the patch!
Version 0, edited 12 years ago by
(next)
#3
@
12 years ago
Hi Jaimie,
Just two things to note for future reference that I've adjusted in 30540.2.patch:
- The parameter is optional so we need to mark it as such with the default value
- Patches should be generated from the project root directory
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
param changed from string to int