diff --git a/install/ui/dialog.js b/install/ui/dialog.js index 324eb9b93..9003aa820 100644 --- a/install/ui/dialog.js +++ b/install/ui/dialog.js @@ -664,5 +664,7 @@ IPA.message_dialog = function(spec) { init(); + that.message_dialog_create = that.create; + return that; }; diff --git a/install/ui/hbactest.js b/install/ui/hbactest.js index 0e77083ca..cab579e13 100644 --- a/install/ui/hbactest.js +++ b/install/ui/hbactest.js @@ -410,19 +410,6 @@ 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; }; @@ -691,22 +678,34 @@ IPA.hbac.test_run_facet = function(spec) { var command = IPA.command({ method: 'hbactest' }); var options = {}; + var validation_results = { + valid: true, + invalid_facets: [] + }; var facet = that.entity.get_facet('user'); facet.save(options); - if (!facet.validate(options)) return; + that.validate_facet(facet, options, validation_results); facet = that.entity.get_facet('targethost'); facet.save(options); - if (!facet.validate(options)) return; + that.validate_facet(facet, options, validation_results); facet = that.entity.get_facet('service'); facet.save(options); - if (!facet.validate(options)) return; + that.validate_facet(facet, options, validation_results); facet = that.entity.get_facet('sourcehost'); facet.save(options); - if (!facet.validate(options)) return; + that.validate_facet(facet, options, validation_results); + + if (!validation_results.valid) { + var dialog = IPA.hbac.validation_dialog({ + validation_results: validation_results + }); + dialog.open(); + return; + } facet = that.entity.get_facet('rules'); facet.save(options); @@ -721,6 +720,17 @@ IPA.hbac.test_run_facet = function(spec) { command.execute(); }; + that.validate_facet = function(facet, options, validation_results) { + + var facet_valid = facet.validate(options); + + validation_results.valid = facet_valid && validation_results.valid; + + if (!facet_valid) { + validation_results.invalid_facets.push(facet); + } + }; + that.get_records_map = function(data) { var records_map = $.ordered_map(); @@ -759,4 +769,64 @@ IPA.hbac.test_run_facet = function(spec) { return that; }; +IPA.hbac.validation_dialog = function(spec) { + + spec = spec || {}; + spec.title = spec.title || IPA.messages.dialogs.validation_title; + spec.message = spec.message || IPA.messages.dialogs.validation_message; + + var that = IPA.message_dialog(spec); + + that.validation_results = spec.validation_results; + + that.create = function() { + + if (that.message) { + that.message_dialog_create(); + } + + if (that.validation_results && that.validation_results.invalid_facets) { + var invalid_facets = that.validation_results.invalid_facets; + + var ul; + + if (invalid_facets.length > 0) { + var div = $('
',{ + text: IPA.messages.objects.hbactest.missing_values + }).appendTo(that.container); + ul = $('