diff options
author | Adam Young <ayoung@redhat.com> | 2011-06-27 15:57:40 -0400 |
---|---|---|
committer | Adam Young <ayoung@redhat.com> | 2011-06-27 20:15:56 -0400 |
commit | 6e967d8fe709150d44b9af8df7312641a8747acf (patch) | |
tree | 829f50ece4af03d10a9965dcc778da9f6a29949d /install/ui/widget.js | |
parent | d2d5278c52650cb90f1d77ac748f0a73f401bf5a (diff) | |
download | freeipa-6e967d8fe709150d44b9af8df7312641a8747acf.tar.gz freeipa-6e967d8fe709150d44b9af8df7312641a8747acf.tar.xz freeipa-6e967d8fe709150d44b9af8df7312641a8747acf.zip |
validate required fields https://fedorahosted.org/freeipa/ticket/1329
overides required with optional.
Diffstat (limited to 'install/ui/widget.js')
-rw-r--r-- | install/ui/widget.js | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/install/ui/widget.js b/install/ui/widget.js index 38826ee9d..344d701c2 100644 --- a/install/ui/widget.js +++ b/install/ui/widget.js @@ -39,6 +39,8 @@ IPA.widget = function(spec) { that.disabled = spec.disabled; that.hidden = spec.hidden; that.conditional = spec.conditional; + that.optional = spec.optional || false; + // read_only is set during initialization that.read_only = spec.read_only; @@ -79,6 +81,12 @@ IPA.widget = function(spec) { var values = that.save(); if (!values || !values.length) { + if (that.param_info && + that.param_info.required && + !that.optional) { + that.valid = false; + that.show_error('required field'); + } return; } |