Closed
Bug 2017179
Opened 3 months ago
Closed 1 month ago
Unable to rename a downloaded file to uppercase (or lowercase)
Categories
(Firefox for Android :: Downloads, defect)
Tracking
()
RESOLVED
FIXED
152 Branch
| Tracking | Status | |
|---|---|---|
| firefox152 | --- | fixed |
People
(Reporter: calu, Assigned: giorga)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid][group4])
Attachments
(1 file)
Allow renaming a downloaded file to an uppercase version of the file name (ie: url.pdf -> url.PDF or URL.pdf or URL.PDF).
The issue is with to.exists() in processFileRenaming (source code]. Java File API exists() is not case sensitive. A potential solution would be to investigate a different API that would allow us to check if the destination file name exists.
Steps to Reproduce:
- Download a file named url.pdf
- From the download screen, rename, and type url.PDF
- Click Save
Actual result: An error says "A file named "url.PDF" already exists in this location
Expected result: Allow to rename
| Reporter | ||
Updated•3 months ago
|
Whiteboard: [fxdroid][group4]
Updated•3 months ago
|
| Assignee | ||
Updated•1 month ago
|
Assignee: nobody → giorga
| Assignee | ||
Comment 1•1 month ago
|
||
Updated•1 month ago
|
Attachment #9571592 -
Attachment is obsolete: true
Updated•1 month ago
|
Attachment #9571592 -
Attachment is obsolete: false
Updated•1 month ago
|
Attachment #9571592 -
Attachment description: Bug 2017179 - Align rename error handling for custom and default download folders during case-only changes → Bug 2017179 - Show a different message for case-only changes for the rename dialog.
Pushed by giorga@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/5b5b74015e51
https://hg.mozilla.org/integration/autoland/rev/b71c7215bd57
Show a different message for case-only changes for the rename dialog. r=android-reviewers,calu,android-l10n-reviewers,flod
Pushed by rperta@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/422cda50336c
https://hg.mozilla.org/integration/autoland/rev/b87209077584
Revert "Bug 2017179 - Show a different message for case-only changes for the rename dialog. r=android-reviewers,calu,android-l10n-reviewers,flod" for causing fenix failures
Backed out for causing fenix failures
Backout link
Push with failures
Failure log(s)
Flags: needinfo?(giorga)
Pushed by giorga@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/73c0bf2335ca
https://hg.mozilla.org/integration/autoland/rev/4cb31ed37c95
Show a different message for case-only changes for the rename dialog. r=android-reviewers,calu,android-l10n-reviewers,flod
| Assignee | ||
Updated•1 month ago
|
Flags: needinfo?(giorga)
Status: NEW → RESOLVED
Closed: 1 month ago
status-firefox152:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•