From 9dc91aa5c027707a1a67dadd7066da448c74a136 Mon Sep 17 00:00:00 2001
From: Jan Schneider <jan@horde.org>
Date: Mon, 25 May 2009 18:37:06 +0200
Subject: [PATCH] Use correct permission for check. Return signature even on
 error.

---
 kronolith/ajax.php | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/kronolith/ajax.php b/kronolith/ajax.php
index c1cc8f2ae..8e810b936 100644
--- a/kronolith/ajax.php
+++ b/kronolith/ajax.php
@@ -18,7 +18,7 @@ function getDriver($cal)
     switch ($driver) {
     case 'internal':
         if (!array_key_exists($calendar,
-                              Kronolith::listCalendars(false, PERMS_READ))) {
+                              Kronolith::listCalendars(false, PERMS_SHOW))) {
             $GLOBALS['notification']->push(_("Permission Denied"), 'horde.error');
             return false;
         }
@@ -116,20 +116,19 @@ try {
     case 'ListEvents':
         $start = new Horde_Date(Util::getFormData('start'));
         $end   = new Horde_Date(Util::getFormData('end'));
-        if (!($kronolith_driver = getDriver($cal = Util::getFormData('cal')))) {
-            $result = true;
+        $cal   = Util::getFormData('cal');
+        $result = new stdClass;
+        $result->cal = $cal;
+        $result->view = Util::getFormData('view');
+        $result->sig = $start->dateString() . $end->dateString();
+        if (!($kronolith_driver = getDriver($cal))) {
             break;
         }
         $events = $kronolith_driver->listEvents($start, $end, true, false, true);
         if (is_a($events, 'PEAR_Error')) {
             $notification->push($events, 'horde.error');
-            $result = true;
             break;
         }
-        $result = new stdClass;
-        $result->cal = $cal;
-        $result->view = Util::getFormData('view');
-        $result->sig = $start->dateString() . $end->dateString();
         if (count($events)) {
             $result->events = $events;
         }
-- 
2.11.0