@flowselections/floriday-verkoop-module 1.0.55 → 1.0.56

package/dist-lib/lib/cart-replan.functions.js

@@ -160,18 +160,14 @@ export async function replanScopeInternal(scope) {
     for (const it of items ?? []) {
         itemToOrder.set(it.id, it.order_id);
     }
-    // Bestaande karren in de scope tellen (voor waarschuwing) en wissen.
     // Nummering én delete-scope liggen altijd op bufferniveau als de order een
-    // buffer heeft; order-scope is uitsluitend terugval.
+    // buffer heeft; order-scope is uitsluitend terugval. Wissen én wegschrijven
+    // gebeuren verderop in ÉÉN transactie via public.replace_order_carts, zodat
+    // een mislukte replan de bestaande indeling laat staan.
     if (!bufferId) {
         console.warn("[cart-replan] geen buffer voor order %s — nummering en delete vallen terug op order-scope", soloOrderId);
     }
-    const cartScope = (q) => bufferId ? q.eq("buffer_id", bufferId) : q.eq("order_id", soloOrderId);
-    const { data: existingCarts } = await cartScope(supabaseAdmin.from("order_carts").select("id, source"));
-    const replacedManualCount = (existingCarts ?? []).filter((c) => c.source === "manual").length;
-    const { error: delErr } = await cartScope(supabaseAdmin.from("order_carts").delete());
-    if (delErr)
-        throw new Error(delErr.message);
+    let replacedManualCount = 0;
     // De karindeling wordt hier altijd zelf gerekend. custom_fields.floriday_packing
     // levert alleen de belading-VELDEN (pieces_per_package / packages_per_layer /
     // layers_per_load_carrier) — de mirror schrijft nooit een voorgekauwde
@@ -278,8 +274,10 @@ export async function replanScopeInternal(scope) {
             }
         }
     }
-    let cartsCreated = 0;
-    for (const { cart, profile } of plannedCarts) {
+    // Payload voor public.replace_order_carts. LET OP: de sleutels hieronder
+    // moeten gelijk lopen met de sleutellijsten in die SQL-functie; ontbreekt er
+    // één, dan gooit de database een fout in plaats van stil NULL te schrijven.
+    const cartPayload = plannedCarts.map(({ cart, profile }) => {
         // Bepaal cart.order_id: null bij mix van orders, anders die ene order.
         const orderIdsOnCart = new Set();
         for (const i of cart.items) {
@@ -288,9 +286,7 @@ export async function replanScopeInternal(scope) {
                 orderIdsOnCart.add(oid);
         }
         const singleOrderId = orderIdsOnCart.size === 1 ? [...orderIdsOnCart][0] : null;
-        const { data: inserted, error: cErr } = await supabaseAdmin
-            .from("order_carts")
-            .insert({
+        return {
             order_id: singleOrderId,
             buffer_id: bufferId,
             cart_number: cart.cart_number,
@@ -304,14 +300,7 @@ export async function replanScopeInternal(scope) {
             carrier_type_id: profile.id,
             max_layers: profile.layers_per_carrier ?? null,
             algorithm_version: CART_PLANNER_VERSION,
-        })
-            .select("id")
-            .single();
-        if (cErr)
-            throw new Error(cErr.message);
-        if (cart.items.length > 0) {
-            const rows = cart.items.map((i) => ({
-                cart_id: inserted.id,
+            items: cart.items.map((i) => ({
                 order_item_id: i.order_item_id,
                 product_id: i.product_id,
                 product_name: i.product_name,
@@ -321,14 +310,28 @@ export async function replanScopeInternal(scope) {
                 layers_per_cart: i.layers_per_cart,
                 layer_fraction: i.layer_fraction,
                 layer_share_pct: i.layer_share_pct,
-            }));
-            const { error: iErr } = await supabaseAdmin
-                .from("order_cart_items")
-                .insert(rows);
-            if (iErr)
-                throw new Error(iErr.message);
-        }
-        cartsCreated++;
+            })),
+        };
+    });
+    // Delete + inserts in één transactie: faalt er iets, dan blijft de oude
+    // karindeling staan in plaats van een lege buffer achter te laten.
+    const { data: replaceRes, error: replaceErr } = await supabaseAdmin.rpc("replace_order_carts", {
+        p_buffer_id: bufferId,
+        p_order_id: bufferId ? null : soloOrderId,
+        p_carts: cartPayload,
+    });
+    if (replaceErr)
+        throw new Error(replaceErr.message);
+    const cartsCreated = Number(replaceRes?.carts_written ?? 0);
+    const itemsWritten = Number(replaceRes?.items_written ?? 0);
+    replacedManualCount = Number(replaceRes?.replaced_manual_count ?? 0);
+    // Controleer dat de database daadwerkelijk heeft weggeschreven wat de
+    // planner opleverde. Nul karren mag, maar alleen als de planner er ook nul
+    // opleverde — niet als gevolg van een fout.
+    const expectedItems = cartPayload.reduce((n, c) => n + c.items.length, 0);
+    if (cartsCreated !== cartPayload.length || itemsWritten !== expectedItems) {
+        throw new Error(`Karindeling niet volledig weggeschreven: ${cartsCreated}/${cartPayload.length} karren, ` +
+            `${itemsWritten}/${expectedItems} artikelen.`);
     }
     return {
         ok: true,
@@ -502,9 +505,13 @@ export const listOrderCarts = createServerFn({ method: "GET" })
                 fail("planqueue lezen", q.error);
             const r = (q.data ?? [])[0];
             if (r && (r.error || r.reason)) {
+                // `reason` in logistics_plan_queue is vaak een sleutel/veldnaam
+                // ("items"), geen leesbare reden. De echte fout staat in `error`.
+                const err = r.error ?? null;
+                const rawReason = r.reason ?? null;
                 blocked = {
-                    reason: r.reason ?? null,
-                    explanation: r.error ?? null,
+                    reason: rawReason,
+                    explanation: err ?? (rawReason ? `Planning geblokkeerd (${rawReason})` : null),
                     at: r.updated_at ?? null,
                 };
             }