Don't assume all properties are set
authorMichael J. Rubinsky <mrubinsk@horde.org>
Sat, 8 May 2010 16:29:51 +0000 (12:29 -0400)
committerMichael J. Rubinsky <mrubinsk@horde.org>
Sat, 8 May 2010 16:29:51 +0000 (12:29 -0400)
framework/Alarm/lib/Horde/Alarm/Object.php

index 2910149..2995c70 100644 (file)
@@ -127,9 +127,8 @@ class Horde_Alarm_Object extends Horde_Alarm
     {
         $user = isset($alarm['user']) ? $alarm['user'] : null;
         $al = &$this->_findAlarm($alarm['id'], $user);
-
         foreach (array('start', 'end', 'methods', 'params', 'title', 'text') as $property) {
-            $al[$property] = $alarm[$property];
+            $al[$property] = isset($alarm[$property]) ? $alarm[$property] : null;
         }
         if (!$keepsnooze) {
             $al['snooze'] = null;