You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by al...@apache.org on 2015/07/21 16:27:16 UTC

[2/2] ambari git commit: AMBARI-12470. Can't switch config group on the host's configs page (alexantonenko)

AMBARI-12470. Can't switch config group on the host's configs page (alexantonenko)


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

Branch: refs/heads/trunk
Commit: 82589fc4d060443a88d3709631a9f07edc11e715
Parents: c0a5296
Author: Alex Antonenko <hi...@gmail.com>
Authored: Tue Jul 21 17:24:23 2015 +0300
Committer: Alex Antonenko <hi...@gmail.com>
Committed: Tue Jul 21 17:24:23 2015 +0300

----------------------------------------------------------------------
 ambari-web/app/mixins/main/service/configs/config_overridable.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/82589fc4/ambari-web/app/mixins/main/service/configs/config_overridable.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/main/service/configs/config_overridable.js b/ambari-web/app/mixins/main/service/configs/config_overridable.js
index e1eeb60..3c51c39 100644
--- a/ambari-web/app/mixins/main/service/configs/config_overridable.js
+++ b/ambari-web/app/mixins/main/service/configs/config_overridable.js
@@ -274,6 +274,7 @@ App.ConfigOverridable = Em.Mixin.create({
    * @method updateConfigurationGroup
    */
   updateConfigurationGroup: function (configGroup, successCallback, errorCallback) {
+    var configSiteTags = configGroup.get('configSiteTags') || [];
     var putConfigGroup = {
       ConfigGroup: {
         group_name: configGroup.get('name'),
@@ -284,7 +285,7 @@ App.ConfigOverridable = Em.Mixin.create({
             host_name: h
           };
         }),
-        desired_configs: configGroup.get('configSiteTags').map(function (cst) {
+        desired_configs: configSiteTags.map(function (cst) {
           return {
             type: cst.get('site'),
             tag: cst.get('tag')