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 2017/02/26 23:41:18 UTC

ambari git commit: AMBARI-20201. Config Groups are shown twice after creation (akovalenko)

Repository: ambari
Updated Branches:
  refs/heads/trunk eb00261b8 -> 2fc354e5c


AMBARI-20201. Config Groups are shown twice after creation (akovalenko)


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

Branch: refs/heads/trunk
Commit: 2fc354e5cb0606622b869317ac98babc13d91fd4
Parents: eb00261
Author: Aleksandr Kovalenko <ak...@hortonworks.com>
Authored: Mon Feb 27 00:25:39 2017 +0200
Committer: Aleksandr Kovalenko <ak...@hortonworks.com>
Committed: Mon Feb 27 00:27:13 2017 +0200

----------------------------------------------------------------------
 .../controllers/main/service/manage_config_groups_controller.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/2fc354e5/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/service/manage_config_groups_controller.js b/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
index 0df5dc5..45c840a 100644
--- a/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
+++ b/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
@@ -560,12 +560,15 @@ App.ManageConfigGroupsController = Em.Controller.extend(App.ConfigOverridable, {
    */
   addHostsCallback: function (selectedHosts) {
     if (selectedHosts) {
+      var sortedHosts;
       var group = this.get('selectedConfigGroup');
       var parentGroupHosts = group.get('parentConfigGroup.hosts');
       var newHostsForParentGroup = parentGroupHosts.filter(function(hostName) {
         return !selectedHosts.contains(hostName);
       });
-      group.set('hosts', group.get('hosts').pushObjects(selectedHosts).slice().sort());
+      group.get('hosts').pushObjects(selectedHosts);
+      sortedHosts = group.get('hosts').sort();
+      group.set('hosts', sortedHosts);
       group.set('parentConfigGroup.hosts', newHostsForParentGroup);
     }
   },