[wp-trac] [WordPress Trac] #48458: Regression: Auto-rotated or scaled image upload can overwrite previously uploaded image

[WordPress Trac]

#48458: Regression: Auto-rotated or scaled image upload can overwrite previously
uploaded image
--------------------------+------------------------------
 Reporter:  ianmjones     |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Media         |     Version:
 Severity:  major         |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------------
Changes (by ianmjones):

 * keywords:  close 2nd-opinion =>


Comment:

 Sorry, missed fixing up a bit of copy-paste...

 Replying to [ticket:48458 ianmjones]:
 > This creates files such as...
 >
 > 2019/10/image-rotated.jpg ''<- base file (full)''
 > 2019/10/image-rotated-150x150.jpg ''<-thumbnail''
 > 2019/10/image.jpg ''<- original_image''

 Should be...

 2019/10/image-rotated.jpg ''<- base file (full)''
 2019/10/image-150x150.jpg ''<-thumbnail''
 2019/10/image.jpg ''<- original_image''

 @desrosj I understand why you say it's not a regression, but seeing as
 we're talking about the auto-generated base file names that should
 definitely be run through `wp_unique_filename` before being used in
 `_wp_attached_file`, I believe this is a regression unique to the new
 rotation and scaling methods.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48458#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform