Use angular.copy to keep a current copy of the config

Author: jeff-phillips-18

src/filters/filter-fields-component.js

@@ -72,12 +72,21 @@ angular.module('patternfly.filters').component('pfFilterFields', {
     }
 
     function setupConfig () {
-      prevConfig = ctrl.config;
+      var fieldFound = false;
+
+      prevConfig = angular.copy(ctrl.config);
 
       if (ctrl.config.fields === undefined) {
         ctrl.config.fields = [];
       }
-      if (!ctrl.currentField || ctrl.config.fields.indexOf((ctrl.currentField) === -1)) {
+
+      if (ctrl.currentField) {
+        fieldFound = _.find(ctrl.config.fields, function (nextField) {
+          return nextField.id === ctrl.currentField.id;
+        });
+      }
+
+      if (!fieldFound) {
         ctrl.currentField = ctrl.config.fields[0];
         ctrl.currentValue = null;
       }

src/filters/filter-results-component.js

@@ -53,7 +53,7 @@ angular.module('patternfly.filters').component('pfFilterResults', {
     };
 
     function setupConfig () {
-      prevConfig = ctrl.config;
+      prevConfig = angular.copy(ctrl.config);
 
       if (!ctrl.config.appliedFilters) {
         ctrl.config.appliedFilters = [];

src/sort/sort-component.js

@@ -31,7 +31,7 @@ angular.module('patternfly.sort').component('pfSort', {
     function setupConfig () {
       var updated = false;
 
-      prevConfig = ctrl.config;
+      prevConfig = angular.copy(ctrl.config);
 
       if (ctrl.config.fields === undefined) {
         ctrl.config.fields = [];

src/toolbars/toolbar-component.js

@@ -34,7 +34,7 @@ angular.module('patternfly.toolbars').component('pfToolbar', {
     };
 
     function setupConfig () {
-      prevConfig = ctrl.config;
+      prevConfig = angular.copy(ctrl.config);
 
       if (ctrl.config && ctrl.config.viewsConfig && ctrl.config.viewsConfig.views) {
         ctrl.config.viewsConfig.viewsList = angular.copy(ctrl.config.viewsConfig.views);