6.0.0-beta1
10/16/25

[#4730] Backend.php: getServerAdditions line 259
Summary Backend.php: getServerAdditions line 259
Queue Synchronization
Type Bug
State Resolved
Priority 1. Low
Owners karsten (at) horde (dot) org
Requester patrick.abiven (at) apitech (dot) fr
Created 12/05/2006 (6890 days ago)
Due
Updated 12/16/2006 (6879 days ago)
Assigned 12/05/2006 (6890 days ago)
Resolved 12/16/2006 (6879 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
12/16/2006 11:22:27 AM Karsten Fourmont Comment #2
State ⇒ Resolved
Reply to this comment
Thank you for the skilled analysis. I removed that line.
12/05/2006 10:52:01 AM Jan Schneider Assigned to Karsten Fourmont
State ⇒ Assigned
 
12/05/2006 08:30:55 AM patrick (dot) abiven (at) apitech (dot) fr Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Backend.php: getServerAdditions line 259
Queue ⇒ Synchronization
State ⇒ Unconfirmed
Reply to this comment
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

Saved Queries