Closed
Bug 909373
Opened 11 years ago
Closed 11 years ago
[Music] Improve view activity saving feature robustness
Categories
(Firefox OS Graveyard :: Gaia::Music, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gerard-majax, Unassigned)
Details
(Whiteboard: MEDIA_TRIAGED)
Attachments
(1 file)
Bug 895726 showed that we should not trust the incoming filename completely. We will ensure that the extension matches the blob mime type so that saving to a specific device storage will not fail.
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → koi?
Comment 1•11 years ago
|
||
I'd suggest that the best way to fix this is to test the filename for correctness before displaying the Save button. If the filename is invalid, don't allow the user to save. I think this is better than allowing the user to save with a modified filename. This way we don't break, but we don't do something different than what was requested by the invoking app.
Updated•11 years ago
|
blocking-b2g: koi? → ---
Whiteboard: MEDIA_TRIAGED
Comment 2•11 years ago
|
||
Since htis is related to Bug 909372, I will work on this as well
Comment 3•11 years ago
|
||
Attachment #803511 -
Flags: review?(dkuo)
Updated•11 years ago
|
Attachment #803511 -
Attachment mime type: text/plain → text/html
Comment 4•11 years ago
|
||
Aras, I have new comments on github and please take a look on them, thanks!
Comment 5•11 years ago
|
||
Comment on attachment 803511 [details]
link to pull request on github
Aras,
The revised patch looks good to me, and after you squashed it, please needinfo me when you are going to land it, I will do it for you, thanks.
Attachment #803511 -
Flags: review?(dkuo) → review+
Comment 6•11 years ago
|
||
I added bug 852864 to the comment and squashed the commits. It should be ready for landing. Thanks Dominic.
Comment 7•11 years ago
|
||
Thanks!(Sorry I didn't notice the comment until today...)
Landed on master: e60b1eb012a3c18cca25671ebeac32df8f9a98b0
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•