Closed Bug 3765 Opened 26 years ago Closed 26 years ago

Mar150940 build: Thread pane does not update after GetNewMsg

Categories

(SeaMonkey :: MailNews: Message Display, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: lchiang, Assigned: Bienvenu)

Details

Mar150940 build: GetNewMail() does not work 1) seamonkey.zip file from Mar150940 build. 2) Install 3) Start apprunner with mailshell.xul file 4) Mail window opens (On another system, send the account you're using a test message) 5) Click Get Msg button 6) Nothing happens. New mail does not appear Note: In the DOS window, it shows "nsMsgAppCore :: GetNewMail() Showed this to mscott already.
QA Contact: 4080 → 4114
Target Milestone: M3
This functionality is supposed to be in Dogfood. Set target milestone to M3. cc: phil.
Assignee: mscott → putterman
Component: Networking → Front End
Summary: Mar150940 build: GetNewMail() does not work → Mar150940 build: Thread pane does not update after GetNewMsg
Update: after further investigation with mscott, we determined that GetNewMsg works, but the thread pane is not updated. Add the following steps to my original report: 7) Exit mail/apprunner 8) View the Inbox file using some text editor. Notice that the new mail message is listed in the Inbox file. Per mscott, reassign this to putterman.
Assignee: putterman → bienvenu
Reassigning to bienvenu. Scott M and I looked at this and it appears that nsPop3Sink is not telling the mail parser that the message being read in is finished. When I receive more than one message at a time this is working, but when I don't it doesn't. We found that by putting a call to m_newMailParser->OnStopBinding(nsnull, NS_OK, nsnull); before destroying m_newMailParser in nsPop3Sink::EndMailDelivery(), the problem goes away.
But I didn't do the pop download code! Fine. I'll fix it.
Sorry I told Scott to assign it to you. I thought you had integrated the pop3 sink with the mailbox parser. Feel free to assign it to me if you want me to check it in. Fix seems straightforward..
waiting for approval to check in from chofmann
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
fixed as suggested by Scottx2
Status: RESOLVED → VERIFIED
Verified Marc160805 seamonkey.zip release build. Verified by sending myself one simple text message and doing a Get Msg. Verified by sending myself more than one simple text message and doing a Get Msg. In both cases, the new messages appeared in dogfood mail's thread pane. (Now, the ordering of the messages in the thread pane sometimes doesn't make sense, but that's a different item than this bug.)
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.