Kronolith::getDefaultCalendar() can return a boolean false, which
authorMichael J. Rubinsky <mrubinsk@horde.org>
Tue, 27 Apr 2010 16:51:13 +0000 (12:51 -0400)
committerMichael J. Rubinsky <mrubinsk@horde.org>
Wed, 28 Apr 2010 16:49:54 +0000 (12:49 -0400)
causes as warning in array_key_exists(), so explicitly check for this also

kronolith/lib/Api.php

index f95015f..598212c 100644 (file)
@@ -492,8 +492,8 @@ class Kronolith_Api extends Horde_Registry_Api
             $calendar = Kronolith::getDefaultCalendar();
         }
 
-        if (!array_key_exists($calendar,
-            Kronolith::listCalendars(false, Horde_Perms::READ))) {
+        if ($calendar === false || 
+            !array_key_exists($calendar, Kronolith::listCalendars(false, Horde_Perms::READ))) {
             throw new Horde_Exception_PermissionDenied();
         }