Closed
Bug 78237
Opened 24 years ago
Closed 24 years ago
Cyrus IMAP: Shouldn't display 2 Inbox mailboxes
Categories
(MailNews Core :: Networking: IMAP, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.2
People
(Reporter: huang, Assigned: naving)
References
Details
(Keywords: imap-interop, Whiteboard: [nsbeta1+][PDT+]Have fix)
Attachments
(3 files)
(deleted),
image/gif
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
Used 04-30-09-trunk build
Display 2 Inbox mailboxes from Cyrus IMAP mail
Setup Info:
Used a Cyrus IMAP migrated profile or create a new profile with "Inbox."
Personal namespace setting (from Advanced IMAP server setting of Mail Account
Settings)
Steps:
1) Login to above setup Cyrus IMAP account
2) After login to this mail account
3) Actual Results: it display 2 Inbox mailboxes on the folder pane.
expected results: Should just display only one Inbox on the folder pane.
Reporter | ||
Comment 1•24 years ago
|
||
Reporter | ||
Comment 2•24 years ago
|
||
Reporter | ||
Comment 3•24 years ago
|
||
Ccing Navin & David.
Summary: Cyrus IMAP: Shouldn't display 2 Inbox mailboxes → Cyrus IMAP: Shouldn't display 2 Inbox mailboxes
Comment 4•24 years ago
|
||
looks like a case-sensitivity problem somewhere with INBOX and Inbox
Reporter | ||
Comment 5•24 years ago
|
||
Adding interop & nsbeta1 for the keywords since without this fix, there are
problems on Cyrus Mail UI and sending messages.....
Comment 6•24 years ago
|
||
marking nsbeta1+ and moving to 0.9.2
Priority: -- → P3
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.2
Assignee | ||
Comment 8•24 years ago
|
||
accepting.
Assignee | ||
Comment 9•24 years ago
|
||
Assignee | ||
Comment 10•24 years ago
|
||
The fix is to detect if we get anything other than INBOX from the server
like inbox, Inbox etc. Delete INBOX and all its children and create
Inbox and its children, true for cyrus imap servers. I have added a param
caseInsensitive to know when to do strcasecmp (like for INBOX) and when to do
strcmp (for other imap folders).
The changes in nsImapProtocol are to detect if the there is a namespace prefix
something like Inbox/inbox etc then to send the list command as Inbox/inbox.
However, sending INBOX would also work but I think previous approach is better.
I need review, david ?
Assignee | ||
Comment 11•24 years ago
|
||
request for review from david and scott.
Assignee | ||
Updated•24 years ago
|
Whiteboard: [nsbeta1+] → [nsbeta1+]Have fix
Comment 12•24 years ago
|
||
r=bienvenu - you should try new and existing profiles on Cyrus and non-cyrus
servers just to make sure everything still works.
Comment 13•24 years ago
|
||
Karen, what are the actual UI and sending problems that occur because of this?
Assignee | ||
Comment 14•24 years ago
|
||
Well from the UI perspective the user would see two inbox and may be confused.
I think you meant receiving instead of sending; there are no receiving problems.
Comment 15•24 years ago
|
||
Karen has a comment towards the beginning of bug that says "there are
problems on Cyrus Mail UI and sending messages". I agree that there are UI
problems I just wanted to know what other kinds of problems we could expect if
this bug didn't get fixed.
Assignee | ||
Comment 17•24 years ago
|
||
I have tested messenger, cyrus UW old and new profiles. I will have to
test courier. any other ?
Comment 18•24 years ago
|
||
sr=mscott
Reporter | ||
Comment 19•24 years ago
|
||
MS Exchange? (poisonoak.mcom.com)
Assignee | ||
Comment 20•24 years ago
|
||
ok tested aol and MS exchange also.
Comment 21•24 years ago
|
||
a= asa@mozilla.org for checkin to the trunk.
(on behalf of drivers)
Blocks: 83989
Assignee | ||
Comment 22•24 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 23•24 years ago
|
||
Verified on WinNT 06-18-09-trunk build
Now, Cyrus IMAP is not displaying 2 Inbox now, marking as verified for this bug
but found bug 86459 & bug 86467 for Inbox UI & Inbox subfolders creation
problems.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•