From 8a41247432c299919b3350482f9e60764b6fc9ca Mon Sep 17 00:00:00 2001
From: Jannis Mattheis <contact@jmattheis.de>
Date: Thu, 2 Jan 2025 17:28:49 +0100
Subject: [PATCH] fix: unify no data check for dashboard entries

---
 ui/src/dashboard/Entry/DashboardEntry.tsx | 54 +++++------------------
 1 file changed, 10 insertions(+), 44 deletions(-)

diff --git a/ui/src/dashboard/Entry/DashboardEntry.tsx b/ui/src/dashboard/Entry/DashboardEntry.tsx
index de67341..4822d27 100644
--- a/ui/src/dashboard/Entry/DashboardEntry.tsx
+++ b/ui/src/dashboard/Entry/DashboardEntry.tsx
@@ -67,63 +67,29 @@ const SpecificDashboardEntry: React.FC<{entry: Dashboards_dashboards_items; rang
             </Center>
         );
     }
+    const firstEntries: Stats_stats_entries[] =
+        (stats.data && stats.data.stats && stats.data.stats[0] && stats.data.stats[0].entries) || [];
+    if (firstEntries.length === 0) {
+        return (
+            <Center>
+                <Typography>no data</Typography>
+            </Center>
+        );
+    }
 
     const entries = (stats.data && stats.data.stats) || [];
     switch (entry.entryType) {
         case EntryType.PieChart:
-            const data: Stats_stats_entries[] = (stats.data && stats.data.stats && stats.data.stats[0].entries) || [];
-            if (data.length === 0) {
-                return (
-                    <Center>
-                        <Typography>no data</Typography>
-                    </Center>
-                );
-            }
-            return <DashboardPieChart entries={data} />;
+            return <DashboardPieChart entries={firstEntries} />;
         case EntryType.BarChart:
-            if (entries.length === 0) {
-                return (
-                    <Center>
-                        <Typography>no data</Typography>
-                    </Center>
-                );
-            }
             return <DashboardBarChart entries={entries} interval={interval} type="normal" total={entry.total} />;
         case EntryType.StackedBarChart:
-            if (entries.length === 0) {
-                return (
-                    <Center>
-                        <Typography>no data</Typography>
-                    </Center>
-                );
-            }
             return <DashboardBarChart entries={entries} interval={interval} type="stacked" total={entry.total} />;
         case EntryType.LineChart:
-            if (entries.length === 0) {
-                return (
-                    <Center>
-                        <Typography>no data</Typography>
-                    </Center>
-                );
-            }
             return <DashboardLineChart entries={entries} interval={interval} total={entry.total} />;
         case EntryType.VerticalTable:
-            if (entries.length === 0) {
-                return (
-                    <Center>
-                        <Typography>no data</Typography>
-                    </Center>
-                );
-            }
             return <DashboardTable mode="vertical" entries={entries} interval={interval} total={entry.total} />;
         case EntryType.HorizontalTable:
-            if (entries.length === 0) {
-                return (
-                    <Center>
-                        <Typography>no data</Typography>
-                    </Center>
-                );
-            }
             return <DashboardTable mode="horizontal" entries={entries} interval={interval} total={entry.total} />;
         default:
             return expectNever(entry.entryType);