Need to consistently key events by string, not by integer, otherwise

when the results are JSON encoded we may get an object or an array.

diff --git a/kronolith/lib/Driver/Ical.php b/kronolith/lib/Driver/Ical.php
index 9f8440b..0719df3 100644 (file)
--- a/kronolith/lib/Driver/Ical.php
+++ b/kronolith/lib/Driver/Ical.php
@@ -79,16 +79,14 @@ class Kronolith_Driver_Ical extends Kronolith_Driver
 
         $components = $iCal->getComponents();
         $events = array();
-        $count = count($components);
         $exceptions = array();
-        for ($i = 0; $i < $count; $i++) {
-            $component = $components[$i];
+        foreach ($components as $component) {
             if ($component->getType() == 'vEvent') {
                 $event = new Kronolith_Event_Ical($this);
                 $event->status = Kronolith::STATUS_FREE;
                 $event->fromiCalendar($component);
                 $event->remoteCal = $this->_calendar;
-                $event->eventID = $i;
+                $event->eventID = hash('md5', uniqid(mt_rand(), true));
 
                 /* Catch RECURRENCE-ID attributes which mark single recurrence
                  * instances. */