From 91b3870f1c73bff525f95f943dae9eafdf9e5a1a Mon Sep 17 00:00:00 2001
From: Yoga Pangestu <ypangestu976@gmail.com>
Date: Sat, 20 Jun 2026 13:40:19 +0700
Subject: [PATCH] fix: improve error handling in API fetch function to manage
 non-JSON responses and session expiration

---
 .../admin/manage/orders/OrderPosForm.vue      |  5 +--
 resources/js/lib/api.ts                       | 31 ++++++++++++++-----
 2 files changed, 24 insertions(+), 12 deletions(-)

diff --git a/resources/js/components/admin/manage/orders/OrderPosForm.vue b/resources/js/components/admin/manage/orders/OrderPosForm.vue
index 4d6148e..1c652d3 100644
--- a/resources/js/components/admin/manage/orders/OrderPosForm.vue
+++ b/resources/js/components/admin/manage/orders/OrderPosForm.vue
@@ -708,13 +708,10 @@ function submit() {
                                 <FieldLabel for="marketing">Marketing</FieldLabel>
                                 <Select v-model="form.marketing_id" :disabled="isMarketingUser">
                                     <SelectTrigger id="marketing" class="w-full">
-                                        <SelectValue placeholder="Pilih marketing (opsional)" />
+                                        <SelectValue placeholder="Pilih marketing" />
                                     </SelectTrigger>
                                     <SelectContent>
                                         <SelectGroup>
-                                            <SelectItem value="none">
-                                                — Tidak ada —
-                                            </SelectItem>
                                             <SelectItem v-for="marketing in marketings" :key="marketing.value"
                                                 :value="String(marketing.value)">
                                                 {{ marketing.label }}
diff --git a/resources/js/lib/api.ts b/resources/js/lib/api.ts
index 291d4c1..bd6c81e 100644
--- a/resources/js/lib/api.ts
+++ b/resources/js/lib/api.ts
@@ -19,17 +19,32 @@ export async function apiFetch<T>(url: string, options: RequestInit = {}): Promi
         credentials: 'same-origin',
     });
 
+    const contentType = response.headers.get('content-type') ?? '';
+    const isJson = contentType.includes('application/json');
+
     if (!response.ok) {
-        const error = await response.json().catch(() => ({})) as {
-            message?: string;
-            errors?: Record<string, string[]>;
-        };
+        if (isJson) {
+            const error = (await response.json().catch(() => ({}))) as {
+                message?: string;
+                errors?: Record<string, string[]>;
+            };
 
-        const firstValidationError = error.errors
-            ? Object.values(error.errors).flat()[0]
-            : undefined;
+            const firstValidationError = error.errors
+                ? Object.values(error.errors).flat()[0]
+                : undefined;
 
-        throw new Error(firstValidationError ?? error.message ?? 'Permintaan gagal diproses.');
+            throw new Error(firstValidationError ?? error.message ?? 'Permintaan gagal diproses.');
+        }
+
+        if (response.status === 401 || response.status === 419) {
+            throw new Error('Sesi Anda telah berakhir. Silakan muat ulang halaman dan coba lagi.');
+        }
+
+        throw new Error(`Permintaan gagal diproses (HTTP ${response.status}).`);
+    }
+
+    if (!isJson) {
+        throw new Error('Server mengembalikan respons yang tidak valid. Silakan muat ulang halaman.');
     }
 
     return response.json() as Promise<T>;