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);
}
},