[Tickets #8474] Re: All day events cause crash with Funambol outlook connector

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/8474
------------------------------------------------------------------------------
  Ticket             | 8474
  Updated By         | Jan Schneider <jan at horde.org>
  Summary            | All day events cause crash with Funambol outlook
                     | connector
  Queue              | SyncML
  Type               | Bug
  State              | Feedback
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2009-08-20 11:45) wrote:

> What about the following :
> I have included the DTSTART case in the getAllAttributes() loop  
> again to streamline the code and modified the DTEND case to use the  
> same formating as the DTSTART. The variable "$start" is only used to  
> compare against "$alarm" as far as i can see and this is working.  
> The "$end" is only used to compute "Duration" which is no SIF-E  
> attribute anyway as far as i can see??

No, the $start is also used inside the loop, and we can't guarantee  
that the DTSTART attribute is always before the other attributes that  
depend on it. Beside that, the $start variable had been overwritten  
inside the loop under certain circumstances.