Tickets :: [#10354] PermissionDenied exception in Kronolith_Api::listBy breaks syncml

6.0.0-beta1

7/4/25

Summary	PermissionDenied exception in Kronolith_Api::listBy breaks syncml
Queue	Kronolith
Queue Version	Git master
Type	Bug
State	Resolved
Priority	3. High
Owners	mrubinsk (at) horde (dot) org
Requester	stefandoesinger (at) gmx (dot) at
Created	07/18/2011 (5100 days ago)
Due
Updated	07/18/2011 (5100 days ago)
Assigned
Resolved	07/18/2011 (5100 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

07/18/2011 11:53:00 PM	stefandoesinger (at) gmx (dot) at	Comment #4	Reply to this comment
That bug was driving me nuts for a day, until I finally had the glorious idea to turn on logging. Thanks for the swift fix!

07/18/2011 09:54:10 PM	Michael Rubinsky	Comment #3 Assigned to Michael Rubinsky State ⇒ Resolved	Reply to this comment
Nice catch, thanks!

07/18/2011 09:53:38 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: Fix variable name, and fix syncing in certain situations. ~~Bug: 10354~~ Signed-off-by: Michael J Rubinsky <mrubinsk@horde.org> 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/5c26b427850de012125f196070f45f512f9dec11

07/18/2011 09:38:57 PM	stefandoesinger (at) gmx (dot) at	Comment #1 Priority ⇒ 3. High New Attachment: kronolith_api.diff Patch ⇒ Yes Milestone ⇒ Queue ⇒ Kronolith Summary ⇒ PermissionDenied exception in Kronolith_Api::listBy breaks syncml Type ⇒ Bug State ⇒ Unconfirmed	Reply to this comment
When trying to sync any syncml client against Kronolith 3.0.5 syncml does not pick up server side changes. This happens due to a PermissionDenied exception thrown in Kronolith_Api::listBy. I suspect that the problem is caused by a typo in this function. The attached diff fixes the issue for me. I'm not sure how correct the change is, but considering that $calendar is empty in this codepath and $c is the iterator over the selected sync calendars it looks like a logical fix.