[Tickets #4730] Backend.php: getServerAdditions line 259

Tue Dec 5 00:30:55 PST 2006

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

Ticket URL: http://bugs.horde.org/ticket/?id=4730
-----------------------------------------------------------------------
 Ticket             | 4730
 Created By         | patrick.abiven at apitech.fr
 Summary            | Backend.php: getServerAdditions line 259  
 Queue              | SyncML
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------

patrick.abiven at apitech.fr (2006-12-05 00:30) wrote:

Hello
Doing a "Reload Device" operation (600 contacts) from a Synthesis client
generates "Invalid Data". This is due to a server timeout. The
getServerAdditions function takes 4 minutes to execute. The time is taken
by the line 259 ($cuid= $this->getCuid(..)) which seems to be not required
(the $cuid variable is not used after). Can this line be removed ?

For a "Reload Device" operation  ($from_ts== 0), as the getServerAdditions
should return all the entries, does the foreach loop (line 249) still
required ?
Regards
Patrick