[Tickets #11587] PHP Fatal error: Call to a member function setTimezone() on a non-object in /var/www/horde5/kronolith/lib/Kronolith.php on line 407

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/11587
------------------------------------------------------------------------------
  Ticket             | 11587
  Created By         | vm08823817073160362v at vmail.inf.h-brs.de
  Summary            | PHP Fatal error:  Call to a member function
                     | setTimezone() on a non-object in
                     | /var/www/horde5/kronolith/lib/Kronolith.php on line 407
  Queue              | Kronolith
  Version            | Git develop
  Type               | Bug
  State              | Unconfirmed
  Priority           | 3. High
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


vm08823817073160362v at vmail.inf.h-brs.de (2012-10-29 14:00) wrote:

This error pops up in syslog when an ics-file is imported. The event  
is the first "complicated" looking one in the file. Attached you'll  
find the event, that triggered the error.

If I look around 407, the loop suggests that $next is to become false  
eventually, but the 2nd if($convert) ignores that. As workaround I  
have added a && $next !== false to it. That seems to work.

             while ($next !== false && $next->compareDate($endDate) <= 0) {
                 if (!$event->recurrence->hasException($next->year,  
$next->month, $next->mday)) {
[snip]
                 }
                 if ($convert) {
                     $next->setTimezone($event->timezone);
                 }
                 $next = $event->recurrence->nextRecurrence(
                     array('year' => $next->year,
                           'month' => $next->month,
                           'mday' => $next->mday + 1,
                           'hour' => $next->hour,
                           'min' => $next->min,
                           'sec' => $next->sec));
                 if ($convert && $next !== false) {
#line 407:
                     $next->setTimezone($timezone);
                 }
             }




vm08823817073160362v at vmail.inf.h-brs.de (2012-10-29 14:00) uploaded:  
sprechstunde.ics.gz

http://bugs.horde.org/h/services/download/?module=whups&actionID=download_file&file=sprechstunde.ics.gz&ticket=11587&fn=%2Fsprechstunde.ics.gz