[wp-trac] [WordPress Trac] #22813: Media Uploader doesn't escape "+" in filenames and doesn't upload file

[WordPress Trac]

#22813: Media Uploader doesn't escape "+" in filenames and doesn't upload file
----------------------------------------+------------------------------
 Reporter:  devinreams                  |       Owner:
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  Awaiting Review
Component:  Media                       |     Version:  3.4.2
 Severity:  normal                      |  Resolution:
 Keywords:  has-patch needs-unit-tests  |
----------------------------------------+------------------------------
Changes (by nacin):

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


Comment:

 I don't really see why we should allow + in filenames. It's nice that we
 mostly handle it, I guess, but that's not enough.

 I think we should close this as a duplicate of #16330. Very similar
 problem, same solution.

 On the other hand, can we figure out why this is broken? As in, how can we
 make this work while still supporting `+` in filenames? Not saying we
 ''should'', but it would help to understand why this is broken.

 This sounds like a good candidate for unit tests.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/22813#comment:14>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software