Closed Bug 105760 Opened 23 years ago Closed 23 years ago

Subscribe UI unable to subscribe/unsubscribe to non-ASCII imap folders

Categories

(MailNews Core :: Internationalization, defect, P2)

Product:
MailNews Core
Component:
Internationalization
Platform:
x86
Windows 2000
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.9

People

(Reporter: ji, Assigned: Bienvenu)

Details

(Keywords: intl)

Attachments

(1 file)

proposed fix
1.54 KB, patch
naving
: review+
mscott
: superreview+
Details | Diff | Splinter Review
****Observed with 10-17-0.9.4 builds***** Uncheck on a non-ASCII imap folder on subscribe window doesn't stay. Steps to reproduce: 1. Have a non-ASCII folder in your imap account, like tëst. 2. Select File | Subscribe, you can see all the imap folders are subscribed initially. 3. Uncheck on the non-ASCII folder and click on OK. 4. Select File | Subscribe again to go back to the subscribe window, you can see the non-ASCII folder is still checked (subscribed). I don't see this problem with an ASCII folder.
Nominating for nsbeta1, since it's a basic functionality.
Keywords: intl, nsbeta1
Reassign to putterman. Is this a regression?
Assignee: nhotta → putterman
reassigning to sspitzer and cc'ing bienvenu. I don't know if this is a backend or frontend issue.
Assignee: putterman → sspitzer
No, it's reproducible with 6.1.
almost certainly an imap mod-utf7 or escaping problem.
Assignee: sspitzer → bienvenu
Keywords: nsbeta1 → nsbeta1+
Priority: -- → P2
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.9
Correcting summary - you can neither subscribe or unsubscribe to non-ascii folders, or folders whose mod-utf7 name is different from their display name.
Summary: Unable to unsubscrible non-ASCII imap folders → Subscribe UI unable to subscribe/unsubscribe to non-ASCII imap folders
Attached patch proposed fix — — Splinter Review
need to convert utf-7 encoded folder name back to unicode before calling subscribe routine.
cc'ing Navin for review
Comment on attachment 69117 [details] [diff] [review] proposed fix sr=mscott
Attachment #69117 - Flags: superreview+
Comment on attachment 69117 [details] [diff] [review] proposed fix r=naving
Attachment #69117 - Flags: review+
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Verified with 03/20 builds. it's fixed.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: