From e3ad6e5428415286eb1b3d450a4bd3cc24d11d1f Mon Sep 17 00:00:00 2001
From: tshimber <tatiana.shimbereva@telekom.de>
Date: Fri, 16 Aug 2024 18:10:00 +0200
Subject: [PATCH] fix: review fix

---
 .../src/components/data-grid/data-grid.tsx         | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/packages/components/src/components/data-grid/data-grid.tsx b/packages/components/src/components/data-grid/data-grid.tsx
index d100ee5eb1..57c8ebf395 100644
--- a/packages/components/src/components/data-grid/data-grid.tsx
+++ b/packages/components/src/components/data-grid/data-grid.tsx
@@ -225,7 +225,7 @@ export class DataGrid {
     }
     this.parseRows();
     this.setInitialRowProps();
-    this.presortIfNeeded();
+    this.presortTable();
     this.dataNeedsCheck = true;
     // Set flag to dirty to redo column width with new data
     this.needsAutoWidthParse = true;
@@ -429,12 +429,6 @@ export class DataGrid {
     this.sortTable(newSortDirection, type, columnIndex);
   }
 
-  presortTable(sortDirection, columnIndex, type): void {
-    this.activeSortingIndex = columnIndex;
-    this.fields[columnIndex].sortDirection = sortDirection;
-    this.sortTable(sortDirection, type, columnIndex);
-  }
-
   sortTable(sortDirection, type, columnIndex) {
     const format = this.fields[columnIndex].format;
     if (sortDirection === 'none') {
@@ -503,7 +497,7 @@ export class DataGrid {
     this.activeSortingIndex = -1;
   }
 
-  presortIfNeeded(): void {
+  presortTable(): void {
     const columnToPresort = this.fields.find(
       (col) => col.sortable && col.presort
     );
@@ -515,7 +509,9 @@ export class DataGrid {
       columnToPresort.presortDirection === 'descending'
         ? 'descending'
         : 'ascending';
-    this.presortTable(direction, columnIndex, columnToPresort.type);
+    this.activeSortingIndex = columnIndex;
+    this.fields[columnIndex].sortDirection = direction;
+    this.sortTable(direction, columnToPresort.type, columnIndex);
   }
 
   // Column resize handlers