-
- Downloads
fix wrong data used for click actions in old pivot table (#17937)
In `getTableClickedObjectRowData`, we try to get the original row of the data from the rowIndex but because the rowIndex of the pivoted table and the original table does not necessarily match, we get the wrong data. This commit fixes it by storing a mapping between pivoted (row, col) and the original row number and using that to determine which row to use for the data. Fixes #17920
Showing
- frontend/src/metabase/lib/data_grid.js 5 additions, 0 deletionsfrontend/src/metabase/lib/data_grid.js
- frontend/src/metabase/visualizations/components/TableInteractive.jsx 8 additions, 1 deletion...c/metabase/visualizations/components/TableInteractive.jsx
- frontend/src/metabase/visualizations/components/TableSimple.jsx 3 additions, 0 deletions...nd/src/metabase/visualizations/components/TableSimple.jsx
- frontend/src/metabase/visualizations/lib/table.js 21 additions, 5 deletionsfrontend/src/metabase/visualizations/lib/table.js
- frontend/test/metabase/lib/data_grid.unit.spec.js 17 additions, 0 deletionsfrontend/test/metabase/lib/data_grid.unit.spec.js
- frontend/test/metabase/scenarios/dashboard/dashboard-drill.cy.spec.js 39 additions, 1 deletion...t/metabase/scenarios/dashboard/dashboard-drill.cy.spec.js
- frontend/test/metabase/visualizations/lib/table.unit.spec.js 63 additions, 9 deletionsfrontend/test/metabase/visualizations/lib/table.unit.spec.js
Loading
Please register or sign in to comment