feat: Ernten und Chargen im Admin loeschbar machen

Neue deleteHarvestAction/deleteBatchAction, jeweils mit vorheriger
Referenzpruefung statt rohem Datenbankfehler: eine Ernte laesst sich nur
loeschen, wenn keine Chargen/Produkte mehr darauf verweisen, eine Charge
nur, wenn sie keinem Produkt zugeordnet ist und in keiner Bestellung
verwendet wurde (Chargen-Code steht auf Rechnungen/QR-Codes und darf aus
Nachweispflicht nie verschwinden). Loeschen-Button erscheint in den
Listen nur, wenn die Loeschung tatsaechlich moeglich ist, mit
Bestaetigungsdialog wie beim bestehenden Benutzer-Loeschen.

Co-Authored-By: Claude Sonnet 5 <[email protected]>

src/app/admin/actions/harvests-batches.ts

@@ -5,7 +5,7 @@ import { revalidatePath } from "next/cache";
 import { prisma } from "@/lib/prisma";
 import { logAdminAction } from "@/lib/audit-log";
 import { isValidUploadPath } from "@/lib/uploads";
-import { requireAdmin, str, num, strArray } from "./_shared";
+import { requireAdmin, fail, str, num, strArray } from "./_shared";
 
 export async function createHarvestAction(formData: FormData): Promise<void> {
   const admin = await requireAdmin();
@@ -68,6 +68,45 @@ export async function updateHarvestAction(formData: FormData): Promise<void> {
   redirect("/admin/ernten");
 }
 
+/**
+ * Loescht eine Ernte unwiderruflich. harvestId ist auf Batch UND
+ * harvestId/batchId auf Product jeweils onDelete: Restrict (siehe
+ * prisma/schema.prisma) — die Datenbank wuerde ein direktes delete() also
+ * ohnehin ablehnen, sobald noch Chargen oder Produkte auf diese Ernte
+ * verweisen. Die Zaehlung hier VOR dem Loeschversuch liefert dafuer eine
+ * konkrete, verstaendliche Fehlermeldung statt eines rohen Datenbankfehlers.
+ */
+export async function deleteHarvestAction(formData: FormData): Promise<void> {
+  const admin = await requireAdmin();
+  const id = str(formData, "id");
+
+  const harvest = await prisma.harvest.findUnique({
+    where: { id },
+    include: { _count: { select: { batches: true, products: true } } },
+  });
+  if (!harvest) fail("/admin/ernten", "Ernte nicht gefunden.");
+
+  if (harvest._count.batches > 0 || harvest._count.products > 0) {
+    fail(
+      "/admin/ernten",
+      `Diese Ernte hat noch ${harvest._count.batches} Charge(n) und ${harvest._count.products} Produkt(e). Lösche oder verschiebe diese zuerst.`
+    );
+  }
+
+  await prisma.harvest.delete({ where: { id } });
+
+  await logAdminAction(admin, {
+    action: "harvest.delete",
+    entityType: "Harvest",
+    entityId: id,
+    detail: `${harvest.type} ${harvest.year}`,
+  });
+
+  revalidatePath("/admin/ernten");
+  revalidatePath("/ernten");
+  redirect("/admin/ernten");
+}
+
 /**
  * Isolierte "Veroeffentlichen"/"Verbergen"-Aktion, getrennt von
  * updateHarvestAction — so kann der Status per Knopfdruck direkt aus der
@@ -136,3 +175,43 @@ export async function updateBatchAction(formData: FormData): Promise<void> {
   revalidatePath("/admin/chargen");
   redirect("/admin/chargen");
 }
+
+/**
+ * Loescht eine Charge unwiderruflich. productId/batchId auf OrderItem sind
+ * onDelete: Restrict (siehe prisma/schema.prisma) — eine Charge, die schon
+ * in mindestens einer Bestellposition verwendet wurde, darf aus
+ * Rechnungs-/Nachweispflicht NIE geloescht werden (Chargen-Code steht auf
+ * dem QR-Code des Glases und in Rechnungen). Batch.products ist ebenfalls
+ * Restrict. Die Zaehlung hier VOR dem Loeschversuch liefert dafuer eine
+ * konkrete, verstaendliche Fehlermeldung statt eines rohen Datenbankfehlers.
+ */
+export async function deleteBatchAction(formData: FormData): Promise<void> {
+  const admin = await requireAdmin();
+  const id = str(formData, "id");
+
+  const batch = await prisma.batch.findUnique({
+    where: { id },
+    include: { _count: { select: { products: true, orderItems: true } } },
+  });
+  if (!batch) fail("/admin/chargen", "Charge nicht gefunden.");
+
+  if (batch._count.orderItems > 0) {
+    fail(
+      "/admin/chargen",
+      `Charge "${batch.code}" wurde bereits in ${batch._count.orderItems} Bestellposition(en) verwendet und kann deshalb nicht gelöscht werden.`
+    );
+  }
+  if (batch._count.products > 0) {
+    fail(
+      "/admin/chargen",
+      `Charge "${batch.code}" ist noch ${batch._count.products} Produkt(en) zugeordnet. Löse diese Zuordnung zuerst.`
+    );
+  }
+
+  await prisma.batch.delete({ where: { id } });
+
+  await logAdminAction(admin, { action: "batch.delete", entityType: "Batch", entityId: id, detail: batch.code });
+
+  revalidatePath("/admin/chargen");
+  redirect("/admin/chargen");
+}

src/app/admin/chargen/page.tsx

@@ -1,17 +1,19 @@
 import Link from "next/link";
 import { prisma } from "@/lib/prisma";
 import { BackLink } from "@/components/back-link";
+import { ConfirmForm } from "@/components/confirm-form";
 import { formatDate } from "@/lib/format";
-import { createBatchAction } from "../actions";
+import { createBatchAction, deleteBatchAction } from "../actions";
 import { getLocale } from "@/i18n/locale";
 import { getDictionary } from "@/i18n/get-dictionary";
 
-export default async function AdminBatchesPage() {
+export default async function AdminBatchesPage(props: PageProps<"/admin/chargen">) {
   const t = getDictionary(await getLocale());
+  const { error } = await props.searchParams;
   const [batches, harvests] = await Promise.all([
     prisma.batch.findMany({
       orderBy: { fillDate: "desc" },
-      include: { harvest: true },
+      include: { harvest: true, _count: { select: { products: true, orderItems: true } } },
     }),
     prisma.harvest.findMany({ orderBy: [{ year: "desc" }] }),
   ]);
@@ -20,6 +22,9 @@ export default async function AdminBatchesPage() {
     <div>
       <BackLink href="/admin" label={t.adminBatchesPage.backToDashboard} />
       <h1 className="text-2xl text-forest mb-8">{t.adminBatchesPage.title}</h1>
+      {error ? (
+        <p className="mb-6 text-sm text-red-700 bg-red-50 border border-red-200 px-4 py-3">{error}</p>
+      ) : null}
       <div className="overflow-x-auto bg-background border border-ink/10">
         <table className="w-full text-sm">
           <thead className="text-left text-ink/65 border-b border-ink/10">
@@ -44,10 +49,22 @@ export default async function AdminBatchesPage() {
                 <td className="p-4">{formatDate(batch.bestBeforeDate)}</td>
                 <td className="p-4">{batch.glassCount}</td>
                 <td className="p-4">{batch.remainingGlasses}</td>
-                <td className="p-4">
+                <td className="p-4 whitespace-nowrap">
-                  <Link href={`/admin/chargen/${batch.id}`} className="text-forest underline">
+                  <Link href={`/admin/chargen/${batch.id}`} className="text-forest underline mr-4">
                     {t.adminBatchesPage.edit}
                   </Link>
+                  {batch._count.products === 0 && batch._count.orderItems === 0 ? (
+                    <ConfirmForm
+                      action={deleteBatchAction}
+                      confirmMessage={t.adminBatchesPage.deleteConfirm(batch.code)}
+                      className="inline"
+                    >
+                      <input type="hidden" name="id" value={batch.id} />
+                      <button type="submit" className="text-red-700 underline">
+                        {t.adminBatchesPage.delete}
+                      </button>
+                    </ConfirmForm>
+                  ) : null}
                 </td>
               </tr>
             ))}

src/app/admin/ernten/page.tsx

@@ -1,14 +1,20 @@
 import Link from "next/link";
 import { prisma } from "@/lib/prisma";
 import { BackLink } from "@/components/back-link";
+import { ConfirmForm } from "@/components/confirm-form";
 import { formatDate } from "@/lib/format";
-import { createHarvestAction, toggleHarvestPublishedAction, syncHarvestsFromApiCoreAction } from "../actions";
+import {
+  createHarvestAction,
+  toggleHarvestPublishedAction,
+  syncHarvestsFromApiCoreAction,
+  deleteHarvestAction,
+} from "../actions";
 import { getLocale } from "@/i18n/locale";
 import { getDictionary } from "@/i18n/get-dictionary";
 
 export default async function AdminHarvestsPage(props: PageProps<"/admin/ernten">) {
   const t = getDictionary(await getLocale());
-  const { synced, updated, skipped, failed, syncError, batchSynced, batchSkipped, batchFailed, batchLoadError } =
+  const { synced, updated, skipped, failed, syncError, batchSynced, batchSkipped, batchFailed, batchLoadError, error } =
     await props.searchParams;
   const harvests = await prisma.harvest.findMany({
     orderBy: [{ year: "desc" }, { harvestDate: "desc" }],
@@ -20,6 +26,9 @@ export default async function AdminHarvestsPage(props: PageProps<"/admin/ernten"
       <BackLink href="/admin" label={t.adminHarvestsPage.backToDashboard} />
       <h1 className="text-2xl text-forest mb-8">{t.adminHarvestsPage.title}</h1>
 
+      {error ? (
+        <p className="mb-6 text-sm text-red-700 bg-red-50 border border-red-200 px-4 py-3">{error}</p>
+      ) : null}
       {synced !== undefined ? (
         <p className="mb-6 text-sm text-forest bg-forest/5 border border-forest/20 px-4 py-3">
           {t.adminHarvestsPage.syncResult(Number(synced), Number(updated ?? 0), Number(skipped ?? 0), Number(failed ?? 0))}
@@ -82,10 +91,22 @@ export default async function AdminHarvestsPage(props: PageProps<"/admin/ernten"
                   <form action={toggleHarvestPublishedAction} className="inline">
                     <input type="hidden" name="id" value={harvest.id} />
                     {harvest.isPublished ? null : <input type="hidden" name="isPublished" value="on" />}
-                    <button type="submit" className="text-forest underline">
+                    <button type="submit" className="text-forest underline mr-4">
                       {harvest.isPublished ? t.adminHarvestsPage.unpublish : t.adminHarvestsPage.publish}
                     </button>
                   </form>
+                  {harvest._count.batches === 0 && harvest._count.products === 0 ? (
+                    <ConfirmForm
+                      action={deleteHarvestAction}
+                      confirmMessage={t.adminHarvestsPage.deleteConfirm(`${harvest.type} ${harvest.year}`)}
+                      className="inline"
+                    >
+                      <input type="hidden" name="id" value={harvest.id} />
+                      <button type="submit" className="text-red-700 underline">
+                        {t.adminHarvestsPage.delete}
+                      </button>
+                    </ConfirmForm>
+                  ) : null}
                 </td>
               </tr>
             ))}

src/i18n/dictionaries/de.ts

@@ -821,6 +821,8 @@ export const de: Dictionary = {
     colTotalGlasses: "Gläser gesamt",
     colRemaining: "Verbleibend",
     edit: "Bearbeiten",
+    delete: "Löschen",
+    deleteConfirm: (code) => `Charge "${code}" wirklich löschen? Dies kann nicht rückgängig gemacht werden.`,
     noBatchesYet: "Noch keine Chargen angelegt.",
     newBatchHeading: "Neue Charge anlegen",
     codeLabel: "Chargen-Code",
@@ -887,6 +889,8 @@ export const de: Dictionary = {
     publish: "Veröffentlichen",
     unpublish: "Verbergen",
     edit: "Bearbeiten",
+    delete: "Löschen",
+    deleteConfirm: (label) => `"${label}" wirklich löschen? Dies kann nicht rückgängig gemacht werden.`,
     noHarvestsYet: "Noch keine Ernten angelegt.",
     newHarvestHeading: "Neue Ernte anlegen",
     typeLabel: "Typ (z.B. Frühtracht)",

src/i18n/dictionaries/en.ts

@@ -821,6 +821,8 @@ export const en: Dictionary = {
     colTotalGlasses: "Total jars",
     colRemaining: "Remaining",
     edit: "Edit",
+    delete: "Delete",
+    deleteConfirm: (code) => `Really delete batch "${code}"? This cannot be undone.`,
     noBatchesYet: "No batches created yet.",
     newBatchHeading: "Create new batch",
     codeLabel: "Batch code",
@@ -887,6 +889,8 @@ export const en: Dictionary = {
     publish: "Publish",
     unpublish: "Unpublish",
     edit: "Edit",
+    delete: "Delete",
+    deleteConfirm: (label) => `Really delete "${label}"? This cannot be undone.`,
     noHarvestsYet: "No harvests created yet.",
     newHarvestHeading: "Create new harvest",
     typeLabel: "Type (e.g. Spring flow)",

src/i18n/dictionary-type.ts

@@ -523,6 +523,8 @@ export interface Dictionary {
     colTotalGlasses: string;
     colRemaining: string;
     edit: string;
+    delete: string;
+    deleteConfirm: (code: string) => string;
     noBatchesYet: string;
     newBatchHeading: string;
     codeLabel: string;
@@ -589,6 +591,8 @@ export interface Dictionary {
     publish: string;
     unpublish: string;
     edit: string;
+    delete: string;
+    deleteConfirm: (label: string) => string;
     noHarvestsYet: string;
     newHarvestHeading: string;
     typeLabel: string;