Fix initial dataset stacks (#11882)

* Update cahced _stacked value after updating stacks * Add test --------- Co-authored-by: Jacco van den Berg <[email protected]>
chartjs · Aug 26, 2024 · 680cd22 · 680cd22
1 parent dd554e0
commit 680cd22
Show file tree

Hide file tree

Showing 4 changed files with 52 additions and 0 deletions.
diff --git a/src/core/core.datasetController.js b/src/core/core.datasetController.js
@@ -420,6 +420,7 @@ export default class DatasetController {
     // if stack changed, update stack values for the whole dataset
     if (stackChanged || oldStacked !== meta._stacked) {
       updateStacks(this, meta._parsed);
+      meta._stacked = isStacked(meta.vScale, meta);
     }
   }
 

diff --git a/test/fixtures/core.datasetController/stacked-initial-render.js b/test/fixtures/core.datasetController/stacked-initial-render.js
@@ -0,0 +1,49 @@
+module.exports = {
+  config: {
+    type: 'line',
+    data: {
+      labels: [0, 1, 2, 3, 4, 5, 6],
+      datasets: [
+        {
+          // option in dataset
+          data: [9, 13, 15, 25, 22, 15, 21],
+          stack: 'construction_stack',
+          borderWidth: 10,
+          borderColor: 'rgb(54, 162, 235)'
+        },
+        {
+          data: [9, 13, 15, 25, 22, 15, 21],
+          stack: 'construction_stack',
+          borderWidth: 10,
+          borderColor: 'rgb(255, 99, 132)'
+        }
+      ]
+    },
+    options: {
+      scales: {
+        x: {
+          ticks: {
+            display: false
+          }
+        },
+        y: {
+          ticks: {
+            display: false
+          }
+        }
+      },
+      plugins: {
+        legend: false,
+        title: false,
+        tooltip: false,
+        filler: false
+      }
+    }
+  },
+  options: {
+    canvas: {
+      height: 512,
+      width: 512
+    }
+  }
+};
diff --git a/test/fixtures/core.datasetController/stacked-initial-render.png b/test/fixtures/core.datasetController/stacked-initial-render.png
diff --git a/test/specs/core.datasetController.tests.js b/test/specs/core.datasetController.tests.js
@@ -1,4 +1,6 @@
 describe('Chart.DatasetController', function() {
+  describe('auto', jasmine.fixture.specs('core.datasetController'));
+
   it('should listen for dataset data insertions or removals', function() {
     var data = [0, 1, 2, 3, 4, 5];
     var chart = acquireChart({