From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Mon, 5 Jul 2010 17:55:08 +0000 (-0400)
Subject: Need to load the tasklists *after* we save the changes.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=88f7e8ba4282d5611509e5ba633937f27fefa291;p=horde.git

Need to load the tasklists *after* we save the changes.

Otherwise, Kronolith::readPermsForm() will overwrite the changes to the share.
Additionally, we can get rid of the perms check here since tasks/updateTasklist will throw
a Horde_Exception_PermissionDenied exception (via Nag::updateTasklist).
---

diff --git a/kronolith/lib/Ajax/Application.php b/kronolith/lib/Ajax/Application.php
index d965016a5..06350d303 100644
--- a/kronolith/lib/Ajax/Application.php
+++ b/kronolith/lib/Ajax/Application.php
@@ -672,13 +672,9 @@ class Kronolith_Ajax_Application extends Horde_Ajax_Application_Base
 
             // Update a task list.
             $calendar_id = substr($calendar_id, 6);
-            $tasklists = $GLOBALS['registry']->tasks->listTasklists(true, Horde_Perms::EDIT);
-            if (!isset($tasklists[$calendar_id])) {
-                $GLOBALS['notification']->push(_("You are not allowed to change this task list."), 'horde.error');
-                return $result;
-            }
             try {
                 $GLOBALS['registry']->tasks->updateTasklist($calendar_id, $calendar);
+                $tasklists = $GLOBALS['registry']->tasks->listTasklists(true, Horde_Perms::EDIT);
                 Kronolith::readPermsForm($tasklists[$calendar_id]);
                 $result->perms = Kronolith::permissionToJson($tasklists[$calendar_id]->getPermission());
             } catch (Exception $e) {