232438 - cleanup palm sync category <-> AB mappings

Status: RESOLVED FIXED

(MailNews Core Graveyard :: Palm Sync, defect)

Description

[David :Bienvenu]

There are a few problems in the mapping between AB's and categories. The
upcoming patch makes the following changes:

1. Only delete palm categories if the corresponding Moz AB has been deleted. We
tell this by setting the position pref for the moz AB to 0. Previously, we just
assumed that missing AB's had been deleted, which caused problems in some
situations.

2. Map Moz Personal Address Book to "Personal" category on Palm.

3. Ignore ldap directories when figuring out which AB's to sync

4. If Moz AB has an existing card with the same display name as a Palm Category,
don't add the card to the Moz AB.

Comment 1

[David :Bienvenu]

Attachment: proposed fix

this patch could use a little cleaning up, but it should fix a whole host of
problems.

Comment 2

[David :Bienvenu]

Attachment: fix dup detection code

the dup detection code was wrong, because the arg is really a char *, not a
TCHAR *. Also, set the ab palm card id of the existing ab card.

Comment 3

[David :Bienvenu]

Attachment: fix Personal-> Personal Address Book mapping

one more place where Personal needs to match Personal Address Book

Comment 4

[David :Bienvenu]

Attachment: fix mappings when preferring work address and other fixes

This fixes handling of the prefer work address pref so that we handle two line
addresses, extend to work city, state, zip, and country, don't hardcode URI for
pab but map actual ldap2_servers.pab to Personal category, and ignore which
contact field is used for the purpose of comparing cards for equality - this
will fix a cause of dups when doing a first time sync after switching conduits.

Comment 5

[David :Bienvenu]

all fixes checked into m4 branch and trunk. I'm going to resolve this as fixed.
We can open new bugs for remaining issues.

Comment 6

[Wayne Mery (:wsmwk)]

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