Fix indexing in the presence of sorting or filtering

jdavidheiser · Feb 6, 2016 · af183fd · af183fd
1 parent 7a3a5e6
commit af183fd
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/qgrid/grid.py b/qgrid/grid.py
@@ -349,7 +349,6 @@ def _handle_qgrid_msg(self, widget, content, buffers=None):
 
         elif content['type'] == 'selection_change':
             self._selected_rows = content['rows']
-            print(self._selected_rows)
 
     def get_selected_rows(self):
         """Get the currently selected rows"""

diff --git a/qgrid/qgridjs/qgrid.widget.js b/qgrid/qgridjs/qgrid.widget.js
@@ -141,14 +141,21 @@ define([path + "widgets/js/widget", path + "widgets/js/manager"], function(widge
             // set up callbacks
             sgrid.onCellChange.subscribe(function(e, args) {
                 var column = columns[args.cell].name;
-                var msg = {'row': args.row, 'column': column,
+                var id = args.grid.getDataItem(args.row).slick_grid_id;
+                var row = Number(id.replace('row', ''))
+                var msg = {'row': row, 'column': column,
                            'value': args.item[column], 'type': 'cell_change'};
                 that.send(msg);
             });
 
             sgrid.onSelectedRowsChanged.subscribe(function(e, args) {
-                var rows = args.rows;
-                var msg = {'rows': args.rows, 'type': 'selection_change'};
+                var rows = [];
+                var grid = args.grid;
+                for (var r = 0; r < args.rows.length; r++) {
+                    var id = grid.getDataItem(args.rows[r]).slick_grid_id;
+                    rows.push(Number(id.replace('row', '')));
+                }
+                var msg = {'rows': rows, 'type': 'selection_change'};
                 that.send(msg);
             });
         },