@flowselections/floriday-klanten-module 1.0.28 → 1.0.30

package/dist-lib/lib/customers.functions.js

@@ -1,8 +1,10 @@
 import { createServerFn } from "@tanstack/react-start";
+import { z } from "zod";
 import { requireSupabaseAuth } from "../integrations/supabase/auth-middleware";
 import { customerCreateSchema, customerUpdateSchema, listFiltersSchema, customerImportSchema, } from "./customer-schemas";
 import { buildPriceGroupsMap } from "./price-groups.functions";
 import { buildCustomerInsertRow } from "./customer-insert";
+import { friendlyError } from "./errors";
 function checkTwinfieldReady(draft) {
     const missing = [];
     if (!draft.company_name?.trim())
@@ -42,7 +44,7 @@ async function getActiveTwinfieldConnection(supabase) {
 }
 export const precheckTwinfieldPush = createServerFn({ method: "POST" })
     .middleware([requireSupabaseAuth])
-    .inputValidator((input) => input)
+    .inputValidator((input) => z.object({ draft: z.record(z.string(), z.any()).default({}) }).parse(input ?? { draft: {} }))
     .handler(async ({ data, context }) => {
     const conn = await getActiveTwinfieldConnection(context.supabase);
     if (!conn) {
@@ -63,36 +65,38 @@ export const listCustomers = createServerFn({ method: "POST" })
     // Floriday-klanten worden niet meer naar `customers` gespiegeld — die
     // worden direct uit floriday_customers_cache gelezen via
     // listFloridayCustomers(). Deze functie levert dus alleen lokale klanten.
-    let q = supabase
-        .from("customers")
-        .select("*")
-        .eq("source", "local")
-        .order("company_name");
-    if (data.customer_class)
-        q = q.eq("customer_class", data.customer_class);
-    if (data.country)
-        q = q.eq("country", data.country);
-    if (data.is_active === "active")
-        q = q.eq("is_active", true);
-    if (data.is_active === "inactive")
-        q = q.eq("is_active", false);
-    if (data.search) {
-        const s = `%${data.search}%`;
-        q = q.or(`company_name.ilike.${s},contact_person.ilike.${s},email.ilike.${s}`);
-    }
-    const { data: rows, error } = await q.limit(500);
-    if (error) {
-        // Vertaal technische Postgres-fouten naar begrijpelijke meldingen.
-        const msg = error.message ?? "";
-        if (/duplicate key/i.test(msg)) {
-            throw new Error("Deze klant bestaat al (dubbele waarde in een uniek veld).");
-        }
-        if (/relation .* does not exist|function .* does not exist/i.test(msg)) {
-            throw new Error("De klant kon niet worden opgeslagen door een configuratieprobleem in de database. Neem contact op met de beheerder.");
+    const build = () => {
+        let q = supabase
+            .from("customers")
+            .select("*")
+            .eq("source", "local")
+            .order("company_name");
+        if (data.customer_class)
+            q = q.eq("customer_class", data.customer_class);
+        if (data.country)
+            q = q.eq("country", data.country);
+        if (data.is_active === "active")
+            q = q.eq("is_active", true);
+        if (data.is_active === "inactive")
+            q = q.eq("is_active", false);
+        if (data.search) {
+            const s = `%${data.search}%`;
+            q = q.or(`company_name.ilike.${s},contact_person.ilike.${s},email.ilike.${s}`);
         }
-        throw new Error(`Klant aanmaken mislukt: ${msg}`);
+        return q;
+    };
+    // Geen maximum: lees in blokken van 1000 door tot het volledige
+    // klantenbestand binnen is (Postgres levert nooit meer per aanvraag).
+    const list = [];
+    for (let offset = 0;; offset += 1000) {
+        const { data: rows, error } = await build().range(offset, offset + 999);
+        if (error)
+            throw new Error(friendlyError(error, "Klanten laden mislukt."));
+        const chunk = rows ?? [];
+        list.push(...chunk);
+        if (chunk.length < 1000)
+            break;
     }
-    const list = rows ?? [];
     const orgIds = list
         .map((r) => r.floriday_organization_id)
         .filter((v) => typeof v === "string" && !!v);
@@ -107,7 +111,7 @@ export const listCustomers = createServerFn({ method: "POST" })
 });
 export const getCustomer = createServerFn({ method: "POST" })
     .middleware([requireSupabaseAuth])
-    .inputValidator((input) => input)
+    .inputValidator((input) => z.object({ id: z.string().uuid() }).parse(input))
     .handler(async ({ data, context }) => {
     const { supabase } = context;
     const [{ data: customer, error: cErr }, { data: audit }, { data: contacts }, { data: deliveryLocations }, { data: tradeSettings }, { data: assortment },] = await Promise.all([
@@ -141,7 +145,7 @@ export const getCustomer = createServerFn({ method: "POST" })
             .limit(500),
     ]);
     if (cErr)
-        throw new Error(cErr.message);
+        throw new Error(friendlyError(cErr, "Klant laden mislukt."));
     if (!customer)
         throw new Error("Klant niet gevonden");
     const { data: orders } = await supabase
@@ -238,7 +242,7 @@ export const createCustomer = createServerFn({ method: "POST" })
         if (/customers_customer_number_key/i.test(error.message ?? "")) {
             throw new Error("Dit klantnummer is al in gebruik door een andere klant.");
         }
-        throw new Error(error.message);
+        throw new Error(friendlyError(error, "Klant aanmaken mislukt."));
     }
     // De DB-trigger `customers_enqueue_twinfield_push` plaatst automatisch een
     // rij in twinfield_push_queue zodra er een actieve koppeling is; we hoeven
@@ -296,7 +300,7 @@ export const updateCustomer = createServerFn({ method: "POST" })
         if (/customers_customer_number_key/i.test(error.message ?? "")) {
             throw new Error("Dit klantnummer is al in gebruik door een andere klant.");
         }
-        throw new Error(error.message);
+        throw new Error(friendlyError(error, "Wijzigingen opslaan mislukt."));
     }
     // De DB-trigger heeft de wijziging al in twinfield_push_queue gezet wanneer
     // er een Twinfield-relevant veld is gewijzigd. Die regel verwerken we direct,
@@ -306,14 +310,14 @@ export const updateCustomer = createServerFn({ method: "POST" })
 });
 export const deleteCustomer = createServerFn({ method: "POST" })
     .middleware([requireSupabaseAuth])
-    .inputValidator((input) => input)
+    .inputValidator((input) => z.object({ id: z.string().uuid() }).parse(input))
     .handler(async ({ data, context }) => {
     const { error } = await context.supabase
         .from("customers")
         .delete()
         .eq("id", data.id);
     if (error)
-        throw new Error(error.message);
+        throw new Error(friendlyError(error, "Klant verwijderen mislukt."));
     return { ok: true };
 });
 export const listFloridayNetwork = createServerFn({ method: "POST" })