Open Bug 63039 Opened 24 years ago Updated 3 years ago

IMAP subscribe UI needs to unescape folder names on servers whose hiearchy delimiter is not '/'

Categories

(MailNews Core :: Networking: IMAP, defect)

x86
All
defect

Tracking

(Not tracked)

People

(Reporter: Bienvenu, Unassigned)

Details

I've changed the way we handle folder names on servers whose hiearchy delimiter is not '/': if there's a slash in the name, I replace it with '^', and I replace '^' with '^^'. Then, when sending the name to the server, I replace '^' with '/' and "^^" with '^' - this is done by nsImapUrl::EscapeSlashes and nsImapUrl::UnescapeSlashes. It looks like the IMAP subscribe UI is displaying the escaped name, and not the pretty name for folders. I need to figure out what its doing and fix it.
accepting
Status: NEW → ASSIGNED
Priority: -- → P2
QA Contact: esther → huang
Cavin has become expert in this code :-)
Assignee: bienvenu → cavin
Status: ASSIGNED → NEW
Just a note for later (while I can still member): OK, we show "a^b" in the subscribe dialog (where in folder pane we show "a/b". But if we were to change the code to ask the front-end UI to display it as "a/b" then it'll end up showing folder "a" with a sub-folder called "b" (ie, there will be a twister associated with "a"). I think this is the way the front-end figures out the folder hierarchy and it means that we probably can't pass "a/b" to front-end either.
Cavin, please see bug 29926 the last comments that I put.
Cavin, do you think that this is fixed along with the fix from bug 94305?
No, Karen, I think this is a separate problem.
QA Contact: huang → gchan
Product: MailNews → Core
Product: Core → MailNews Core
Assignee: cavin → nobody
QA Contact: grylchan → networking.imap
Severity: normal → minor
OS: Windows NT → All
Priority: P2 → --
Is this still relevant? Does anybody know of such a server to test?
Flags: needinfo?(gds)
I have a test account that, IIRC, has dot (.) as the delimiter, not slash (/). But if the user wanted to make a folder with / in the name, I guess it's possible but unusual. I would vote to just ignore this unless someone actually complains.
Flags: needinfo?(gds)
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.