From ec322dfd8d735a929d128030d144ef4d120f88e3 Mon Sep 17 00:00:00 2001 From: SBIN2010 Date: Wed, 24 Sep 2025 17:44:01 +0300 Subject: [PATCH] fix(Mixed Chart): Tooltip incorrectly displays numbers with optional Y-axis format and showQueryIdentifiers set to true (#35224) --- .../src/MixedTimeseries/transformProps.ts | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/superset-frontend/plugins/plugin-chart-echarts/src/MixedTimeseries/transformProps.ts b/superset-frontend/plugins/plugin-chart-echarts/src/MixedTimeseries/transformProps.ts index 86ead62ed59..b25245fd9b8 100644 --- a/superset-frontend/plugins/plugin-chart-echarts/src/MixedTimeseries/transformProps.ts +++ b/superset-frontend/plugins/plugin-chart-echarts/src/MixedTimeseries/transformProps.ts @@ -322,12 +322,6 @@ export default function transformProps( primarySeries.add(seriesOption.id as string); } }; - rawSeriesA.forEach(seriesOption => - mapSeriesIdToAxis(seriesOption, yAxisIndex), - ); - rawSeriesB.forEach(seriesOption => - mapSeriesIdToAxis(seriesOption, yAxisIndexB), - ); const showValueIndexesA = extractShowValueIndexes(rawSeriesA, { stack, onlyTotal, @@ -460,7 +454,11 @@ export default function transformProps( theme, }, ); - if (transformedSeries) series.push(transformedSeries); + + if (transformedSeries) { + series.push(transformedSeries); + mapSeriesIdToAxis(transformedSeries, yAxisIndex); + } }); rawSeriesB.forEach(entry => { @@ -528,7 +526,11 @@ export default function transformProps( theme, }, ); - if (transformedSeries) series.push(transformedSeries); + + if (transformedSeries) { + series.push(transformedSeries); + mapSeriesIdToAxis(transformedSeries, yAxisIndexB); + } }); // default to 0-100% range when doing row-level contribution chart