[Tickets #5236] Re: Recurring events missing RRULE field

[bugs at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=5236
-----------------------------------------------------------------------
 Ticket             | 5236
 Updated By         | horde at eltofts.homelinux.com
 Summary            | Recurring events missing RRULE field
 Queue              | Kronolith
 Version            | HEAD
 Type               | Bug
 State              | Assigned
 Priority           | 2. Medium
 Owners             | Karsten Fourmont
-----------------------------------------------------------------------


horde at eltofts.homelinux.com (2007-04-12 04:24) wrote:

> I did some more research, it seems that the kronolith_event_sql 
> object used in _kronolith_export in kronolith/lib/api.php is broken. 
> I've modified that file to write the print_r($event, true) value to a 
> file.
>
> When I create an event with or without an enddate, and synchronize it 
> to my pda, and examine the contents of $event, it contains no 
> recurrence information.
>
> When I create an event with an recurcount, it creates an event with 
> 'recurtype' set to the number of recurrences (12 in this case):
> [recurCount] =>
> [recurType] => 12
> [recurInterval] => 1
>
> When doing something similar in kronolith/lib/Driver/sql.php's 
> getByUID function, I noticed that the data being pulled from the db 
> had the same discrepancies. Upon closer inspection, there was a , 
> missing from the query after event_recurcount, on line 395. Because 
> event_recurtype is the next in the list, mysql assumed the query 
> meant 'event_recurcount AS event_recurtype', messing up the data.
>
> Fixing this fixed the problem.
>
> I've attached a patch, and I'll be submitting another ticket and 
> patch for a seperate problem with recurrence in the SyncML module.

I have added the missing "," and this has sorted it... thanks :)