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

WordPress Trac noreply at wordpress.org
Thu Jul 17 02:06:05 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:
---------------------------------------------+-----------------------
Changes (by ericmann):

 * keywords:  needs-unit-tests => commit has-patch has-unit-tests


Comment:

 Latest patch builds on previous ones - replacing both `%20` and `+` with
 `-`. It adds multiple test conditions for spaces, though, testing
 unencoded spaces, encoded spaces, and a mix of all three possible versions
 (`%20`, `+`, and ` `).

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


More information about the wp-trac mailing list