[wp-trac] [WordPress Trac] #30540: Docblock for wp_get_nav_menu_to_edit in /wp-admin/includes/nav-menu.php incorrectly declares $menu_id param as string

WordPress Trac noreply at wordpress.org
Sun Nov 30 04:24:27 UTC 2014


#30540: Docblock for wp_get_nav_menu_to_edit in /wp-admin/includes/nav-menu.php
incorrectly declares $menu_id param as string
--------------------------+---------------------------
 Reporter:  dustyf        |       Owner:  DrewAPicture
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  4.1
Component:  Menus         |     Version:
 Severity:  normal        |  Resolution:  fixed
 Keywords:  has-patch     |     Focuses:  docs
--------------------------+---------------------------
Changes (by DrewAPicture):

 * owner:   => DrewAPicture
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"30627"]:
 {{{
 #!CommitTicketReference repository="" revision="30627"
 Use the correct type (integer) for the `$menu_id` parameter in the
 `wp_get_nav_menu_to_edit()` DocBlock.

 Also properly mark the parameter as optional and specify the default
 value.

 Props jaimieolmstead for the initial patch.
 Fixes #30540.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/30540#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list