From 88f7e8ba4282d5611509e5ba633937f27fefa291 Mon Sep 17 00:00:00 2001 From: "Michael J. Rubinsky" Date: Mon, 5 Jul 2010 13:55:08 -0400 Subject: [PATCH] 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). --- kronolith/lib/Ajax/Application.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) 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) { -- 2.11.0