diff options
author | Rob Crittenden <rcritten@redhat.com> | 2007-12-12 15:18:28 -0500 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2007-12-12 15:18:28 -0500 |
commit | b5af99c51d1ea8d4e6593a634d347b38a5d97271 (patch) | |
tree | e36a3d05f8925cdd3fb9d0d47a8ce194ac381af9 /ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py | |
parent | 6976f92862c1dba3f7a25baa1893c41a67590b23 (diff) | |
download | freeipa.git-b5af99c51d1ea8d4e6593a634d347b38a5d97271.tar.gz freeipa.git-b5af99c51d1ea8d4e6593a634d347b38a5d97271.tar.xz freeipa.git-b5af99c51d1ea8d4e6593a634d347b38a5d97271.zip |
Fix the UI indicators for mandatory fields.
The source and target groups weren't highlighted but the entire block of
checkboxes was and it looked absolutely horrible.
Diffstat (limited to 'ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py')
-rw-r--r-- | ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py b/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py index cee239e7..9f4ec6c4 100644 --- a/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py +++ b/ipa-server/ipa-gui/ipagui/subcontrollers/delegation.py @@ -292,4 +292,11 @@ class DelegationController(IPAController): @validate(form=delegate_form) @identity.require(identity.not_anonymous()) def delegatevalidate(self, tg_errors=None, **kw): + # We are faking this because otherwise it shows up as one huge + # block of color in the UI when it has a not empty validator. + if not kw.get('attrs'): + if not tg_errors: + tg_errors = {} + tg_errors['attrs'] = _("Please select at least one value") + cherrypy.request.validation_errors = tg_errors return tg_errors, kw |