140477 - Mdn/Offline: Sending Return Receipt sent while offline doesn't

Status: NEW

(MailNews Core :: Backend, defect)

Description

[grylchan]

Using commerical branch
2002042606 Nt 4.0, 2002-04-26-05-1.00 Mac 10.1.3
& commercial trunk:
2002-04-26-10 linux 2.2

Discovered this testing offline.

If you have the following offline pref set:
'Make the messages in my Inbox available when I am working offline'
and you receive a mesg, in your inbox, that has a Return Receipt
request on it, you will not send a Return Receipt even though
you have the pref set to 'ask me' or 'always send'.

MDN works perfectly fine if you uncheck the offline pref.

No problems with filtered folders or folders 'selected for offline usage'.
Only prob is with that one pref & inbox.

Steps to reproduce:
1.Set up a imap mail acnt
2.Set either Global or Customized MDN:
   Check the box 'When sending mesgs, always request a Return Receipt'
   Allow return receipts for some mesgs
    -in all other cases: ask me or always send
3.Mail&Newsgroup Accnt Settings|Offline&DiskSpace and check:
 'Make the messages in my Inbox available when I am working offline'
4.Compose a mesg to yourself
5.Verify in Compose Menu: Options|Return Receipt is checked
6.Send the mesg
7.Get the mesg
8.Click on the new mesg

result: No prompt asking you to send RR or auto send of RR
expected: prompt or auto send of RR

I'm guessing the problem is we are downloading mesg while reading it
and there by confusing the code to generate RR prompt or Auto send of RR.

Comment 1

[grylchan]

adding nsbeta1

Comment 2

[Jeff Tsai]

I believe if we check in the fix for bug 138054 this will be fixed too.

Comment 3

[David :Bienvenu]

yep, the imap mdn code is not called in this case - I think it's just because
the mdn code is in the wrong place, or needs to be called in another place as
well (the same place where we mark the message read in the db in this situation
should invoke the mdn code too).

Comment 4

[David :Bienvenu]

nsImapMailFolder::StoreImapFlags(nsImapMailFolder * const 0x04048094, int
0x00000001, int 0x00000001, unsigned int * 0x094c0270, int 0x00000001) line 2978
nsImapMailFolder::MarkMessagesRead(nsImapMailFolder * const 0x04047fec,
nsISupportsArray * 0x0906cc20, int 0x00000001) line 1624 + 48 bytes
nsImapUrl::SetMsgLoadingFromCache(nsImapUrl * const 0x079d3e70, int 0x00000001)
line 1279
nsImapMockChannel::ReadFromLocalCache() line 7517
nsImapMockChannel::AsyncOpen(nsImapMockChannel * const 0x07d5d8e0,
nsIStreamListener * 0x079b92f8, nsISupports * 0x00000000) line 7547 + 8 bytes

this is where we mark this message read (note that
nsImapMailFolder::NormalEndMsgWriteStream is not called with markRead set to
true in this case, which is why the mdn code is not called). My suggestion is
that we have a method in nsImapMailFolder that does the mdn stuff, and call it
from nsImapUrl::SetMsgLoadingFromCache. Perhaps it could be a method of
nsIImapMailFolder.

Comment 5

[Michael Buckland]

Discussed at mail news bug meeting.  Decided to remove RTM from this bug.  This
means fixing this bug is not currently planned for RTM.

Comment 6

[Wayne Mery (:wsmwk)]

(In reply to comment #4)
> 
> this is where we mark this message read (note that
> nsImapMailFolder::NormalEndMsgWriteStream is not called with markRead set to
> true in this case, which is why the mdn code is not called). My suggestion is
> that we have a method in nsImapMailFolder that does the mdn stuff, and call it
> from nsImapUrl::SetMsgLoadingFromCache. Perhaps it could be a method of
> nsIImapMailFolder.

Comment 8

[Magnus Melin [:mkmelin]]

[updating summary, and i can confirm it still happens on linux/trunk]

Comment 9

[Wayne Mery (:wsmwk)]

TCW can you reproduce this?

Comment 10

[Arthur K. (he/him)]

Set either Global or Customized MDN

A what?