diff options
| author | Rob Crittenden <rcritten@redhat.com> | 2008-03-25 09:48:23 -0400 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2008-03-25 09:48:23 -0400 |
| commit | 1a1e02025824ba86ca847d59435541545b6f0aec (patch) | |
| tree | e5136ca12e9e1421f329c148622092f8e428de44 /ipa-server/ipa-gui/ipagui/subcontrollers/principal.py | |
| parent | 402187c8386acd28456f7e4391fd5b317b35d412 (diff) | |
| download | freeipa.git-1a1e02025824ba86ca847d59435541545b6f0aec.tar.gz freeipa.git-1a1e02025824ba86ca847d59435541545b6f0aec.tar.xz freeipa.git-1a1e02025824ba86ca847d59435541545b6f0aec.zip | |
Add additional detail to error messages.
This runs the risk of showing too much and confusing users but on the other
hand it often includes required information detailing why the error
occurred such as what attribute the user lacks write access too and why
changing a password failed.
438057
Diffstat (limited to 'ipa-server/ipa-gui/ipagui/subcontrollers/principal.py')
| -rw-r--r-- | ipa-server/ipa-gui/ipagui/subcontrollers/principal.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ipa-server/ipa-gui/ipagui/subcontrollers/principal.py b/ipa-server/ipa-gui/ipagui/subcontrollers/principal.py index 28a221fb..3c3d9463 100644 --- a/ipa-server/ipa-gui/ipagui/subcontrollers/principal.py +++ b/ipa-server/ipa-gui/ipagui/subcontrollers/principal.py @@ -103,7 +103,7 @@ class PrincipalController(IPAController): return dict(form=principal_new_form, principal=kw, tg_template='ipagui.templates.principalnew') except ipaerror.IPAError, e: - turbogears.flash("Service principal add failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Service principal add failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) return dict(form=principal_new_form, principal=kw, tg_template='ipagui.templates.principalnew') @@ -137,7 +137,7 @@ class PrincipalController(IPAController): principals[i].setValue('hostname', h[0]) except ipaerror.IPAError, e: - turbogears.flash("principal list failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("principal list failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) raise turbogears.redirect("/principal/list") return dict(principals=principals, hostname=hostname, fields=ipagui.forms.principal.PrincipalFields()) @@ -184,7 +184,7 @@ class PrincipalController(IPAController): turbogears.flash("Service principal deleted") raise turbogears.redirect('/principal/list') except (SyntaxError, ipaerror.IPAError), e: - turbogears.flash("Service principal deletion failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Service principal deletion failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) raise turbogears.redirect('/principal/list') @validate(form=principal_new_form) |