https://fedorahosted.org/freeipa/ticket/4523
From 27275d46251452175c4cb66222ffeda089a15e2e Mon Sep 17 00:00:00 2001 From: "Thierry bordaz (tbordaz)" <tbor...@redhat.com> Date: Tue, 7 Oct 2014 18:41:44 +0200 Subject: [PATCH] permission-add gives confusing error when adding ACI to generated tree Error reports that the modified entry does not exists https://fedorahosted.org/freeipa/ticket/4523 Reviewed by: --- ipalib/plugins/permission.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ipalib/plugins/permission.py b/ipalib/plugins/permission.py index 9028f02483bc113c19c75b94d70dd1b133272524..30d3266e4618a85e3974fc839c0d640542374d6d 100644 --- a/ipalib/plugins/permission.py +++ b/ipalib/plugins/permission.py @@ -943,7 +943,11 @@ class permission_add(baseldap.LDAPCreate): # the whole command, not just the callbacks def execute(self, *keys, **options): self.obj.preprocess_options(options, merge_targetfilter=True) - return super(permission_add, self).execute(*keys, **options) + try: + res = super(permission_add, self).execute(*keys, **options) + except errors.NotFound: + raise errors.NotFound(reason=_('Entry %s does not exist') % self.obj.get_dn(*keys, **options)) + return res def get_args(self): for arg in super(permission_add, self).get_args(): -- 1.7.11.7
_______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel