fix: Reset sorting bar issue in Barchart (apache#19371)

* fix: Reset sorting bar issue in Barchart * improving performance * fix lint error
ValtechMobility · Jun 9, 2022 · 36fede4 · 36fede4
1 parent 2244d50
commit 36fede4
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/superset-frontend/plugins/legacy-preset-chart-nvd3/src/NVD3Vis.js b/superset-frontend/plugins/legacy-preset-chart-nvd3/src/NVD3Vis.js
@@ -434,7 +434,11 @@ function nvd3Vis(element, props) {
         chart.stacked(isBarStacked);
         if (orderBars) {
           data.forEach(d => {
-            d.values.sort((a, b) => (tryNumify(a.x) < tryNumify(b.x) ? -1 : 1));
+            const newValues = [...d.values]; // need to copy values to avoid redux store changed.
+            // eslint-disable-next-line no-param-reassign
+            d.values = newValues.sort((a, b) =>
+              tryNumify(a.x) < tryNumify(b.x) ? -1 : 1,
+            );
           });
         }
         if (!reduceXTicks) {