Skip to content

Commit

Permalink
handle case with number + category split in wrong order
Browse files Browse the repository at this point in the history
  • Loading branch information
adrianmroz-allegro committed Dec 5, 2022
1 parent 580ba0e commit ddd167c
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion src/common/visualization-manifests/line-chart/line-chart.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,13 +90,27 @@ const rulesEvaluator = visualizationDependentEvaluatorBuilder

const colorSplit = splits.getSplit(1);
const colorDimension = findDimensionByName(dataCube.dimensions, colorSplit.reference);

const newColorSplit = adjustColorSplit(colorSplit, colorDimension, series, appSettings.customization.visualizationColors);

return Resolve.automatic(8, {
splits: new Splits({ splits: List([newColorSplit, newTimeSplit]) })
});
})
.when(Predicates.areExactSplitKinds("number", "*"))
.then(({ splits, series, dataCube, appSettings }) => {
const numberSplit = splits.getSplit(0);
const dimension = findDimensionByName(dataCube.dimensions, numberSplit.reference);

const newNumberSplit = fixNumberSplit(numberSplit, dimension);

const colorSplit = splits.getSplit(1);
const colorDimension = findDimensionByName(dataCube.dimensions, colorSplit.reference);
const newColorSplit = adjustColorSplit(colorSplit, colorDimension, series, appSettings.customization.visualizationColors);

return Resolve.automatic(8, {
splits: new Splits({ splits: List([newColorSplit, newNumberSplit]) })
});
})

.when(Predicates.areExactSplitKinds("*", "time"))
.then(({ splits, series, dataCube, appSettings }) => {
Expand Down

0 comments on commit ddd167c

Please sign in to comment.