6.0.0-RC7
6/27/26

[#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/5/06 (7144 days ago)
Due
Updated 12/16/06 (7133 days ago)
Assigned 12/5/06 (7144 days ago)
Resolved 12/16/06 (7133 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
2711 Karsten Fourmont Comment #2
State ⇒ Resolved
Reply to this comment
Thank you for the skilled analysis. I removed that line.
110 Jan Schneider Assigned to Karsten Fourmont
State ⇒ Assigned
 
558 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