diff options
author | Endi Sukma Dewata <edewata@redhat.com> | 2011-12-07 20:39:52 -0600 |
---|---|---|
committer | Petr Vobornik <pvoborni@redhat.com> | 2011-12-09 15:55:13 +0100 |
commit | 7cb39b8e33561d549c671389d38a8fa2c708ef3c (patch) | |
tree | 1cafac449cf34c1a34a60c9384125c2f06070751 /install/ui/hbactest.js | |
parent | b70ebe5c9f718d184ae6a60931fdccf0e22b9c48 (diff) | |
download | freeipa-7cb39b8e33561d549c671389d38a8fa2c708ef3c.tar.gz freeipa-7cb39b8e33561d549c671389d38a8fa2c708ef3c.tar.xz freeipa-7cb39b8e33561d549c671389d38a8fa2c708ef3c.zip |
Added HBAC Test input validation.
The HBAC Test pages have been modified to validate required input
before executing the test.
Ticket #388
Diffstat (limited to 'install/ui/hbactest.js')
-rw-r--r-- | install/ui/hbactest.js | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/install/ui/hbactest.js b/install/ui/hbactest.js index 916a2380..f2256a31 100644 --- a/install/ui/hbactest.js +++ b/install/ui/hbactest.js @@ -407,6 +407,25 @@ IPA.hbac.test_select_facet = function(spec) { } }; + that.validate = function(record) { + if (record[that.name]) return true; + + var dialog = IPA.message_dialog({ + title: IPA.messages.dialogs.validation_title, + message: IPA.messages.dialogs.validation_message + }); + + dialog.on_ok = function() { + var state = {}; + state[that.entity.name+'-facet'] = that.name; + IPA.nav.push_state(state); + }; + + dialog.open(); + + return false; + }; + init(); return that; @@ -675,15 +694,19 @@ IPA.hbac.test_run_facet = function(spec) { var facet = that.entity.get_facet('user'); facet.save(options); + if (!facet.validate(options)) return; facet = that.entity.get_facet('targethost'); facet.save(options); + if (!facet.validate(options)) return; facet = that.entity.get_facet('service'); facet.save(options); + if (!facet.validate(options)) return; facet = that.entity.get_facet('sourcehost'); facet.save(options); + if (!facet.validate(options)) return; facet = that.entity.get_facet('rules'); facet.save(options); |