Terminplaner: Bestaetigung prueft vorher, dass die Zellen der Haltung ihr noch gehoeren

bestaetige zieht unter der Buchungssperre fehlende eigene Zellen nach (belegeZellen mit ausser). Hat ein anderer
Gast sie uebernommen, wird die Buchung verfallen und es kommt dieselbe 410-Antwort wie bei abgelaufener Haltung,
ohne Mails.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

server/services/termine/ablauf.js

@@ -476,12 +476,43 @@ export async function vorschau(token, now) {
 
 // ---------------------------------------------------------------- Bestaetigen
 
+/**
+ * Gehoeren die Zellen der Haltung ihr noch? Ein anderer Gast, dessen Anfrage mit einem spaeteren `now` lief, darf die
+ * Zellen einer schon abgelaufenen Haltung geloescht und selbst belegt haben (zellen.js: eine abgelaufene Haltung gilt
+ * als frei), waehrend diese Anfrage noch `heldUntil > now` sieht. Darum wird vor der Bestaetigung (unter der
+ * Buchungssperre) nachgezogen, was von den eigenen Zellen fehlt (belegeZellen mit ausser = eigene ID). Gelingt das nicht
+ * (SLOT_VERGEBEN), ist die Zeit weg: Buchung `verfallen` und dieselbe Antwort wie bei einer abgelaufenen Haltung (410),
+ * ohne Mails. Andere Fehler (z. B. Datenbank) gehen durch, die Buchung bleibt `gehalten`.
+ * Sperrenfolge termine-buchung -> termine-res (in belegeZellen); in umgekehrter Richtung nimmt niemand beide.
+ */
+async function sichereZellen(buchung, ctx, now) {
+  try {
+    await belegeZellen({
+      featureId: buchung.featureId,
+      ressourceId: buchung.ressourceId,
+      bookingId: buchung.$id,
+      start: new Date(zeitMs(buchung.startsAt)),
+      blockEnde: new Date(zeitMs(buchung.blockEndsAt)),
+      now,
+      ausser: buchung.$id,
+    })
+  } catch (err) {
+    if (err?.code !== 'SLOT_VERGEBEN') throw err
+    try {
+      await aendere(buchung.$id, { status: 'verfallen', updatedAt: iso(now) })
+    } catch (fehler) {
+      protokolliere('Haltung nicht als verfallen vermerkt', buchung.$id, fehler)
+    }
+    throw abgelaufenMit(410, ctx)
+  }
+}
+
 /**
  * E-Mail bestaetigt (Klick auf den Knopf der Gaesteseite): `gehalten` mit heldUntil > now wird bei einer sofort
  * buchbaren Leistung `bestaetigt` (bestaetigtAt, G02 an den Gast mit Einladung, K20 an den Betrieb), bei einer Anfrage
  * `angefragt` (G03 an den Gast, K20-Block anfrage); in beiden Faellen wird der Hash des Verwaltungstokens gespeichert.
  * Ein weiterer Klick aendert nichts und meldet `schon`. Die Haltung selbst laesst der Minutenjob verfallen; hier gilt
- * nur heldUntil > now.
+ * nur heldUntil > now, und die Zellen der Haltung muessen ihr noch gehoeren (sichereZellen).
  * @param {string} token Bestaetigungstoken aus dem Link
  * @param {Date} now
  * @returns {Promise<{ status: string, schon?: true }>}
@@ -499,6 +530,7 @@ export async function bestaetige(token, now) {
     if (!frisch) throw unbekannt()
     if (frisch.status === 'verfallen' || (frisch.status === 'gehalten' && !(zeitMs(frisch.heldUntil) > nowMs))) throw abgelaufenMit(410, ctx)
     if (frisch.status !== 'gehalten') return { schon: true, status: frisch.status }
+    await sichereZellen(frisch, ctx, now)
     const anfrage = frisch.buchungsart === 'anfrage'
     const buchung = await aendere(id, {
       status: anfrage ? 'angefragt' : 'bestaetigt',