[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         | jochem at mondrian.nl
 Summary            | Recurring events missing RRULE field
 Queue              | Kronolith
 Version            | HEAD
 Type               | Bug
 State              | Assigned
 Priority           | 2. Medium
 Owners             | Karsten Fourmont
+New Attachment     | sql.php.patch
-----------------------------------------------------------------------


jochem at mondrian.nl (2007-04-12 04:12) 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.