[Tickets #13866] Re: CalDAV PUT parses multiple VTIMEZONE data incorrectly, even its own ones

[noreply at bugs.horde.org]

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

Ticket URL: https://bugs.horde.org/ticket/13866
------------------------------------------------------------------------------
  Ticket             | 13866
  Updated By         | Joerg.Pulz at frm2.tum.de
  Summary            | CalDAV PUT parses multiple VTIMEZONE data incorrectly,
                     | even its own ones
  Queue              | Synchronization
  Version            | FRAMEWORK_5_2
  Type               | Bug
  State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Jan Schneider
+New Attachment     |  
patch-framework_Icalendar_lib_Horde_Icalendar_Vtimezone.php
------------------------------------------------------------------------------


Joerg.Pulz at frm2.tum.de (2015-04-15 16:30) wrote:

Forget about the last patch, i was plain wrong.

The gmmktime() call has to use $year of the VEVENT for RRULEs without  
UNTIL parameter to get the correct dates for the CET/CEST switch in  
the year of the event and has to use $result['end'] for all RRULEs  
with UNTIL parameter to calculate the correct switch dates for this  
rules.

I reworked the patch to honor this.
Tested with em Client and Thunderbird/Lightning and seem to be okay.

Would be nice to see this committed as we soon start our internal  
scheduling for the beginning of 2016.

Kind regards
Joerg



Joerg.Pulz at frm2.tum.de (2015-04-15 16:30) uploaded:  
patch-framework_Icalendar_lib_Horde_Icalendar_Vtimezone.php

https://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=patch-framework_Icalendar_lib_Horde_Icalendar_Vtimezone.php&ticket=13866&fn=%2Fpatch-framework_Icalendar_lib_Horde_Icalendar_Vtimezone.php