Patrick Ohly wrote: > On Thu, 2016-08-25 at 08:50 +0200, deloptes wrote: >> Patrick Ohly wrote: >> >> The story about ITEM_NEEDS_MERGE however is still a bit unclear to me. >> >> Is it a suggestion - at least I understand it as such? >> > >> > It's a bit more than a suggestion. It's a request to the engine to do >> > the merge. What happens is: >> > 1. add new item -> ITEM_NEEDS_MERGE without changing the database >> > 2. read old item with luid as provided with ITEM_NEEDS_MERGE >> > 3. update old item with merged data >> > >> >> So in this case I should not DEL+ADD in the backend, but just notify with >> ITEM_NEEDS_MERGE? Correct? > > Correct. >
I think it's good to go now. I did not excessively tested it, but following your advise I changed the relevant places. I'm just not confident what it should return - in terms of id and revision, but what I tested performed without issues. It would be nice if I can upload the code to where it belongs and we get opportunity to build packages from the source for our desktop as well. regards _______________________________________________ SyncEvolution mailing list [email protected] https://lists.syncevolution.org/mailman/listinfo/syncevolution
