AMBARI-13042 Manage Config Groups: revert changes takes too much time. (atkach)

ambari-web/app/controllers/main/service/info/configs.js

@@ -51,7 +51,7 @@ App.MainServiceInfoConfigsController = Em.Controller.extend(App.ConfigsLoader, A
    */
   configGroups: function() {
     return this.get('groupsStore').filterProperty('serviceName', this.get('content.serviceName'));
-  }.property('content.serviceName', 'groupsStore.length', 'groupStore.@each.name'),
+  }.property('content.serviceName', 'groupsStore'),
 
   dependentConfigGroups: function() {
     if (this.get('dependentServiceNames.length') === 0) return [];

ambari-web/app/controllers/main/service/manage_config_groups_controller.js

@@ -94,6 +94,12 @@ App.ManageConfigGroupsController = Em.Controller.extend(App.ConfigOverridable, {
    */
   clusterHosts: [],
 
+  /**
+   * trigger <code>selectDefaultGroup</code> after group delete
+   * @type {null}
+   */
+  groupDeleteTrigger: null,
+
   /**
    * List of available service components for <code>serviceName</code>
    * @type {{componentName: string, displayName: string, selected: boolean}[]}
@@ -560,6 +566,7 @@ App.ManageConfigGroupsController = Em.Controller.extend(App.ConfigOverridable, {
     this.deleteHosts();
     this.get('configGroups').removeObject(selectedConfigGroup);
     this.set('selectedConfigGroup', this.get('configGroups').findProperty('isDefault'));
+    this.propertyDidChange('groupDeleteTrigger');
   },
 
   /**
@@ -769,20 +776,25 @@ App.ManageConfigGroupsController = Em.Controller.extend(App.ConfigOverridable, {
       },
 
       onClose: function () {
-        this.resetGroupChanges();
+        //<code>_super</code> has to be called before <code>resetGroupChanges</code>
+        var originalGroups = this.get('subViewController.originalConfigGroups').slice(0);
         this._super();
+        this.resetGroupChanges(originalGroups);
       },
 
       onSecondary: function () {
-        this.resetGroupChanges();
-        this._super();
+        this.onClose();
       },
 
-      resetGroupChanges: function () {
+      /**
+       * reset group changes made by user
+       * @param {Array} originalGroups
+       */
+      resetGroupChanges: function (originalGroups) {
         if (this.get('subViewController.isHostsModified')) {
           App.ServiceConfigGroup.find().clear();
           App.store.commit();
-          App.store.loadMany(App.ServiceConfigGroup, this.get('subViewController.originalConfigGroups'));
+          App.store.loadMany(App.ServiceConfigGroup, originalGroups);
           App.store.commit();
         }
       },

ambari-web/app/mixins/main/service/configs/config_overridable.js

@@ -135,27 +135,27 @@ App.ConfigOverridable = Em.Mixin.create({
         } else {
           var newConfigGroupName = this.get('newConfigGroupName').trim();
           var newConfigGroup = {
-            id: null,
+            id: serviceName + "_NEW_" + configGroups.length,
             name: newConfigGroupName,
             description: Em.I18n.t('config.group.description.default').format(new Date().toDateString()),
             service_id: serviceId,
             hosts: [],
             desired_configs: []
           };
+          App.store.load(App.ServiceConfigGroup, newConfigGroup);
+          App.store.commit();
           if (!isInstaller) {
             self.postNewConfigurationGroup(newConfigGroup);
           }
-          if (newConfigGroup) {
-            newConfigGroup.set('parentConfigGroup', configGroups.findProperty('isDefault'));
-            configGroups.pushObject(newConfigGroup);
-            if (isInstaller) {
-              self.persistConfigGroups();
-            } else {
-              self.saveGroupConfirmationPopup(newConfigGroupName);
-            }
-            this.hide();
-            callback(newConfigGroup);
+          newConfigGroup = App.ServiceConfigGroup.find(newConfigGroup.id);
+          configGroups.pushObject(newConfigGroup);
+          if (isInstaller) {
+            self.persistConfigGroups();
+          } else {
+            self.saveGroupConfirmationPopup(newConfigGroupName);
           }
+          this.hide();
+          callback(newConfigGroup);
         }
       },
       onSecondary: function () {
@@ -235,6 +235,7 @@ App.ConfigOverridable = Em.Mixin.create({
     var sendData = {
       name: 'config_groups.create',
       data: {
+        'mock_id': newConfigGroupData.id,
         'group_name': newConfigGroupData.name,
         'service_id': newConfigGroupData.service_id,
         'description': newConfigGroupData.description,
@@ -243,6 +244,10 @@ App.ConfigOverridable = Em.Mixin.create({
       success: 'successFunction',
       error: 'errorFunction',
       successFunction: function (response, opt, params) {
+        var configGroupData = App.router.get('manageConfigGroupsController').generateOriginalConfigGroups([App.ServiceConfigGroup.find(params.mock_id)]);
+        App.configGroupsMapper.deleteRecord(App.ServiceConfigGroup.find(params.mock_id));
+        configGroupData[0].id = response.resources[0].ConfigGroup.id;
+        App.store.load(App.ServiceConfigGroup, configGroupData[0]);
         App.ServiceConfigGroup.find().clear();
         if (callback) {
           callback();

ambari-web/app/views/main/service/manage_config_groups_view.js

@@ -134,16 +134,6 @@ App.MainServiceManageConfigGroupView = Em.View.extend({
     this.set('controller.selectedHosts', []);
   }.observes('selectedConfigGroup'),
 
-  /**
-   * Select first config group after all groups are loaded
-   * @method onLoad
-   */
-  onLoad: function () {
-    if (this.get('controller.isLoaded')) {
-      this.set('selectedConfigGroup', this.get('controller.configGroups')[0])
-    }
-  }.observes('controller.isLoaded', 'controller.configGroups'),
-
   /**
    * Select default config group after all config groups are loaded
    * @method selectDefaultGroup
@@ -152,6 +142,6 @@ App.MainServiceManageConfigGroupView = Em.View.extend({
     if (this.get('controller.isLoaded')) {
       this.set('selectedConfigGroup', [this.get('controller.configGroups').findProperty('isDefault')]);
     }
-  }.observes('controller.isLoaded')
+  }.observes('controller.isLoaded', 'controller.groupDeleteTrigger')
 
 });

ambari-web/test/controllers/main/service/manage_config_groups_controller_test.js

@@ -183,6 +183,7 @@ describe('App.ManageConfigGroupsController', function() {
 
       c.deleteConfigGroup();
 
+      expect(c.get('configGroups.length')).to.equal(1);
       expect(c.get('selectedConfigGroup.hosts')).to.include.members(['h1','h2','h3']);
       expect(c.get('selectedConfigGroup.isDefault')).to.be.true;
     });