Mirrored_Repos/InvenTree
1
0
Fork 0
mirror of https://github.com/inventree/InvenTree synced 2024-08-30 18:33:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix for BOM pricing donut chart (#7917)

Browse Source 
- Mantine charts tooltip can't handle '.' character.
This commit is contained in:
Oliver 2024-08-19 16:33:02 +10:00 committed by GitHub
parent 0ee06ec13e
commit d7d908b74f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/frontend/src/pages/part/pricing/BomPricingPanel.tsx
View File

@ -32,13 +32,18 @@ function BomPieChart({
}) {
// Construct donut data
const maxPricing = useMemo(() => {
return data.map((entry, index) => {
return {
name: entry.name,
value: entry.total_price_max,
color: CHART_COLORS[index % CHART_COLORS.length] + '.5'
};
});
return (
data
?.filter((el: any) => !!el.total_price_max)
.map((entry, index) => {
return {
// Note: Replace '.' in name to avoid issues with tooltip
name: entry?.name?.replace('.', '') ?? '',
value: entry?.total_price_max,
color: CHART_COLORS[index % CHART_COLORS.length] + '.5'
};
}) ?? []
);
}, [data]);
return (
@ -170,8 +175,8 @@ export default function BomPricingPanel({
const [chartType, setChartType] = useState<string>('pie');
const hasData: boolean = useMemo(() => {
return !table.isLoading && bomPricingData.length > 0;
}, [table.isLoading, bomPricingData.length]);
return !table.isLoading && bomPricingData && bomPricingData.length > 0;
}, [table.isLoading, bomPricingData]);
return (
<Stack gap="xs">