[wp-trac] [WordPress Trac] #35106: Responsive images break uploads with full path stored in metadata

[WordPress Trac]

#35106: Responsive images break uploads with full path stored in metadata
--------------------------------------+------------------------
 Reporter:  SergeyBiryukov            |       Owner:  joemcgill
     Type:  defect (bug)              |      Status:  accepted
 Priority:  normal                    |   Milestone:  4.4.1
Component:  Media                     |     Version:  4.4
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:2 joemcgill]:
 > The fix looks like it will work but makes me wonder if we should
 institutionalize this knowledge in the form of a helper function for
 calculating the file path of an image, which could be used in both places?

 Good point, [attachment:35106.2.patch] introduces
 `_wp_get_attachment_relative_path()` and uses it in both places.

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