diff options
| author | Alexander Bokovoy <abokovoy@redhat.com> | 2012-09-20 14:02:15 +0300 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2012-09-20 14:57:19 +0200 |
| commit | b080744e86134ed4cd38d3eba93e52f294ab606f (patch) | |
| tree | 10b11cacf1ef5aecea72a50924f6fa59b28925cf /ipalib | |
| parent | 7d2dbe56cb987bb8d9b197260edfe28de99c371b (diff) | |
| download | freeipa.git-b080744e86134ed4cd38d3eba93e52f294ab606f.tar.gz freeipa.git-b080744e86134ed4cd38d3eba93e52f294ab606f.tar.xz freeipa.git-b080744e86134ed4cd38d3eba93e52f294ab606f.zip | |
Fix error messages and use proper ImportError for dcerpc import
Diffstat (limited to 'ipalib')
| -rw-r--r-- | ipalib/plugins/group.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/ipalib/plugins/group.py b/ipalib/plugins/group.py index 01158720..ae00aa8a 100644 --- a/ipalib/plugins/group.py +++ b/ipalib/plugins/group.py @@ -26,7 +26,7 @@ if api.env.in_server and api.env.context in ['lite', 'server']: try: import ipaserver.dcerpc _dcerpc_bindings_installed = True - except Exception, e: + except ImportError: _dcerpc_bindings_installed = False __doc__ = _(""" @@ -328,14 +328,13 @@ class group_add_member(LDAPAddMember): result = (completed, dn) if 'ipaexternalmember' in options: if not _dcerpc_bindings_installed: - raise errors.NotFound(name=_('AD Trust'), - reason=_('''Cannot perform external member validation without Samba 4 support installed. - Make sure you have installed server-trust-ad sub-package of IPA on the server''')) + raise errors.NotFound(reason=_('Cannot perform external member validation without ' + 'Samba 4 support installed. Make sure you have installed ' + 'server-trust-ad sub-package of IPA on the server')) domain_validator = ipaserver.dcerpc.DomainValidator(self.api) if not domain_validator.is_configured(): - raise errors.NotFound(name=_('AD Trust setup'), - reason=_('''Cannot perform join operation without own domain configured. - Make sure you have run ipa-adtrust-install on the IPA server first''')) + raise errors.NotFound(reason=_('Cannot perform join operation without own domain configured. ' + 'Make sure you have run ipa-adtrust-install on the IPA server first')) sids = [] failed_sids = [] for sid in options['ipaexternalmember']: |