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

WordPress Trac noreply at wordpress.org
Thu Jul 24 22:08:12 UTC 2014


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

 * owner:   => wonderboymusic
 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 In [changeset:"29290"]:
 {{{
 #!CommitTicketReference repository="" revision="29290"
 In `sanitize_file_name()`, replace `%20` and `+` with dashes. Remove
 unnecessary code from `_wp_handle_upload()`.

 Adds unit tests.

 Props ericmann.
 Fixes #16330.
 }}}

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


More information about the wp-trac mailing list