You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ak...@apache.org on 2016/12/12 19:05:27 UTC

ambari git commit: AMBARI-19169. Add Service Page in Atlas doesn't show config on change (akovalenko)

Repository: ambari
Updated Branches:
  refs/heads/trunk e4db57093 -> b69bcc1f2


AMBARI-19169. Add Service Page in Atlas doesn't show config on change (akovalenko)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/b69bcc1f
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/b69bcc1f
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/b69bcc1f

Branch: refs/heads/trunk
Commit: b69bcc1f2d14bff050571bc165110a59e701dd83
Parents: e4db570
Author: Aleksandr Kovalenko <ak...@hortonworks.com>
Authored: Mon Dec 12 18:52:57 2016 +0200
Committer: Aleksandr Kovalenko <ak...@hortonworks.com>
Committed: Mon Dec 12 19:57:21 2016 +0200

----------------------------------------------------------------------
 .../views/common/configs/service_configs_by_category_view.js   | 6 +++++-
 .../app/views/common/configs/widgets/config_widget_view.js     | 2 ++
 2 files changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/b69bcc1f/ambari-web/app/views/common/configs/service_configs_by_category_view.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/views/common/configs/service_configs_by_category_view.js b/ambari-web/app/views/common/configs/service_configs_by_category_view.js
index 16258cf..3ebc2cf 100644
--- a/ambari-web/app/views/common/configs/service_configs_by_category_view.js
+++ b/ambari-web/app/views/common/configs/service_configs_by_category_view.js
@@ -107,7 +107,11 @@ App.ServiceConfigsByCategoryView = Em.View.extend(App.UserPref, App.ConfigOverri
   },
 
   setVisibleCategoryConfigsOnce: function () {
-    Em.run.once(this, 'addConfigToCategoryConfigs');
+    if (this.get('controller.isChangingConfigAttributes')) {
+      this.setVisibleCategoryConfigs();
+    } else {
+      Em.run.once(this, 'addConfigToCategoryConfigs');
+    }
   }.observes('categoryConfigsAll.@each.isVisible'),
 
   setCategoryConfigsAll: function () {

http://git-wip-us.apache.org/repos/asf/ambari/blob/b69bcc1f/ambari-web/app/views/common/configs/widgets/config_widget_view.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/views/common/configs/widgets/config_widget_view.js b/ambari-web/app/views/common/configs/widgets/config_widget_view.js
index 62b6057..2c90cc3 100644
--- a/ambari-web/app/views/common/configs/widgets/config_widget_view.js
+++ b/ambari-web/app/views/common/configs/widgets/config_widget_view.js
@@ -529,6 +529,7 @@ App.ConfigWidgetView = Em.View.extend(App.SupportsDependentConfigs, App.WidgetPo
     var serviceConfigs = this.get('controller.stepConfigs').findProperty('serviceName',serviceName).get('configs');
     var action = isConditionTrue ? configCondition.get("then") : configCondition.get("else");
     var valueAttributes = action.property_value_attributes;
+    this.set('controller.isChangingConfigAttributes', true);
     for (var key in valueAttributes) {
       if (valueAttributes.hasOwnProperty(key)) {
         var valueAttribute = App.StackConfigValAttributesMap[key] || key;
@@ -541,6 +542,7 @@ App.ConfigWidgetView = Em.View.extend(App.SupportsDependentConfigs, App.WidgetPo
         }
       }
     }
+    this.set('controller.isChangingConfigAttributes', false);
   },
 
   /**