From fcace7981319c5c28eac01ff71e1c3a3de1a3269 Mon Sep 17 00:00:00 2001
From: Florian Schlichting <fsfs@debian.org>
Date: Fri, 30 Dec 2016 16:25:53 +0100
Subject: [PATCH] display an error message when not allowed to delete something
 on the admin page

---
 inc/ui/principal-edit.php | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/inc/ui/principal-edit.php b/inc/ui/principal-edit.php
index 182588c2..95d5a880 100644
--- a/inc/ui/principal-edit.php
+++ b/inc/ui/principal-edit.php
@@ -74,6 +74,9 @@ function handle_subaction( $subaction ) {
           return false;
         }
       }
+      else {
+        $c->messages[] = i18n('You are not allowed to delete collections for this principal.');
+      }
       break;
 
     case 'delete_principal':
@@ -97,6 +100,9 @@ function handle_subaction( $subaction ) {
           return false;
         }
       }
+      else {
+        $c->messages[] = i18n('You are not allowed to delete principals.');
+      }
       break;
 
     case 'delete_ticket':
@@ -121,6 +127,9 @@ function handle_subaction( $subaction ) {
           return false;
         }
       }
+      else {
+        $c->messages[] = i18n('You are not allowed to delete tickets for this principal.');
+      }
       break;
 
     case 'delete_bind_in':
@@ -129,7 +138,7 @@ function handle_subaction( $subaction ) {
       if ( $session->AllowedTo('Admin')
                 || ($id > 0 && $session->principal_id == $id) ) {
         if ( $session->CheckConfirmationHash('GET', 'confirm') ) {
-          dbg_error_log('admin-principal-edit',':handle_action: Allowed to delete ticket "%s" for principal %d', $_GET['bind_id'], $id );
+          dbg_error_log('admin-principal-edit',':handle_action: Allowed to delete binding "%s" for principal %d', $_GET['bind_id'], $id );
           $qry = new AwlQuery('DELETE FROM dav_binding WHERE bind_id=?;', $_GET['bind_id'] );
           if ( $qry->Exec() ) {
             $c->messages[] = i18n('Binding deleted');
@@ -151,6 +160,9 @@ function handle_subaction( $subaction ) {
           return false;
         }
       }
+      else {
+        $c->messages[] = i18n('You are not allowed to delete bindings for this principal.');
+      }
       break;
 
       default: