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/delegation.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/delegation.py')
| -rw-r--r-- | ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py b/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py index 73b0cbe6..c6523698 100644 --- a/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py +++ b/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py @@ -144,7 +144,7 @@ class DelegationController(IPAController): pass except ipaerror.IPAError, e: - turbogears.flash("Delgate add failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Delgate add failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) return dict(form=delegate_form, delegate=kw, tg_template='ipagui.templates.delegatenew') @@ -174,7 +174,7 @@ class DelegationController(IPAController): return dict(form=delegate_form, delegate=delegate) except (SyntaxError, ipaerror.IPAError), e: - turbogears.flash("Delegation edit failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Delegation edit failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) raise turbogears.redirect('/delegate/list') @@ -259,7 +259,7 @@ class DelegationController(IPAController): turbogears.flash("delegate updated") raise turbogears.redirect('/delegate/list') except (SyntaxError, ipaerror.IPAError), e: - turbogears.flash("Delegation update failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Delegation update failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) return dict(form=delegate_form, delegate=kw, tg_template='ipagui.templates.delegateedit') @@ -272,7 +272,7 @@ class DelegationController(IPAController): try: aci_entry = client.get_aci_entry(aci_fields) except ipaerror.IPAError, e: - turbogears.flash("Delegation list failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Delegation list failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) raise turbogears.redirect('/') aci_str_list = aci_entry.getValues('aci') @@ -351,7 +351,7 @@ class DelegationController(IPAController): turbogears.flash("delegate deleted") raise turbogears.redirect('/delegate/list') except (SyntaxError, ipaerror.IPAError), e: - turbogears.flash("Delegation deletion failed: " + str(e) + "<br/>" + e.detail[0]['desc']) + turbogears.flash("Delegation deletion failed: " + str(e) + "<br/>" + e.detail[0].get('desc','') + ". " + e.detail[0].get('info','')) raise turbogears.redirect('/delegate/list') @expose("ipagui.templates.delegategroupsearch") |