Closed Bug 6405 Opened 26 years ago Closed 26 years ago

Can only get msgs from one of multiple accounts listed

Categories

(MailNews Core :: Backend, defect, P1)

All
Other

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: esther, Assigned: mscott)

Details

Using build dated 19990513 (May 13th and before) if you have more than one account you can only get mail from the 1st account. 1. Set up preference for (2) pop accounts. 2. Launch Apprunner, then Messenger 3. Send a message to your 1st account (the one listed first in preference, which doesn't get listed first in Sidebar). 4. Get message. The message appears in the Inbox 5. Send a message to your 2nd account 6. Get message. The message does appear in the Inbox
QA Contact: 4080 → 4111
Priority: P3 → P1
Target Milestone: M6
this also applies to news. If you have a news and a pop account, only the first will work.
Scott/Scott - did either of you fix this already? I thought I remembered mail about this a few days ago.
I fixed local and I think mscott fixed Imap. I don't know if news is working.
I haven't fixed news yet. scotts: so having imap and pop at the same time works?
Assignee: alecf → mscott
News isn't working yet. get new messages for news still calls GetCurrentServer. I can change this after the tree opens (or Seth can). re-assigning to me or seth. Although nbaca is saying the problem occurrs for 2 pop accts. so there must still be a bug somewhere.
I haven't tried all of the combinations, but I made my local mail the 2nd account and it works. Scott wrote a function for nsMsgFolder.cpp called GetServer. It requires you to have implemented GetHostName. When GetNewMessages is called, instead of calling GetCurrentServer, you can call GetServer which will give you the server for the selected folder.
I checked in the local mail changes over the weekend so the first build they'd show up in is 5/17. This bug is on 5/13's build. Do we have more recent information that this isn't working?
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
ahh good catch on the date scott. This bug is already fixed (or at least we believe it is) for multiple pop accts. QA, can you try it out on todays or yesterdays builds? I'm going to mark this as fixed as Scott and I landed code to fix this problem over the weekend. However news, won't work unless it is the first server until Seth makes his change in: nsMsgNewsFolder::GetNewMessages() Seth, just replace GetCurrentServer with GetServer and you should be in business. If necessary, we can open a new bug on seth for the news aspect.
Using 1999051808 :This appears to be OK for profile with 2 POP accounts on Windows (NT 4.0). Will try other platforms.
As soon as the tree opens, I'll check in the necessary changes to make news & mail work at the same time.
Using 1999051709: Appears okay with 1 POP and 1 IMAP account on NT4.0. I used the 5/17 build since the 5/18 build doesn't have imap working.
my fix is checked in, so tomorrow you will be able to use pop, imap and news at the same time.
Using 1999051809 on Linux: - Appears okay with 2 POP. - Appears okay with 1 POP and 1 IMAP. On Esther's system she had 2 POP, 1 IMAP and 2 News accounts and was able to send and get messages from all mail accounts. She didn't try news since it is not enabled for this build.
this should be working for pop, imap and news, all at the same time on all platforms. has that been confirmed?
Using 1999052409M6 on Win32/NT4. Verified fixed on Win32. Configured for 2 POP, 2 IMAP and 1 News Server, sent one message to all accounts and was able to retrieve messages into all accounts, including a newsgroup on news.mozilla.org. I will try Linux and Mac scenarios next.
Status: RESOLVED → VERIFIED
Verified Fixed. Using 1999052408M6 on Linux and Mac. Configured for 1 POP, 1 IMAP and 1 News Server, sent one message to all accounts and was able to retrieve messages into all accounts, including a newsgroup on news.mozilla.org.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.