From 1a66a66aabbf02e515e10d43513c62fd39371e77 Mon Sep 17 00:00:00 2001
From: Matthias Hochmeister <matthias.hochmeister@apple.com>
Date: Fri, 27 Mar 2026 17:43:31 +0100
Subject: [PATCH] fix: guard vendor detail loading on instead of isPending to
 avoid stuck skeleton

---
 frontend/src/pages/LieferantDetail.tsx | 25 ++++++++++++-------------
 1 file changed, 12 insertions(+), 13 deletions(-)

diff --git a/frontend/src/pages/LieferantDetail.tsx b/frontend/src/pages/LieferantDetail.tsx
index ee39973..7e2b42b 100644
--- a/frontend/src/pages/LieferantDetail.tsx
+++ b/frontend/src/pages/LieferantDetail.tsx
@@ -58,7 +58,7 @@ export default function LieferantDetail() {
   }, [isNew]);
 
   // ── Query ──
-  const { data: vendor, isPending, isLoading, isError } = useQuery({
+  const { data: vendor, isError } = useQuery({
     queryKey: ['lieferant', vendorId],
     queryFn: () => bestellungApi.getVendor(vendorId),
     enabled: !isNew && !!vendorId,
@@ -138,7 +138,17 @@ export default function LieferantDetail() {
   }
 
   // ── Loading / Error ──
-  if (!isNew && isPending) {
+  if (!isNew && !vendor) {
+    if (isError) {
+      return (
+        <DashboardLayout>
+          <Box sx={{ p: 4, textAlign: 'center' }}>
+            <Typography color="error">Lieferant nicht gefunden.</Typography>
+            <Button sx={{ mt: 2 }} onClick={() => navigate('/bestellungen?tab=1')}>Zurück</Button>
+          </Box>
+        </DashboardLayout>
+      );
+    }
     return (
       <DashboardLayout>
         <Box sx={{ display: 'flex', alignItems: 'center', gap: 2, mb: 3 }}>
@@ -154,17 +164,6 @@ export default function LieferantDetail() {
     );
   }
 
-  if (!isNew && !isPending && (isError || !vendor)) {
-    return (
-      <DashboardLayout>
-        <Box sx={{ p: 4, textAlign: 'center' }}>
-          <Typography color="error">Lieferant nicht gefunden.</Typography>
-          <Button sx={{ mt: 2 }} onClick={() => navigate('/bestellungen?tab=1')}>Zurück</Button>
-        </Box>
-      </DashboardLayout>
-    );
-  }
-
   const isSaving = createVendor.isPending || updateVendor.isPending;
 
   return (