@flowselections/floriday-klanten-module 1.0.27 → 1.0.29

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" })

package/dist-lib/lib/errors.d.ts

@@ -0,0 +1,7 @@
+/**
+ * Zet technische database-/servermeldingen om naar begrijpelijke Nederlandse
+ * tekst. Onbekende fouten krijgen een nette algemene melding, zodat interne
+ * kolom- en constraintnamen nooit in de UI belanden.
+ */
+export declare function friendlyError(err: unknown, fallback?: string): string;
+//# sourceMappingURL=errors.d.ts.map

package/dist-lib/lib/errors.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"errors.d.ts","sourceRoot":"","sources":["../../src/lib/errors.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AACH,wBAAgB,aAAa,CAAC,GAAG,EAAE,OAAO,EAAE,QAAQ,SAA2C,GAAG,MAAM,CA0BvG"}

package/dist-lib/lib/errors.js

@@ -0,0 +1,36 @@
+/**
+ * Zet technische database-/servermeldingen om naar begrijpelijke Nederlandse
+ * tekst. Onbekende fouten krijgen een nette algemene melding, zodat interne
+ * kolom- en constraintnamen nooit in de UI belanden.
+ */
+export function friendlyError(err, fallback = "Er ging iets mis. Probeer het opnieuw.") {
+    const raw = err instanceof Error ? err.message : typeof err === "string" ? err : "";
+    if (!raw)
+        return fallback;
+    // Meldingen die we zelf al in het Nederlands schrijven, tonen we ongewijzigd.
+    if (/[a-z]/i.test(raw) && !looksTechnical(raw))
+        return raw;
+    if (/duplicate key|unique constraint/i.test(raw)) {
+        return "Deze gegevens bestaan al. Controleer of het record niet dubbel is.";
+    }
+    if (/violates foreign key/i.test(raw)) {
+        return "Deze actie kan niet worden uitgevoerd omdat er nog gekoppelde gegevens zijn.";
+    }
+    if (/violates not-null|null value in column/i.test(raw)) {
+        return "Er ontbreken verplichte gegevens. Vul alle verplichte velden in.";
+    }
+    if (/permission denied|row-level security|not authorized|unauthorized/i.test(raw)) {
+        return "Je hebt geen rechten voor deze actie.";
+    }
+    if (/could not find the .* column|relation .* does not exist|function .* does not exist|schema cache/i.test(raw)) {
+        return "Er is een configuratieprobleem in de database. Neem contact op met de beheerder.";
+    }
+    if (/failed to fetch|networkerror|timeout/i.test(raw)) {
+        return "Geen verbinding met de server. Controleer je internetverbinding en probeer opnieuw.";
+    }
+    return fallback;
+}
+function looksTechnical(msg) {
+    return (/duplicate key|constraint|column|relation |schema cache|row-level security|permission denied|violates|pgrst|jwt|syntax error|null value/i
+        .test(msg));
+}

package/dist-lib/lib/external-customers.functions.d.ts

@@ -39680,8 +39680,8 @@ export declare const listExternalCustomers: import("@tanstack/start-client-core"
 }, undefined, undefined, undefined>], (input: unknown) => {
     search: string;
     status: "active" | "all";
-    limit: number;
     providers?: string[] | undefined;
+    limit?: number | undefined;
 }, Promise<{
     customers: GroupedCustomer[];
     providers: ExternalProvider[];