[wp-trac] Re: [WordPress Trac] #4701: ID attribute for adminmenu
items
WordPress Trac
wp-trac at lists.automattic.com
Thu Aug 16 20:19:52 GMT 2007
#4701: ID attribute for adminmenu items
----------------------------+-----------------------------------------------
Reporter: leonid | Owner: anonymous
Type: enhancement | Status: new
Priority: normal | Milestone: 2.3 (trunk)
Component: Administration | Version:
Severity: normal | Resolution:
Keywords: has-patch |
----------------------------+-----------------------------------------------
Comment (by leonid):
I wish that part of menu-header.php was written somehow simpler. I have
to drink five pints before I can make any sense of it, and I do forget it
all by the next morning. :)
On the serious note, would calling sanitize_file_name($item[2]) be
sufficient or is there something else to be careful about? I can redo the
patch, no problem. But I really have issues understanding some of that
code (a bunch of if-then-else statements with re-indexing of a heavy
array).
--
Ticket URL: <http://trac.wordpress.org/ticket/4701#comment:4>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list