Updated Branches: refs/heads/trunk b5d5576f6 -> bbb767f76
AMBARI-2761. Customize Services page - Misc tab: incorrect behavior of popup window for changing user names. (Antonenko Alexander via yusaku) Project: http://git-wip-us.apache.org/repos/asf/incubator-ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ambari/commit/bbb767f7 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ambari/tree/bbb767f7 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ambari/diff/bbb767f7 Branch: refs/heads/trunk Commit: bbb767f76547b9f97bfc88f761c31917129ede97 Parents: b5d5576 Author: Yusaku Sako <[email protected]> Authored: Mon Jul 29 14:18:03 2013 -0700 Committer: Yusaku Sako <[email protected]> Committed: Mon Jul 29 14:18:24 2013 -0700 ---------------------------------------------------------------------- ambari-web/app/controllers/wizard/step7_controller.js | 6 ++++-- ambari-web/app/views/common/configs/services_config.js | 11 +++++------ 2 files changed, 9 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-ambari/blob/bbb767f7/ambari-web/app/controllers/wizard/step7_controller.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/controllers/wizard/step7_controller.js b/ambari-web/app/controllers/wizard/step7_controller.js index 2d19126..a305b97 100644 --- a/ambari-web/app/controllers/wizard/step7_controller.js +++ b/ambari-web/app/controllers/wizard/step7_controller.js @@ -40,9 +40,11 @@ App.WizardStep7Controller = Em.Controller.extend({ secureConfigs: require('data/secure_mapping'), + miscModalVisible: false, //If miscConfigChange Modal is shown + isSubmitDisabled: function () { - return !this.stepConfigs.filterProperty('showConfig', true).everyProperty('errorCount', 0); - }.property('[email protected]'), + return (!this.stepConfigs.filterProperty('showConfig', true).everyProperty('errorCount', 0) || this.get("miscModalVisible")); + }.property('[email protected]', 'miscModalVisible'), selectedServiceNames: function () { return this.get('content.services').filterProperty('isSelected', true).filterProperty('isInstalled', false).mapProperty('serviceName'); http://git-wip-us.apache.org/repos/asf/incubator-ambari/blob/bbb767f7/ambari-web/app/views/common/configs/services_config.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/views/common/configs/services_config.js b/ambari-web/app/views/common/configs/services_config.js index fb38e19..1580c65 100644 --- a/ambari-web/app/views/common/configs/services_config.js +++ b/ambari-web/app/views/common/configs/services_config.js @@ -51,7 +51,6 @@ App.ServiceConfigsByCategoryView = Ember.View.extend({ classNameBindings: ['category.name', 'isShowBlock::hidden'], content: null, - miscModalVisible: false, //If miscConfigChange Modal is shown category: null, service: null, canEdit: true, // View is editable or read-only? @@ -180,7 +179,7 @@ App.ServiceConfigsByCategoryView = Ember.View.extend({ ); } } - if (this.affectedProperties.length > 0 && !this.get("miscModalVisible")) { + if (this.affectedProperties.length > 0 && !this.get("controller.miscModalVisible")) { this.newAffectedProperties = this.affectedProperties; var self = this; return App.ModalPopup.show({ @@ -192,18 +191,18 @@ App.ServiceConfigsByCategoryView = Ember.View.extend({ self.get("controller.stepConfigs").findProperty("serviceName", item.serviceName).get("configs") .findProperty("name", item.propertyName).set("value", item.newValue); }); - self.set("miscModalVisible", false); + self.get("controller").set("miscModalVisible", false); this.hide(); }, onIgnore: function () { - self.set("miscModalVisible", false); + self.get("controller").set("miscModalVisible", false); this.hide(); }, onUndo: function () { var affected = self.get("newAffectedProperties").objectAt(0); self.get("controller.stepConfigs").findProperty("serviceName", "MISC").get("configs") .findProperty("name", affected.changedPropertyName).set("value", $.trim(affected.curValue)); - self.set("miscModalVisible", false); + self.get("controller").set("miscModalVisible", false); this.hide(); }, footerClass: Ember.View.extend({ @@ -221,7 +220,7 @@ App.ServiceConfigsByCategoryView = Ember.View.extend({ controller: this, propertyChange: self.get("newAffectedProperties"), didInsertElement: function () { - self.set("miscModalVisible", true); + self.get("controller").set("miscModalVisible", true); } }) });
