Remove sort by column if column is removed

elastic · May 9, 2020 · 7d8a8bb · 7d8a8bb
1 parent 4571536
commit 7d8a8bb
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/src/plugins/discover/public/application/angular/discover.js b/src/plugins/discover/public/application/angular/discover.js
@@ -559,7 +559,8 @@ function discoverController(
       getSortForSearchSource(
         $scope.state.sort,
         $scope.indexPattern,
-        config.get('discover:sort:defaultOrder')
+        config.get('discover:sort:defaultOrder'),
+        $scope.state.columns
       )
     );
     searchSource.setField('highlight', null);
@@ -583,24 +584,25 @@ function discoverController(
   };
 
   function getStateDefaults() {
+    const columns =
+      savedSearch.columns.length > 0 ? savedSearch.columns : config.get('defaultColumns').slice();
     const query =
       $scope.searchSource.getField('query') ||
       getDefaultQuery(
         localStorage.get('kibana.userQueryLanguage') || config.get('search:queryLanguage')
       );
     return {
       query,
-      sort: getSortArray(savedSearch.sort, $scope.indexPattern),
-      columns:
-        savedSearch.columns.length > 0 ? savedSearch.columns : config.get('defaultColumns').slice(),
+      sort: getSortArray(savedSearch.sort, $scope.indexPattern, columns),
+      columns,
       index: $scope.indexPattern.id,
       interval: 'auto',
       filters: _.cloneDeep($scope.searchSource.getOwnField('filter')),
     };
   }
 
   $scope.state.index = $scope.indexPattern.id;
-  $scope.state.sort = getSortArray($scope.state.sort, $scope.indexPattern);
+  $scope.state.sort = getSortArray($scope.state.sort, $scope.indexPattern, $scope.columns);
 
   $scope.getBucketIntervalToolTipText = () => {
     return i18n.translate('discover.bucketIntervalTooltip', {
@@ -943,7 +945,8 @@ function discoverController(
         getSortForSearchSource(
           $scope.state.sort,
           indexPattern,
-          config.get('discover:sort:defaultOrder')
+          config.get('discover:sort:defaultOrder'),
+          $scope.state.columns
         )
       )
       .setField('query', $scope.state.query || null)
@@ -977,7 +980,8 @@ function discoverController(
   $scope.removeColumn = function removeColumn(columnName) {
     $scope.indexPattern.popularizeField(columnName, 1);
     const columns = columnActions.removeColumn($scope.state.columns, columnName);
-    setAppState({ columns });
+    const sort = $scope.state.sort.filter(s => s[0] !== columnName);
+    setAppState({ columns, sort });
   };
 
   $scope.moveColumn = function moveColumn(columnName, newIndex) {