I guess it's not a bug in our code then?

But strange that it breaks with older pear packages...

Hello

It seems to be more complex that what I suggested.

We have two horde instances (test, production) having different pear packages.

In the test instance, we need to add the last entry of each message 
(such as I suggested), otherwise the entry is missing.

HTTP_Request 1.4.0

XML_Parser 1.2.7

XML_RPC     1.5.1



But in the production instance, the CVS code works fine (If we add the 
suggested code, the last entry of each message is doubled).

HTTP_Request 1.3.0

XML_Parser 1.0.1

WML_RPC   1.1.0



Regards

Patrick

Hello

When synchronizing (reload device) a large number of turba contacts 
(525), the last contact of each SyncML message is added (log 'add: 
<suid>') but the corresponding uidMap is not created (the log 'created 
Map for .. ' is missing). So at the end, we get 512 contacts instead 
of 525.



It seems that in Sync.php a few lines are missing in createSyncOutput().

After detecting that 'max msg size approached', we still need to add 
the last entry, before calling the return statement.



}

line 438: + $output->outputSyncCommand('Add',

                                                                       
         $clientContent,

                                                                       
         $clientContentType,

                                                                       
         $clientEncodingType,

                                                                       
        null,

                                                                       
        $suid);

line 444: + $this->_server_add_count++;



$messageFull = true;



Hope this helps.

Regards

Patrick