6.0.0-beta1
8/9/25

[#5080] Wrong multiple iCalendar events detection during SyncML
Summary Wrong multiple iCalendar events detection during SyncML
Queue Kronolith
Queue Version HEAD
Type Bug
State Resolved
Priority 1. Low
Owners karsten (at) horde (dot) org
Requester x3n (at) molinier (dot) eu
Created 03/07/2007 (6730 days ago)
Due
Updated 04/27/2007 (6679 days ago)
Assigned 04/01/2007 (6705 days ago)
Resolved 04/27/2007 (6679 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
04/27/2007 03:12:57 PM Jan Schneider State ⇒ Resolved
 
04/01/2007 12:37:26 PM Karsten Fourmont State ⇒ Feedback
 
04/01/2007 12:37:05 PM Karsten Fourmont Comment #2 Reply to this comment
Submitted something similar. Please test and report if this is working 
for you. Thanks.


03/07/2007 07:58:10 PM Chuck Hagenbuch Assigned to Karsten Fourmont
State ⇒ Assigned
 
03/07/2007 05:57:44 PM x3n (at) molinier (dot) eu Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Wrong multiple iCalendar events detection during SyncML
Queue ⇒ Kronolith
State ⇒ Unconfirmed
Reply to this comment
in kronolith/lib/api.php:760, there is a analysis of the VCALENDAR 
sent by the client. Is this analysis, the program checks if there is 
on or more VEVENT element in the content.

If there are several element, Kronolith cannot handle it. So there is 
an exception.



The bug is the following: we are not couting the number of VEVENT 
element in VCALENDAR but the number of element in whole. There is a 
problem when the client is sending a VTIMEZONE object at the same 
time...



To solve this, I propose to leave away everything except VEVENT (we 
don't care about VTIMEZONE here!):



-line 757 -

             $components=array();

             // Filtering components to keep only VEVENT objects

             foreach($iCal->getComponents() as $component) {

               if(is_a($component, 'Horde_iCalendar_vevent')) {

                 array_push($components, $component);

               }

             }



             switch (count($components)) {

...



Best regards,



x3n

http://www.molinier.eu

Saved Queries