AMBARI-13000. Disable Ranger Hive Plugin is not reverting all the required properties (alexantonenko)

ambari-web/app/controllers/wizard.js

@@ -43,7 +43,8 @@ App.WizardController = Em.Controller.extend(App.LocalStorage, App.ThemesMappingM
     'allHostNames',
     'installOptions',
     'allHostNamesPattern',
-    'serviceComponents'
+    'serviceComponents',
+    'fileNamesToUpdate'
   ],
 
   sensibleConfigs: [
@@ -883,7 +884,7 @@ App.WizardController = Em.Controller.extend(App.LocalStorage, App.ThemesMappingM
    */
   saveServiceConfigProperties: function (stepController) {
     var serviceConfigProperties = [];
-    var fileNamesToUpdate = [];
+    var fileNamesToUpdate = this.getDBProperty('fileNamesToUpdate') || [];
     var installedServiceNames = stepController.get('installedServiceNames') || [];
     var installedServiceNamesMap = {};
     var notAllowed = ['masterHost', 'masterHosts', 'slaveHosts', 'slaveHost'];

ambari-web/app/mixins/common/configs/enhanced_configs.js

@@ -562,6 +562,9 @@ App.EnhancedConfigsMixin = Em.Mixin.create({
         Em.keys(attributes).forEach(function (attributeName) {
           if (attributeName == 'delete') {
             if (!updateOnlyBoundaries) {
+              var fileName = App.config.getOriginalFileName(siteName);
+              var modifiedFileNames = self.get('modifiedFileNames');
+              var wizardController = self.get('wizardController');
               var dependentProperty = self.get('_dependentConfigValues').filterProperty('propertyName', propertyName).filterProperty('fileName', siteName).findProperty('configGroup', group && Em.get(group,'name'));
               if (dependentProperty) {
                 Em.set(dependentProperty, 'toDelete', true);
@@ -585,6 +588,15 @@ App.EnhancedConfigsMixin = Em.Mixin.create({
                   recommendedValue: null
                 });
               }
+              if (modifiedFileNames && !modifiedFileNames.contains(fileName)) {
+               modifiedFileNames.push(fileName);
+              } else if (wizardController) {
+                var fileNamesToUpdate = wizardController.getDBProperty('fileNamesToUpdate') || [];
+                if (!fileNamesToUpdate.contains(fileName)) {
+                  fileNamesToUpdate.push(fileName);
+                  wizardController.setDBProperty('fileNamesToUpdate', fileNamesToUpdate);
+                }
+              }
             }
           } else if (stackProperty) {
             if (selectedConfigGroup) {

ambari-web/app/views/common/configs/service_configs_by_category_view.js

@@ -580,7 +580,18 @@ App.ServiceConfigsByCategoryView = Em.View.extend(App.UserPref, App.ConfigOverri
     }
     // push config's file name if this config was stored on server
     if (!serviceConfigProperty.get('isNotSaved')) {
-      this.get('controller').get('modifiedFileNames').push(serviceConfigProperty.get('filename'));
+      var modifiedFileNames = this.get('controller.modifiedFileNames'),
+        wizardController = this.get('controller.wizardController'),
+        filename = serviceConfigProperty.get('filename');
+      if (modifiedFileNames && !modifiedFileNames.contains(filename)) {
+        modifiedFileNames.push(serviceConfigProperty.get('filename'));
+      } else if (wizardController) {
+        var fileNamesToUpdate = wizardController.getDBProperty('fileNamesToUpdate') || [];
+        if (!fileNamesToUpdate.contains(filename)) {
+          fileNamesToUpdate.push(filename);
+          wizardController.setDBProperty('fileNamesToUpdate', fileNamesToUpdate);
+        }
+      }
     }
     Em.$('body>.tooltip').remove(); //some tooltips get frozen when their owner's DOM element is removed
   },

ambari-web/test/controllers/wizard_test.js

@@ -914,11 +914,13 @@ describe('App.WizardController', function () {
     beforeEach(function () {
       c.set('content', {});
       sinon.stub(c, 'setDBProperties', Em.K);
+      sinon.stub(c, 'getDBProperty').withArgs('fileNamesToUpdate').returns([]);
       sinon.stub(App.config, 'shouldSupportFinal').returns(true);
     });
 
     afterEach(function () {
       c.setDBProperties.restore();
+      c.getDBProperty.restore();
       App.config.shouldSupportFinal.restore();
     });