78237 - Cyrus IMAP: Shouldn't display 2 Inbox mailboxes

Status: VERIFIED FIXED

(MailNews Core :: Networking: IMAP, defect, P3)

Description

[Karen Huang]

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.

Comment 1

[Karen Huang]

Attachment: Screen Shot for two Inbox mailboxes on Cyrus IMAP folder pane

Comment 2

[Karen Huang]

Attachment: Cyrus IMAP log for 043009 trunk build

Comment 3

[Karen Huang]

Ccing Navin & David.

Comment 4

[David :Bienvenu]

looks like a case-sensitivity problem somewhere with INBOX and Inbox

Comment 5

[Karen Huang]

Adding interop & nsbeta1 for the keywords since without this fix, there are 
problems on Cyrus Mail UI and sending messages.....

Comment 6

[scottputterman]

marking nsbeta1+ and moving to 0.9.2

Comment 7

[Scott MacGregor]

naving, mind if i load balance this to you? 

Comment 8

[Navin Gupta]

accepting. 

Comment 9

[Navin Gupta]

Attachment: proposed fix.

Comment 10

[Navin Gupta]

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 ?

Comment 11

[Navin Gupta]

request for review from david and scott.

Comment 12

[David :Bienvenu]

r=bienvenu - you should try new and existing profiles on Cyrus and non-cyrus
servers just to make sure everything still works.

Comment 13

[scottputterman]

Karen, what are the actual UI and sending problems that occur because of this?

Comment 14

[Navin Gupta]

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

[scottputterman]

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.

Comment 16

[scottputterman]

adding PDT+

Comment 17

[Navin Gupta]

I have tested messenger, cyrus UW old and new profiles. I will have to 
test courier. any other ? 

Comment 18

[Scott MacGregor]

sr=mscott

Comment 19

[Karen Huang]

MS Exchange? (poisonoak.mcom.com)

Comment 20

[Navin Gupta]

ok tested aol and MS exchange also. 

Comment 21

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 22

[Navin Gupta]

fix checked in. 

Comment 23

[Karen Huang]

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.