[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