[wp-trac] [WordPress Trac] #54798: Check for int in menu priority

WordPress Trac noreply at wordpress.org
Tue Apr 26 03:42:24 UTC 2022


#54798: Check for int in menu priority
--------------------------------------+-----------------------
 Reporter:  kirtan95                  |       Owner:  audrasjb
     Type:  defect (bug)              |      Status:  closed
 Priority:  normal                    |   Milestone:  6.0
Component:  Administration            |     Version:
 Severity:  normal                    |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+-----------------------
Changes (by peterwilsoncc):

 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 I think this can be closed with the commits on #40927

 `add_menu_page()` position:

 * can be either a float or an integer in line with common plugin practices
 * the default position, `null`, adds the menu as the last item
 * a `_doing_it_wrong()` notice is triggered if the position is not
 numberic
 * `float` types are cast to `string` to ensure PHP doesn't internally cast
 the number to an integer
 * the position is sorted as a human would expect it, eg: `1, "1.043", "2",
 3`.

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


More information about the wp-trac mailing list