From 455a061d090d75f9e39a15a65b9cf0089c65fa90 Mon Sep 17 00:00:00 2001 From: "Dardenne Florent (dafl)" Date: Wed, 3 Aug 2022 13:09:38 +0000 Subject: [PATCH] [FIX] web: stacked bar is properly displayed Stacked bar does not support the stacked option in the y axes. This option was introduced in the stacked line generalization and was set to False if the user is in the bar mode. closes odoo/odoo#97410 Signed-off-by: Simon Genin (ges@odoo) --- addons/web/static/src/views/graph/graph_renderer.js | 2 +- addons/web/static/tests/views/graph_view_tests.js | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/addons/web/static/src/views/graph/graph_renderer.js b/addons/web/static/src/views/graph/graph_renderer.js index 9b480482227..8bf2b352e1c 100644 --- a/addons/web/static/src/views/graph/graph_renderer.js +++ b/addons/web/static/src/views/graph/graph_renderer.js @@ -439,7 +439,7 @@ export class GraphRenderer extends Component { suggestedMax: 0, suggestedMin: 0, }, - stacked: mode === "line" && stacked, + stacked: mode === "line" && stacked ? stacked : undefined, }; return { xAxes: [xAxe], yAxes: [yAxe] }; } diff --git a/addons/web/static/tests/views/graph_view_tests.js b/addons/web/static/tests/views/graph_view_tests.js index 1b6865c0b40..add22121e76 100644 --- a/addons/web/static/tests/views/graph_view_tests.js +++ b/addons/web/static/tests/views/graph_view_tests.js @@ -916,9 +916,9 @@ QUnit.module("Views", (hooks) => { "graph should be a classic line chart." ); assert.strictEqual( - getScaleY(graph).every((y) => y.stacked), - false, - "The y axes should have stacked property set to false" + getScaleY(graph).every((y) => y.stacked == undefined), + true, + "The y axes should have stacked property set to undefined" ); });