[Tickets #12523] Re: AS: mix of unix timestamp and modseq breaks sync

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12523
------------------------------------------------------------------------------
  Ticket             | 12523
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | AS: mix of unix timestamp and modseq breaks sync
  Queue              | Synchronization
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2013-08-07 22:06) wrote:

> We should replace the time() call in importMessageMove(), too.
> Even though moving groupware objects is currently unsupported,
> it will be a PITA to debug in the future if we ever support it.

I changed it, but there is nowhere to move groupware objects to since  
there is only one collection of each object type.

>> Reading my comments, it seems like I did it on purpose, but to be
>> honest, I'm not sure that the points I make are really necessary.
>> I'll have to revisit this.
>
> I'll have a look at the git history tomorrow, too.
> May be there's a hint in there.

It had to do with conflict detection/resolution but re-reading the  
specs, DELETE always takes precedence anyway so it was not needed  
after all. Changed.