Closed Bug 494328 Opened 11 years ago Closed 10 years ago
Media file display names should use decode
URIComponent instead of decode URI
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-GB; rv:1.9.1b4) Gecko/20090423 Firefox/3.5b4 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-GB; rv:1.9.1b4) Gecko/20090423 Firefox/3.5b4 Just nitpicking really, since we're only displaying the filename of the media file, there are no special characters to watch out for. Reproducible: Always Steps to Reproduce: Open a podcast feed with files named like "Episode #4", this will be encoded in the XML as "Episode%20%234" Actual Results: Will display as "Episode %234" Expected Results: Should display as "Episode #4"
Attachment #379033 - Flags: review? → review?(sayrer)
Comment on attachment 379033 [details] [diff] [review] Really simple patch Requested review from sayrer.
This is a simple testcase.
This is the actual testcase.
Attachment #437042 - Attachment is obsolete: true
We're waiting on an automated testcase to land this.
Attachment #450612 - Flags: review?(sayrer) → review?(mano)
Geoff: Do you have check-in privileges?
This requires approval to land now.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b3
You need to log in before you can comment on or make changes to this bug.