diff options
author | Abhijeet Kasurde <akasurde@redhat.com> | 2015-10-27 17:21:17 +0530 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2015-10-29 13:41:41 +0100 |
commit | c60cec4fa7adf004d383b68b78f6fd51d5cecb21 (patch) | |
tree | f3b0039fb2b17cae5e7dc62ad0528d1cecc6ae45 /ipalib/plugins/dns.py | |
parent | 82fd4250b9b4f408174edec7c7f070dc9fc73ab0 (diff) | |
download | freeipa-c60cec4fa7adf004d383b68b78f6fd51d5cecb21.tar.gz freeipa-c60cec4fa7adf004d383b68b78f6fd51d5cecb21.tar.xz freeipa-c60cec4fa7adf004d383b68b78f6fd51d5cecb21.zip |
Added user friendly error message for dnszone enable and disable
Added try-except block in dns plugin in order to provide user
friendly message to end user.
https://fedorahosted.org/freeipa/ticket/4811
Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
Reviewed-By: Martin Basti <mbasti@redhat.com>
Diffstat (limited to 'ipalib/plugins/dns.py')
-rw-r--r-- | ipalib/plugins/dns.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/ipalib/plugins/dns.py b/ipalib/plugins/dns.py index ef282c946..48d6f740e 100644 --- a/ipalib/plugins/dns.py +++ b/ipalib/plugins/dns.py @@ -2231,7 +2231,11 @@ class DNSZoneBase_disable(LDAPQuery): ldap = self.obj.backend dn = self.obj.get_dn(*keys, **options) - entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass']) + try: + entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass']) + except errors.NotFound: + self.obj.handle_not_found(*keys) + if not _check_entry_objectclass(entry, self.obj.object_class): self.obj.handle_not_found(*keys) @@ -2252,7 +2256,11 @@ class DNSZoneBase_enable(LDAPQuery): ldap = self.obj.backend dn = self.obj.get_dn(*keys, **options) - entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass']) + try: + entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass']) + except errors.NotFound: + self.obj.handle_not_found(*keys) + if not _check_entry_objectclass(entry, self.obj.object_class): self.obj.handle_not_found(*keys) |