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

bugs at bugs.horde.org bugs at bugs.horde.org
Thu May 24 21:11:11 UTC 2007


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




More information about the bugs mailing list