DataGrid - getSelectedRowKeys method returns incorrect keys if deferred selection is enabled (T1280037) (DevExpress#29929)

Author: tongsonbarbs

packages/devextreme/js/__internal/ui/selection/m_selection.strategy.deferred.ts

@@ -69,19 +69,14 @@ export default class DeferredStrategy extends SelectionStrategy {
   }
 
   isItemKeySelected(itemData) {
-    const { selectionFilter, sensitivity } = this.options;
+    const { selectionFilter } = this.options;
 
     if (!selectionFilter) {
       return true;
     }
 
-    const queryParams = {
-      langParams: {
-        collatorOptions: {
-          sensitivity,
-        },
-      },
-    };
+    const queryParams = this._getQueryParams();
+
     // @ts-expect-error
     return !!dataQuery([itemData], queryParams).filter(selectionFilter).toArray().length;
   }

packages/devextreme/js/__internal/ui/selection/m_selection.strategy.ts

@@ -137,13 +137,32 @@ export default class SelectionStrategy {
     return remoteFilter;
   }
 
+  _getQueryParams() {
+    const { sensitivity } = this.options;
+
+    if (!sensitivity) {
+      return;
+    }
+
+    return {
+      langParams: {
+        collatorOptions: {
+          sensitivity,
+        },
+      },
+    };
+  }
+
   _loadFilteredData(remoteFilter, localFilter?: any, select?: any, isSelectAll?: boolean) {
     const filterLength = encodeURI(JSON.stringify(this._removeTemplateProperty(remoteFilter))).length;
     const needLoadAllData = this.options.maxFilterLengthInRequest && (filterLength > this.options.maxFilterLengthInRequest);
     const deferred = Deferred();
+    const queryParams = this._getQueryParams();
+
     const loadOptions = {
       filter: needLoadAllData ? undefined : remoteFilter,
       select: needLoadAllData ? this.options.dataFields() : select || this.options.dataFields(),
+      ...queryParams,
     };
 
     if (remoteFilter && remoteFilter.length === 0) {

packages/devextreme/testing/tests/DevExpress.ui.widgets.dataGrid/selection.tests.js

@@ -4643,6 +4643,33 @@ QUnit.module('Deferred selection', {
         // assert
         assert.deepEqual(this.option('selectionFilter'), []);
     });
+
+    QUnit.test('DataGrid - getSelectedRowKeys method should return correct keys if deferred selection.sensitivity is "variant" (T1280037)', function(assert) {
+        this.setupDataGrid({
+            dataSource: [
+                { ID: 'aaa', Name: 'Name a' },
+                { ID: 'AAA', Name: 'Name A' },
+            ],
+            keyExpr: 'ID',
+            columns: ['ID', 'Name'],
+            showBorders: true,
+            selection: {
+                sensitivity: 'variant',
+                mode: 'multiple',
+                deferred: true,
+            },
+            selectionFilter: ['ID', '=', 'aaa'],
+        });
+
+        let selectedRowKeys = [];
+
+        this.getSelectedRowKeys().done((keys) => {
+            selectedRowKeys = keys;
+        });
+        this.clock.tick();
+
+        assert.deepEqual(selectedRowKeys, ['aaa'], 'selection filter is case sensitive');
+    });
 });
 
 QUnit.module('Selection with virtual scrolling', {