Patch attached

This probably should go to 4.0.x, 4.1 and master

--
Martin Basti

From 0c34bd80c6d22456b1194ec769c16dd289166e74 Mon Sep 17 00:00:00 2001
From: Martin Basti <mba...@redhat.com>
Date: Wed, 24 Sep 2014 16:46:27 +0200
Subject: [PATCH] FIX: dnszone-remove-permission should raise error

dnszone-remove-permission shoudl raise Not Found error if permission was
not found
---
 ipalib/plugins/dns.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/ipalib/plugins/dns.py b/ipalib/plugins/dns.py
index ea57b7407322b19faf43578323c38199b3ff2d69..64dc6f4bd9e1ed93ac325bf66a2d4859b8b03fb9 100644
--- a/ipalib/plugins/dns.py
+++ b/ipalib/plugins/dns.py
@@ -2058,10 +2058,7 @@ class DNSZoneBase_remove_permission(LDAPQuery):
             pass
 
         permission_name = self.obj.permission_name(keys[-1])
-        try:
-            self.obj._remove_permission(keys[-1])
-        except errors.NotFound:
-            pass
+        self.obj._remove_permission(keys[-1])
 
         return dict(
             result=True,
-- 
1.8.3.1

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to