Dalen Kruse Reporter
	Description • 22 years ago

When importing an address book from Outlook, the addresses for mailing lists are
not imported.  An entry is created for the list, but no e-mail addresses for the
list is displayed.

If requested, I can send the Outlook address book file.

	Ninoschka Baca
	Comment 1 • 22 years ago

Branch build 2002-07-01: WinMe
Reproduced the problem. 
Marking nsbeta1 so this can be fixed for this release.

	Cavin Song Assignee
	Comment 2 • 22 years ago

Yes, we need to add code to extract members of the lists from MAPI in order to 
fix the problem (see nsOutlookMail::CreateList()).

	Ninoschka Baca
	Comment 3 • 22 years ago

Cavin, bug# 82791/comment# 9 states "...Members of lists are not imported due to 
not being able to retrieve the member values." This is the same issue, right?

	Cavin Song Assignee
	Comment 4 • 22 years ago

Right. So it's good that we have this bug to keep track of the member issue.

	Samir Gehani
	Comment 5 • 22 years ago

Mail triage team: nsbeta1-

	Daniel Wang
	Comment 6 • 22 years ago

*** Bug 187095 has been marked as a duplicate of this bug. ***

	Jason M. Abels
	Comment 7 • 21 years ago

*** Bug 148801 has been marked as a duplicate of this bug. ***

	Jason M. Abels
	Comment 8 • 21 years ago

*** Bug 205663 has been marked as a duplicate of this bug. ***

	Jason M. Abels
	Comment 9 • 21 years ago

*** Bug 200469 has been marked as a duplicate of this bug. ***

	wind li
	Comment 10 • 21 years ago

Add myself to the cc list.I am working on OpenOffice Address Book Integration
Project.And meet the same problem.

	wind li
	Comment 11 • 21 years ago

This is a work through patch.I test it with mozilla 1.5b and outlook 2002.

	wind li
	Comment 12 • 21 years ago

[tracking] StarOffice/OpenOffice AddressBook integration

	wind li
	Comment 13 • 21 years ago

Comment on attachment 131429 [details] [diff] [review]
patch for 1.5b to add outlook mailist support

any one review it?

	wind li
	Comment 14 • 21 years ago

Comment on attachment 131429 [details] [diff] [review]
patch for 1.5b to add outlook mailist support

David can you review this?

	David :Bienvenu
	Comment 15 • 21 years ago

Comment on attachment 131429 [details] [diff] [review]
patch for 1.5b to add outlook mailist support

+	[noscript] void AddListCardColumnsToRow(in nsIAbCard pCard, in
nsIMdbRow pListRow, in PRUint32 pos,out nsIAbCard pNewCard, in PRBool
aInMailingList);
+	[noscript] void GetCardFromDB(in nsIAbCard newCard, in nsIMdbRow
cardRow);
+	[noscript] void SetListAddressTotal(in nsIMdbRow listRow, in PRUint32
total);
+	[noscript] boolean FindRow(in nsIAbCard card, out nsIMdbRow oldRow);
why are these [noscript]? they could be scriptable, right? You should also use
idl types when possible, so  unsigned long instead of PRUint32, boolean instead
of PRBool.

also, all the argument names should begin with a (for argument), so it would be
"in nsIAbCard aPCard".

FindRow could return an nsIMdbRow instead of boolean, and return a null row in
case it couldn't find the row. And, why is the arg called "oldRow"? It looks
like it's the returned row, not "oldRow". Also, there seem to be a lot of tabs
in the patch - those need to be replaced with two space indents.

you can use nsCOMPtr's here:
					nsIMdbRow* newRow = nsnull;
+					nsIMdbRow* oldRow = nsnull;
. That simplifies the ref-counting code.

The GetCardFromDB method is not named correctly (I realize you didn't name it,
but you're making it a public interface, so it's more important that its name
reflect what it does) I suggest something like InitCardFromRow.

Fix these things and I'll review a new patch as quickly as I can. Thx for
working on this!

	wind li
	Comment 17 • 21 years ago

->why are these [noscript]? they could be scriptable, right? 
No.The whole interface is definded noscript.
->The GetCardFromDB method is not named correctly 
GetCardFromDB is the origin name of this function.And a nsIMdbRow is a db row.So
I think matbe GetCardFromDB.
OK?

	wind li
	Comment 18 • 21 years ago

Comment on attachment 133684 [details] [diff] [review]
updated

>Index: mailnews/addrbook/public/nsIAddrDatabase.idl
>===================================================================
>RCS file: /cvsroot/mozilla/mailnews/addrbook/public/nsIAddrDatabase.idl,v
>retrieving revision 1.37
>diff -u -w -b -i -r1.37 nsIAddrDatabase.idl
>--- mailnews/addrbook/public/nsIAddrDatabase.idl	18 Mar 2003 18:39:32 -0000	1.37
>+++ mailnews/addrbook/public/nsIAddrDatabase.idl	20 Oct 2003 13:21:10 -0000
>@@ -257,4 +257,14 @@
>      *        The column value (example: jabroni316)
>      */
>     [noscript] void addRowValue(in nsIMdbRow aRow, in ACString aLDIFAttributeName, in AString aColValue);
>+
>+    [noscript] void AddListCardColumnsToRow(in nsIAbCard aPCard,
>+                                            in nsIMdbRow aPListRow,
>+                                            in unsigned long aPos,
>+                                            out nsIAbCard aPNewCard,
>+                                            in boolean aInMailingList);
>+    [noscript] void GetCardFromDB(in nsIAbCard aNewCard,in nsIMdbRow aCardRow);
>+    [noscript] void SetListAddressTotal(in nsIMdbRow aListRow, in PRUint32 aTotal);
>+    [noscript] nsIMdbRow FindRow(in nsIAbCard aCard);
>+
> };
>Index: mailnews/addrbook/src/nsAddrDatabase.cpp
>===================================================================
>RCS file: /cvsroot/mozilla/mailnews/addrbook/src/nsAddrDatabase.cpp,v
>retrieving revision 1.116
>diff -u -w -b -i -r1.116 nsAddrDatabase.cpp
>--- mailnews/addrbook/src/nsAddrDatabase.cpp	23 May 2003 21:33:47 -0000	1.116
>+++ mailnews/addrbook/src/nsAddrDatabase.cpp	20 Oct 2003 13:22:43 -0000
>@@ -1618,7 +1618,7 @@
>     return rv;
> }
> 
>-nsresult nsAddrDatabase::AddListCardColumnsToRow
>+NS_IMETHODIMP nsAddrDatabase::AddListCardColumnsToRow
> (nsIAbCard *pCard, nsIMdbRow *pListRow, PRUint32 pos, nsIAbCard** pNewCard, PRBool aInMailingList)
> {
>   if (!pCard && !pListRow )
>@@ -1782,9 +1782,20 @@
>     return count;
> }
> 
>-nsresult nsAddrDatabase::SetListAddressTotal(nsIMdbRow* listRow, PRUint32 total)
>+NS_IMETHODIMP nsAddrDatabase::SetListAddressTotal(nsIMdbRow* aListRow, PRUint32 aTotal)
> {
>-    return AddIntColumn(listRow, m_ListTotalColumnToken, total);
>+    return AddIntColumn(aListRow, m_ListTotalColumnToken, aTotal);
>+}
>+
>+NS_IMETHODIMP nsAddrDatabase::FindRow(nsIAbCard * aCard,nsIMdbRow **_retval)
>+{
>+    PRUnichar *aPrimaryEmail;
>+    nsIMdbRow *newRow=nsnull;
>+    aCard->GetPrimaryEmail(&aPrimaryEmail);
>+    nsresult rv=GetRowForCharColumn (aPrimaryEmail,m_PriEmailColumnToken,PR_TRUE,&newRow);
>+
>+    *_retval=newRow;
>+    return rv;
> }
> 
> 
>@@ -2607,7 +2618,7 @@
>   return rv;
> }
> 
>-nsresult nsAddrDatabase::GetCardFromDB(nsIAbCard *newCard, nsIMdbRow* cardRow)
>+NS_IMETHODIMP nsAddrDatabase::GetCardFromDB(nsIAbCard *newCard, nsIMdbRow* cardRow)
> {
>     nsresult    err = NS_OK;
>     if (!newCard || !cardRow)
>Index: mailnews/addrbooksrc/nsAddrDatabase.h
>===================================================================
>RCS file: /cvsroot/mozilla/mailnews/addrbook/src/nsAddrDatabase.h,v
>retrieving revision 1.44
>diff -u -w -b -i -r1.44 nsAddrDatabase.h
>--- mailnews/addrbook/src/nsAddrDatabase.h	25 Apr 2003 03:05:12 -0000	1.44
>+++ mailnews/addrbook/src/nsAddrDatabase.h	20 Oct 2003 13:22:47 -0000
>@@ -322,6 +322,12 @@
> 
>   static void PRTime2Seconds(PRTime prTime, PRUint32 *seconds);
>   
>+
>+    NS_IMETHOD AddListCardColumnsToRow(nsIAbCard *aPCard, nsIMdbRow *aPListRow, PRUint32 aPos, nsIAbCard** aPNewCard, PRBool aInMailingList);
>+    NS_IMETHOD GetCardFromDB(nsIAbCard *aNewCard, nsIMdbRow* aCardRow);
>+    NS_IMETHOD SetListAddressTotal(nsIMdbRow* aListRow, PRUint32 aTotal);
>+    NS_IMETHOD FindRow(nsIAbCard * card,nsIMdbRow **_retval);
>+
> protected:
>   
>   static void		AddToCache(nsAddrDatabase* pAddrDB) {GetDBCache()->AppendElement(pAddrDB);}
>@@ -346,16 +352,13 @@
> 	nsresult GetIntColumn(nsIMdbRow *cardRow, mdb_token outToken, 
> 							PRUint32* pValue, PRUint32 defaultValue);
> 	nsresult GetBoolColumn(nsIMdbRow *cardRow, mdb_token outToken, PRBool* pValue);
>-	nsresult GetCardFromDB(nsIAbCard *newCard, nsIMdbRow* cardRow);
> 	nsresult GetListCardFromDB(nsIAbCard *listCard, nsIMdbRow* listRow);
> 	nsresult GetListFromDB(nsIAbDirectory *newCard, nsIMdbRow* listRow);
> 	nsresult AddRecordKeyColumnToRow(nsIMdbRow *pRow);
> 	nsresult AddAttributeColumnsToRow(nsIAbCard *card, nsIMdbRow *cardRow);
>-	nsresult AddListCardColumnsToRow(nsIAbCard *pCard, nsIMdbRow *pListRow, PRUint32 pos, nsIAbCard** pNewCard, PRBool aInMailingList);
> 	nsresult AddListAttributeColumnsToRow(nsIAbDirectory *list, nsIMdbRow *listRow);
> 	nsresult CreateCard(nsIMdbRow* cardRow, mdb_id listRowID, nsIAbCard **result);
> 	nsresult CreateCardFromDeletedCardsTable(nsIMdbRow* cardRow, mdb_id listRowID, nsIAbCard **result);
>-	nsresult SetListAddressTotal(nsIMdbRow* listRow, PRUint32 total);
> 	nsresult DeleteCardFromListRow(nsIMdbRow* pListRow, mdb_id cardRowID);
> 	void DeleteCardFromAllMailLists(mdb_id cardRowID);
> 	nsresult NotifyListEntryChange(PRUint32 abCode, nsIAbDirectory *dir, nsIAddrDBListener *instigator);
>Index: mailnews/import/outlook/src/MapiApi.cpp
>===================================================================
>RCS file: /cvsroot/mozilla/mailnews/import/outlook/src/MapiApi.cpp,v
>retrieving revision 1.23
>diff -u -w -b -i -r1.23 MapiApi.cpp
>--- mailnews/import/outlook/src/MapiApi.cpp	19 Feb 2002 22:49:08 -0000	1.23
>+++ mailnews/import/outlook/src/MapiApi.cpp	20 Oct 2003 13:24:47 -0000
>@@ -505,13 +505,12 @@
> {	
> 	ULONG		ulObjType;
> 	HRESULT		hr;
>-	hr =	lpMdb->OpenEntry(	cbEntry,
>+    hr = m_lpSession->OpenEntry(cbEntry,
> 								pEntryId,
> 								NULL,
> 								0,
> 								&ulObjType,
> 								(LPUNKNOWN *) ppOpen);
>-
> 	if (FAILED(hr)) {
> 		MAPI_TRACE2( "OpenMdbEntry failed: 0x%lx, %d\n", (long)hr, (int)hr);
> 		return( FALSE);
>Index: mailnews/import/outlook/src/nsOutlookMail.cpp
>===================================================================
>RCS file: /cvsroot/mozilla/mailnews/import/outlook/src/nsOutlookMail.cpp,v
>retrieving revision 1.29
>diff -u -w -b -i -r1.29 nsOutlookMail.cpp
>--- mailnews/import/outlook/src/nsOutlookMail.cpp	6 Nov 2002 02:58:04 -0000	1.29
>+++ mailnews/import/outlook/src/nsOutlookMail.cpp	20 Oct 2003 13:25:06 -0000
>@@ -1005,7 +1005,7 @@
>           pVal = m_mapi.GetMapiProperty( lpMsg, PR_SUBJECT);
>           if (pVal)
>             m_mapi.GetStringFromProp( pVal, subject);
>-          CreateList(subject.get(), pDb);
>+          CreateList(subject.get(), pDb, lpMsg, pFieldMap);
>         }
> 			}			
> 
>@@ -1017,8 +1017,10 @@
>   rv = pDb->Commit(nsAddrDBCommitType::kLargeCommit);
> 	return rv;
> }
>-
>-nsresult nsOutlookMail::CreateList( const PRUnichar * pName, nsIAddrDatabase *pDb)
>+nsresult nsOutlookMail::CreateList( const PRUnichar * pName,
>+                                   nsIAddrDatabase *pDb,
>+                                   LPMAPIPROP pUserList,
>+                                   nsIImportFieldMap *pFieldMap)
> {
>   // If no name provided then we're done.
>   if (!pName || !(*pName))
>@@ -1029,17 +1031,105 @@
>   if (!pDb)
>     return rv;
> 
>-  nsCOMPtr <nsIMdbRow> newRow;
>-  rv = pDb->GetNewListRow(getter_AddRefs(newRow));
>+  nsCOMPtr <nsIMdbRow> newListRow;
>+  rv = pDb->GetNewListRow(getter_AddRefs(newListRow));
>+  NS_ENSURE_SUCCESS(rv, rv);
>+  nsCAutoString column;
>+  column.AssignWithConversion(pName);
>+  rv = pDb->AddListName(newListRow, column.get());
>   NS_ENSURE_SUCCESS(rv, rv);
> 
>-  rv = pDb->AddListName(newRow, NS_ConvertUCS2toUTF8(pName).get());
>+    HRESULT             hr;
>+    LPSPropValue aValue = NULL ;
>+    ULONG aValueCount = 0 ;
>+
>+    LPSPropTagArray properties = NULL ;
>+    m_mapi.MAPIAllocateBuffer(CbNewSPropTagArray(1),
>+                       (void **)&properties) ;
>+    properties->cValues = 1;
>+    properties->aulPropTag [0] = m_mapi.GetEmailPropertyTag(pUserList, 0x8054);
>+    hr = pUserList->GetProps(properties, 0, &aValueCount, &aValue) ;
>+
>+    SBinaryArray *sa=(SBinaryArray *)&aValue->Value.bin;
>+
>+    LPENTRYID    lpEid;
>+    ULONG        cbEid;
>+    PRInt32        idx;
>+    LPMESSAGE        lpMsg;
>+    nsCString        type;
>+    LPSPropValue    pVal;
>+    nsString        subject;
>+    PRUint32 total;
>+    
>+
>+    total=sa->cValues;
>+    for (idx=0;idx<sa->cValues ;idx++)
>+    {
>+        lpEid= (LPENTRYID) sa->lpbin[idx].lpb;
>+        cbEid = sa->lpbin[idx].cb;
>+
>+
>+        if (!m_mapi.OpenEntry(cbEid, lpEid, (LPUNKNOWN *) &lpMsg))
>+        {
>+            
>+            IMPORT_LOG1( "*** Error opening messages in mailbox: %S\n", pName);
>+            return( NS_ERROR_FAILURE);
>+        }
>+        {
>+            {
>+                // This is a contact, add it to the address book!
>+                subject.Truncate( 0);
>+                pVal = m_mapi.GetMapiProperty( lpMsg, PR_SUBJECT);
>+                if (pVal)
>+                    m_mapi.GetStringFromProp( pVal, subject);
>+                
>+                    nsIMdbRow* newRow = nsnull;
>+                    nsIMdbRow* oldRow = nsnull;
>+                    pDb->GetNewRow( &newRow);
>+                    if (newRow) {
>+                        if (BuildCard( subject.get(), pDb, newRow, lpMsg, pFieldMap)) 
>+                        {
>+                            nsCOMPtr <nsIAbCard> userCard;
>+                            nsCOMPtr <nsIAbCard> newCard;
>+                            userCard = do_CreateInstance(NS_ABMDBCARD_CONTRACTID, &rv);
>   NS_ENSURE_SUCCESS(rv, rv);
>-  rv = pDb->AddCardRowToDB(newRow);
>+                            pDb->GetCardFromDB(userCard,newRow);
>+                            
>+                            //add card to db
>+                            PRBool bl=PR_FALSE;
>+                            pDb->FindRow(userCard,&oldRow);
>+                            if (oldRow)
>+                            {
>+                                NS_RELEASE(newRow);
>+                                newRow = oldRow;
>+                            }
>+                            else
>+                            {
>+                                pDb->AddCardRowToDB( newRow);
>+                            }
>+                            
>+                            //add card list
>+                            pDb->AddListCardColumnsToRow(userCard,
>+                                              newListRow,idx+1,
>+                                              getter_AddRefs(newCard),PR_TRUE);
>+
>+                            NS_RELEASE(newRow);
>+                        }
>+                    }
>+
>+
>+            }
>+        }
>+    }
>+  
>+  rv = pDb->AddCardRowToDB(newListRow);
>   NS_ENSURE_SUCCESS(rv, rv);
>-  rv = pDb->AddListDirNode(newRow);
>+
>+  rv = pDb->SetListAddressTotal(newListRow, total);
>+  rv = pDb->AddListDirNode(newListRow);
>   return rv;
> }
>+
> 
> void nsOutlookMail::SanitizeValue( nsString& val)
> {
>Index: mailnews/import/outlook/src/nsOutlookMail.h
>===================================================================
>RCS file: /cvsroot/mozilla/mailnews/import/outlook/src/nsOutlookMail.h,v
>retrieving revision 1.11
>diff -u -w -b -i -r1.11 nsOutlookMail.h
>--- mailnews/import/outlook/src/nsOutlookMail.h	6 Nov 2002 02:58:05 -0000	1.11
>+++ mailnews/import/outlook/src/nsOutlookMail.h	20 Oct 2003 13:25:06 -0000
>@@ -78,7 +78,7 @@
> 	void		SanitizeValue( nsString& val);
> 	void		SplitString( nsString& val1, nsString& val2);
> 	PRBool		BuildCard( const PRUnichar *pName, nsIAddrDatabase *pDb, nsIMdbRow *newRow, LPMAPIPROP pUser, nsIImportFieldMap *pFieldMap);
>-  nsresult  CreateList( const PRUnichar * pName, nsIAddrDatabase *pDb);
>+  nsresult  CreateList( const PRUnichar * pName, nsIAddrDatabase *pDb, LPMAPIPROP pUserList, nsIImportFieldMap *pFieldMap);
>   void SetDefaultContentType(CMapiMessage &msg, nsCString &cType);
> 
> private:

	wind li
	Comment 20 • 21 years ago

Comment on attachment 133720 [details] [diff] [review]
update patch to trunk

->why are these [noscript]? they could be scriptable, right? 
No.The whole interface is definded noscript.
->The GetCardFromDB method is not named correctly 
GetCardFromDB is the origin name of this function.And a nsIMdbRow is a db
row.So
I think matbe GetCardFromDB.
OK?

	David :Bienvenu
	Comment 21 • 21 years ago

As I said before, "the GetCardFromDB method is not named correctly (I realize
you didn't name it, but you're making it a public interface, so it's more
important that its name reflect what it does) I suggest something like
InitCardFromRow."

>No.The whole interface is definded noscript.

Not sure what you mean - there are many methods in nsIAddrDatabase.idl that are
scriptable. Your new ones should also be scriptable.

	David :Bienvenu
	Comment 23 • 21 years ago

Thx for fixing those things. Here are the rest of my comments:

You can use nsCOMPtrs in this code - I think I didn't give you enough context in
my previous comment about this. Sorry about that. So, this code:

+                    nsIMdbRow* newRow = nsnull;
+                    nsIMdbRow* oldRow = nsnull;
+                    pDb->GetNewRow( &newRow);
+                    if (newRow) {
+                        if (BuildCard( subject.get(), pDb, newRow, lpMsg,
pFieldMap)) 
+                        {
+                            nsCOMPtr <nsIAbCard> userCard;
+                            nsCOMPtr <nsIAbCard> newCard;
+                            userCard =
do_CreateInstance(NS_ABMDBCARD_CONTRACTID, &rv);
   NS_ENSURE_SUCCESS(rv, rv);
-  rv = pDb->AddCardRowToDB(newRow);
+                            pDb->InitCardFromRow(userCard,newRow);
+                            
+                            //add card to db
+                            PRBool bl=PR_FALSE;
+                            pDb->FindRow(userCard,&oldRow);
+                            if (oldRow)
+                            {
+                                NS_RELEASE(newRow);
+                                newRow = oldRow;
+                            }
+                            else
+                            {
+                                pDb->AddCardRowToDB( newRow);
+                            }
+                            
+                            //add card list
+                            pDb->AddListCardColumnsToRow(userCard,
+                                              newListRow,idx+1,
+                                              getter_AddRefs(newCard),PR_TRUE);
+
+                            NS_RELEASE(newRow);
+                        }

can look like this code below instead.  However, I'm confused by this code.
After CreateInstance of userCard, you add the new row to the db with ths line:

  rv = pDb->AddCardRowToDB(newRow);

then, later, if you don't find the row, you add the new row to the db again. Am
I misunderstanding something?

                           pDb->FindRow(userCard,getter_AddRefs(oldRow));
                            if (oldRow)
                                newRow = oldRow;
                            else
                                pDb->AddCardRowToDB( newRow);

So, here's what the code would look like with comptrs.

                    nsCOMPtr <nsIMdbRow> newRow;
                    nsCOMPtr <nsIMdbRow> oldRow;
                    pDb->GetNewRow( getter_AddRefs(newRow));
                    if (newRow) {
                        if (BuildCard( subject.get(), pDb, newRow, lpMsg,
pFieldMap)) 
                        {
                            nsCOMPtr <nsIAbCard> userCard;
                            nsCOMPtr <nsIAbCard> newCard;
                            userCard =
do_CreateInstance(NS_ABMDBCARD_CONTRACTID, &rv);
   NS_ENSURE_SUCCESS(rv, rv);
  rv = pDb->AddCardRowToDB(newRow);
                            pDb->InitCardFromRow(userCard,newRow);
                            
                            //add card to db
                            PRBool bl=PR_FALSE;
                            pDb->FindRow(userCard,getter_AddRefs(oldRow));
                            if (oldRow)
                                newRow = oldRow;
                            else
                                pDb->AddCardRowToDB( newRow);
                            
                            //add card list
                            pDb->AddListCardColumnsToRow(userCard,
                                              newListRow,idx+1,
                                              getter_AddRefs(newCard),PR_TRUE);

                        }

this code doesn't follow the argument naming standard for the arg, but uses it a
for a local var. and it also looks like it leaks the primary e-mail string.
+NS_IMETHODIMP nsAddrDatabase::FindRow(nsIAbCard * aCard,nsIMdbRow **_retval)
+{
+    PRUnichar *aPrimaryEmail;
+    nsIMdbRow *newRow=nsnull;
+    aCard->GetPrimaryEmail(&aPrimaryEmail);
+    nsresult rv=GetRowForCharColumn
(aPrimaryEmail,m_PriEmailColumnToken,PR_TRUE,&newRow);
+
+    *_retval=newRow;
+    return rv;
 }
 

so, this code should look more like this:

NS_IMETHODIMP nsAddrDatabase::FindRow(nsIAbCard * aCard,nsIMdbRow **aRow)
{
    nsXPIDLString primaryEmail;
    aCard->GetPrimaryEmail(getter_Copies(primaryEmail));
    return GetRowForCharColumn(aPrimaryEmail, m_PriEmailColumnToken, PR_TRUE, aRow);
}
 
+    NS_IMETHOD FindRow(nsIAbCard * card,nsIMdbRow **_retval);

should be:
+    NS_IMETHOD FindRow(nsIAbCard * card,nsIMdbRow **aRow);


Again, thx for doing this.

	wind li
	Comment 24 • 21 years ago

David , thanks for your particular comments.I have got many benefits from your
kindness.I am working on other an outlook interrelated bug. You can find it
here http://bugzilla.mozilla.org/show_bug.cgi?id=219559 .

	David :Bienvenu
	Comment 25 • 21 years ago

Comment on attachment 134089 [details] [diff] [review]
update

looks good, r=bienvenu, thx.

	wind li
	Comment 26 • 21 years ago

Can you suggest a sr David?I am a beginner to Mozilla community.Thx.

	wind li
	Comment 27 • 21 years ago

Comment on attachment 134089 [details] [diff] [review]
update

Seth,Can you have a look at this?

	David :Bienvenu
	Comment 28 • 21 years ago

yes, wind li, this can be checked in. Do you want me to check it in for you when
the tree opens for 1.6b?

	wind li
	Comment 29 • 21 years ago

Yes,I do.Thanks,David.

	David :Bienvenu
	Comment 30 • 21 years ago

fix checked in. thx again wind li

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 31 • 21 years ago

Fix doesn't work for Outlook Express. Testing with OE6 and Mozilla 1.6 still
results in an empty list but all entries were imported. Result is a broken
mailing list.

-> Reopen because bugs marked as dupes refers also to other products as Outlook.

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 32 • 21 years ago

Damn, bug 222774 depends on that bug. I'm very sorry.

-> Resetting to Fixed.