Tags mode is confused by tag with exclamation mark in it
Categories
(Thunderbird :: Folder and Message Lists, defect)
Tracking
(thunderbird128 fixed)
Tracking | Status | |
---|---|---|
thunderbird128 | --- | fixed |
People
(Reporter: darktrojan, Assigned: darktrojan)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Bug 1901926 - Fix folder pane tag folders for tags with special characters. r=#thunderbird-reviewers
48 bytes,
text/x-phabricator-request
|
corey
:
approval-comm-beta+
|
Details | Review |
I'm surprised I've not noticed this before, because I usually have a tag named "pink!" in my test profile.
The smart mailboxes code will go looking for a folder at mailbox://nobody@smart%20mailboxes/tags/pink!
but won't find it because the actual URL ends with pink%21
. Then it will try to create the folder pink!
and fail because it already exists. Folder names aren't converted to URLs using normal URL encoding, everything gets encoded except the alphanumeric characters. So we have to do that too when constructing URLs for the folder pane.
Assignee | ||
Comment 1•7 months ago
|
||
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/fe8f5d042ea0
Fix folder pane tag folders for tags with special characters. r=aleca
Assignee | ||
Updated•7 months ago
|
Assignee | ||
Comment 3•7 months ago
|
||
Comment on attachment 9406899 [details]
Bug 1901926 - Fix folder pane tag folders for tags with special characters. r=#thunderbird-reviewers
[Approval Request Comment]
Regression caused by (bug #): never worked
User impact if declined: the tags folder mode misses tags with special characters
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky):
Comment 4•7 months ago
|
||
Comment on attachment 9406899 [details]
Bug 1901926 - Fix folder pane tag folders for tags with special characters. r=#thunderbird-reviewers
[Triage Comment]
Approved for beta
Geoff, In the future I'd like to see all topics filled in for the approval request comment. I appreciate the tests you included in your commit! (Hopefully my exclamation point doesn't break anything here ;)
Comment 5•7 months ago
|
||
bugherder uplift |
Thunderbird 128.0b3:
https://hg.mozilla.org/releases/comm-beta/rev/c4be0ca54d0d
Description
•