diff options
Diffstat (limited to 'install/ui/details.js')
| -rw-r--r-- | install/ui/details.js | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/install/ui/details.js b/install/ui/details.js index 022b005b..98f48d0f 100644 --- a/install/ui/details.js +++ b/install/ui/details.js @@ -374,7 +374,7 @@ IPA.details_facet = function(spec) { icon: 'reset-icon', 'class': 'details-reset action-button-disabled', click: function() { - if(!that.update_button.hasClass('action-button-disabled')) { + if (!that.update_button.hasClass('action-button-disabled')) { that.reset(); } return false; @@ -387,9 +387,19 @@ IPA.details_facet = function(spec) { icon: 'update-icon', 'class': 'details-update action-button-disabled', click: function() { - if(!that.update_button.hasClass('action-button-disabled')) { - that.update(); + if (that.update_button.hasClass('action-button-disabled')) return false; + + if (!that.validate()) { + var dialog = IPA.message_dialog({ + title: IPA.messages.dialogs.validation_title, + message: IPA.messages.dialogs.validation_message + }); + dialog.open(); + return false; } + + that.update(); + return false; } }).appendTo(that.controls); @@ -628,15 +638,6 @@ IPA.details_facet = function(spec) { on_error: on_error }); - if (!that.validate()) { - var dialog = IPA.message_dialog({ - title: IPA.messages.dialogs.validation_title, - message: IPA.messages.dialogs.validation_message - }); - dialog.open(); - return; - } - var record = {}; that.save(record); |