[Tickets #5401] Moving an event doesn't show error message if not permitted

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=5401
-----------------------------------------------------------------------
 Ticket             | 5401
 Created By         | thomas.jarosch at intra2net.com
 Summary            | Moving an event doesn't show error message if not permitted
 Queue              | Kronolith
 Version            | HEAD
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
+New Attachment     | kronolith-fix-move-error-handling.patch
-----------------------------------------------------------------------


thomas.jarosch at intra2net.com (2007-05-24 14:11) wrote:

Hello,

moving an event doesn't show an error message if you don't have all the
needed permissions due a missing getEvent() call. The error handling was a
bit messy so I tried to improve it. I also added error output for the case
you don't have PERMS_DELETE in the source share. Please see the attached
patch.

I've a question about kronolith->close(). It's currently a dummy stub for
all Drivers, so what about removing it? I'm still wondering what the
original intention was directly after the move of the event to the target
share.

Thomas