134296 - Code that determines default delete model needs to look at logon redirector

Status: VERIFIED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[David :Bienvenu]

We have some code that determines the default delete model for redirected logon
servers - that code was looking at some special capabilities returned from the
server, but it needs to look at the logon redirection type instead, because when
it's called, we haven't talked to the server yet. It's unfortunate the code has
to be here, but the new patch is a step in the right direction.

Comment 1

[David :Bienvenu]

Attachment: proposed fix

Comment 2

[David :Bienvenu]

Comment on attachment 76798 [details] [diff] [review]
proposed fix

this has r=naving, sr=sspitzer

Comment 3

[David :Bienvenu]

cc'ing naving, sspitzer.

Comment 4

[Asa Dotzler [:asa]]

Comment on attachment 76798 [details] [diff] [review]
proposed fix

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 5

[David :Bienvenu]

fixed

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

Verified in version 1.251 of nsIMAPIncomingServer.cpp

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=nsImapIncomingServer.cpp&root=/cvsroot&subdir=mozilla/mailnews/imap/src&command=DIFF_FRAMESET&rev1=1.250&rev2=1.251

Verified FIXED.  (I do hope there is a feature bug for this, probably in Bugscape?)