diff options
author | Adam Young <ayoung@redhat.com> | 2011-07-06 15:43:50 -0400 |
---|---|---|
committer | Endi S. Dewata <edewata@redhat.com> | 2011-07-08 01:36:52 +0000 |
commit | 0a5f103733236d41b0fc921264098180069a02c8 (patch) | |
tree | 47a695086c00c0871e2ef40ae085285a4276c4e7 /install/ui/add.js | |
parent | 6dc6c4b2c6f39c774807fa01ffbc98e4bcc2e2f6 (diff) | |
download | freeipa-0a5f103733236d41b0fc921264098180069a02c8.tar.gz freeipa-0a5f103733236d41b0fc921264098180069a02c8.tar.xz freeipa-0a5f103733236d41b0fc921264098180069a02c8.zip |
check required on add
previsouly was checked on key down, but that does the check too soon.
Next attempt was on blur, but that had numerous problems. This now checkes when the add button is clicked.
works for entity_select widget, too
Checks upon form submission
https://fedorahosted.org/freeipa/ticket/1437
Diffstat (limited to 'install/ui/add.js')
-rw-r--r-- | install/ui/add.js | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/install/ui/add.js b/install/ui/add.js index 50b6124c0..0a414b74b 100644 --- a/install/ui/add.js +++ b/install/ui/add.js @@ -118,11 +118,14 @@ IPA.add_dialog = function (spec) { for (var i=0; i<fields.length; i++) { fields[i].validate(); } - + var required_fields_filled = true; for (i=0; i<fields.length; i++) { field = fields[i]; if (!field.valid) return; + required_fields_filled = field.check_required() && + required_fields_filled; + value = record[field.name]; if (!value) continue; @@ -141,6 +144,8 @@ IPA.add_dialog = function (spec) { for (var k=0; k<section_fields.length; k++) { field = section_fields[k]; if (!field.valid) return; + required_fields_filled = field.check_required() && + required_fields_filled; value = record[field.name]; if (!value) continue; @@ -155,7 +160,9 @@ IPA.add_dialog = function (spec) { //alert(JSON.stringify(command.to_json())); - command.execute(); + if (required_fields_filled){ + command.execute(); + } }; that.add_dialog_init = that.init; |