[wp-trac] Re: [WordPress Trac] #9726: wp_unique_post_slug() not used when inserting attachment from media / add new

WordPress Trac wp-trac at lists.automattic.com
Tue May 26 09:12:56 GMT 2009


#9726: wp_unique_post_slug() not used when inserting attachment from media / add
new
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |        Owner:  Denis-de-Bernardy
     Type:  defect (bug)       |       Status:  accepted         
 Priority:  normal             |    Milestone:  2.8              
Component:  Upload             |      Version:  2.8              
 Severity:  normal             |   Resolution:                   
 Keywords:  needs-patch        |  
-------------------------------+--------------------------------------------
Changes (by Denis-de-Bernardy):

  * keywords:  has-patch tested commit => needs-patch
  * owner:  => Denis-de-Bernardy
  * status:  reopened => accepted


Comment:

 Replying to [comment:7 azaozz]:
 > Perhaps the proper solution is to check attachment slugs against other
 attachments and also pages.

 That's precisely what wp_unique_post_slug() is supposed to do. :-)

 > To reproduce with the patch: upload a file to a post; upload another
 file with the same name to different post; visit the attachment post for
 one of the files. Both attachment posts are displayed (may be theme
 dependent but happens in the default theme).

 OK. Will give it a shot and nail it down.

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


More information about the wp-trac mailing list