325589 - nsImapMailFolder::m_eventQueue bug when creating C++ accounts

Status: RESOLVED WORKSFORME

(Thunderbird :: General, defect)

Description

[Xavier]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0)
Build Identifier: 

In a C++ program, I launch Thunderbird in a new thread and I create IMAP accounts in the main thread using :

[...]
accountManager.CreateAccount()
[...]

Strangely I can create subdirectories only for the second account !


Reproducible: Always

Steps to Reproduce:



Expected Results:  
In fact, I can't create subdirectories in other accounts because the 'm_eventQueue' attribute of nsImapMailFolder class is null. I resolve this by replacing the use of 'm_eventQueue' (in nsImapMailFolder.cpp) with the call of a 'getEventQueue' method :

nsImapMailFolder::getEventQueue
{
    if (!m_eventQueue)
    {
        nsresult rv;
        nsCOMPtr<nsIEventQueueService> eqs = do_GetService(hEventQueueServiceCID, &rv);
        if (NS_SUCCEEDED(rv) && eqs) eqs->GetThreadEventQueue(NS_CURRENT_THREAD, getter_AddRefs(m_eventQueue));
    }
    return m_eventQueue;
}


I didn't try to see if the bug occurs when creating accounts with the Wizard...

I can't say if my correction doesn't have an impact somewhere else... but it doesn't seem.

Comment 1

[Wayne Mery (:wsmwk)]

who's doing accounts lately? :)

Comment 2

[Wayne Mery (:wsmwk)]

reporter's email address is dead.

Comment 3

[David :Bienvenu]

this isn't an account manager bug per se - it seems like the reporter was trying to embed all of the mailnews backend, and having issues with imap. m_eventQueue is no longer part of nsImapMailFolder, so I'm just going to mark this WFM. If someone else tries to do this, and encounters a similar problem, I'm sure they'll file a bug :-) And when we get tests working with a fake imap server, we'd probably also see any related issues.