Alternative fix for #312. Extract the timezone, even when there's no master event.

This fix uses the TZID from the RECURRENCE-ID to figure out the timezone. This is
more accurate than just grabbing the first VTIMEZONE. RECURRENCE-ID must always
be in the same timezone as DTSTART from the master event.

Author: evert

lib/ITip/Broker.php

@@ -813,7 +813,8 @@ protected function parseEventForAttendee(VCalendar $calendar, array $eventInfo,
      * 7. attendees
      * 8. sequence
      * 9. exdate
-     * 10. timezone
+     * 10. timezone - strictly the timezone on which the recurrence rule is
+     *                based on.
      * 11. significantChangeHash
      * 12. status
      * @param VCalendar $calendar
@@ -826,7 +827,7 @@ protected function parseEventInfo(VCalendar $calendar = null) {
         $organizerName = null;
         $organizerForceSend = null;
         $sequence = null;
-        $timezone = $calendar->VTIMEZONE ? $calendar->VTIMEZONE->getTimeZone() : null;
+        $timezone = null;
         $status = null;
         $organizerScheduleAgent = 'SERVER';
 
@@ -884,9 +885,13 @@ protected function parseEventInfo(VCalendar $calendar = null) {
                 $status = strtoupper($vevent->STATUS->getValue());
             }
 
-            $recurId = isset($vevent->{'RECURRENCE-ID'})?$vevent->{'RECURRENCE-ID'}->getValue():'master';
-            if ($recurId==='master') {
-                $timezone = $vevent->DTSTART->getDateTime()->getTimeZone();
+            $recurId = isset($vevent->{'RECURRENCE-ID'}) ? $vevent->{'RECURRENCE-ID'}->getValue() : 'master';
+            if (is_null($timezone)) {
+                if ($recurId === 'master') {
+                    $timezone = $vevent->DTSTART->getDateTime()->getTimeZone();
+                } else {
+                    $timezone = $vevent->{'RECURRENCE-ID'}->getDateTime()->getTimeZone();
+                }
             }
             if(isset($vevent->ATTENDEE)) {
                 foreach($vevent->ATTENDEE as $attendee) {

tests/VObject/ITip/BrokerTimezoneInParseEventInfoWithoutMasterTest.php

@@ -71,7 +71,7 @@ function testTimezoneInParseEventInfoWithoutMaster()
         $reflectionMethod = new \ReflectionMethod($broker, 'parseEventInfo');
         $reflectionMethod->setAccessible(true);
         $data = $reflectionMethod->invoke($broker, $calendar);
-
+        $this->assertInstanceOf('DateTimeZone', $data['timezone']);
         $this->assertEquals($data['timezone']->getName(), 'Europe/Minsk');
     }
 }