[wp-trac] [WordPress Trac] #16330: media_sideload_image() broken with filenames containing strange characters (e.g., +, %)

WordPress Trac noreply at wordpress.org
Fri Jul 18 01:10:42 UTC 2014


#16330: media_sideload_image() broken with filenames containing strange characters
(e.g., +, %)
---------------------------------------------+-----------------------
 Reporter:  Coolkevman                       |       Owner:
     Type:  defect (bug)                     |      Status:  reopened
 Priority:  normal                           |   Milestone:  4.0
Component:  Media                            |     Version:  3.1
 Severity:  major                            |  Resolution:
 Keywords:  commit has-patch has-unit-tests  |     Focuses:
---------------------------------------------+-----------------------

Comment (by SergeyBiryukov):

 The replacements in `_wp_handle_upload()` (in the first part of the patch)
 are redundant since [11178], so removing them makes sense.

 [attachment:16330.6.diff] looks good to me.

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


More information about the wp-trac mailing list