[wp-trac] [WordPress Trac] #12349: wp_insert_term doesn't check for already existing slugs if the slug is implicit

WordPress Trac wp-trac at lists.automattic.com
Sat May 22 23:30:34 UTC 2010


#12349: wp_insert_term doesn't check for already existing slugs if the slug is
implicit
--------------------------+-------------------------------------------------
 Reporter:  nbachiyski    |       Owner:  dd32     
     Type:  defect (bug)  |      Status:  assigned 
 Priority:  high          |   Milestone:  3.0      
Component:  Taxonomy      |     Version:           
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by ryan):

 That check does prevent the user from doing really silly things with
 slugs. Like creating an "Apple" tag with "apple" slug and then creating an
 "Orange" tag with an "apple" slug (which will end up apple-2 and two tags
 with the same name will appear).  Perhaps a better patch is to do the dupe
 check only if the provided slug does not match the slug generated from the
 name. This way people who fill in the slug to match the name won't get a
 dupe error.

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


More information about the wp-trac mailing list