189140 - Rename category w/records, hotsync, and no change on desktop

Status: VERIFIED FIXED

(MailNews Core Graveyard :: Palm Sync, defect)

Description

[Ninoschka Baca]

Mozilla 1.2.1: WinMe

Overview: Rename a category on the palm device which includes records, HotSync
and no change occurs on Mozilla's address book.

Steps to reproduce:
1. Start with a category on the palm device that also exists on the MozAB (i.e.
AB1). They should also have atleast one record/card. At this point the
category/address book and their records are the same.
2. Rename the category on the palm device (i.e. AB2)
3. HotSync

Actual Results: 
MozAB - No change in the MozAB
Palm device: retained the renamed category

Expected Results: 
1. First preference (which depends on bug# 17230)
a. MozAB - the address book is renamed (i.e. AB1 becomes AB2)
b. Palm device: retain the renamed category (AB2)

2. Second preference
a. MozAB - creates another address book using the newly renamed category. In
this case it would retain the original address book and have another one (i.e.
AB1 and AB2). 
b. Palm device: retain the renamed category (AB2)
Note: Later on if a change was made to the category on the palm device then a
HotSync would occur between AB2 on the palm device and AB2 in the MozAB.

Comment 1

[Rajiv Dayal]

Hi Ninoschka, Can u please do a quick test: After you change the category name
on Palm, add an entry in that category and then sync. Can u then check whether
it creates AB2 or whether it adds a card for the entry in AB1 in Mozilla?

Comment 2

[Cavin Song]

Taking

Comment 3

[Cavin Song]

Attachment: Proposed patch, v1

This patch fixes a few problems:
1. Categories that were renamed on palm handheld are now renamed on mozilla. If
changes were also made to the same addressbooks on mozilla side, the changes
are merged into the renamed mozilla addressbook. Since mozilla does not allow
addrbooks to be renamed there’s no issue in this scenario.
2. Deleting addrbooks and/or categories on mozilla and/or palm should also
delete the corresponding addrbooks/categories on the other side (bug 188444,
183574).
3. Empty addrbooks/categories are now synced (bug 189123).
4. Deleting records in palm should also remove the corresponding mozilla cards
(bug 189995).
5. Added logging facility to the conduit dll via settings on environment
variable MOZ_CONDUIT_LOG. For example, set MOZ_CONDUIT_LOG to a file path like
“C:\\TEMP\\ConduitLog.txt”.

Comment 4

[Cavin Song]

Attachment: Proposed patch, v2

Added start and end time to the log. Removed extra 'nsresult rv;' decalration
in CPalmSyncImp::nsUpdateABSyncInfo.

Comment 5

[Cavin Song]

Fix checked in on 01/21.

Comment 6

[Cavin Song]

Marking fixed.

Comment 7

[Ninoschka Baca]

Trunk build 2003-02-12: WinXP
Verified Fixed, using the original scenario. It results in the expected
behavior(1a and 1b).