[wp-trac] [WordPress Trac] #16600: AdminMenu rendering code chokes on uppercase

WordPress Trac wp-trac at lists.automattic.com
Sun Feb 20 14:47:44 UTC 2011


#16600: AdminMenu rendering code chokes on uppercase
----------------------------+------------------------------
 Reporter:  jltallon        |       Owner:
     Type:  defect (bug)    |      Status:  reopened
 Priority:  normal          |   Milestone:  Awaiting Review
Component:  Administration  |     Version:  3.1
 Severity:  minor           |  Resolution:
 Keywords:  needs-patch     |
----------------------------+------------------------------

Comment (by jltallon):

 wp-includes/post.php, line 822:

    $post_type = sanitize_user($post_type, true);

 ( whereas taxonomy.php, line 313 reads "$args [ ' object_type ' ] =
 (array) $object_type;" )

 The problem being that sanitize_user, in strict mode, changes the intended
 "custom post type name".
 Moreover, it looks like a C-P-T (or taxonomy, for that matter) name
 shouldn't contain the 'at' sign, right?

 Hence, I propose to introduce a new sanitize_object_name function (not
 filter) for this matter.

 (patch follows)

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16600#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list