From 0f51a1cc3b4f24092a984cb0c3c3d2f11e4aabca Mon Sep 17 00:00:00 2001
From: duffyduck <info@hacker-net.de>
Date: Sat, 8 Aug 2026 21:29:43 +0200
Subject: [PATCH] Gutschriften/Kuendigung: DRAFT-endDate-Schutz + Belege
 staff-only

Pentest R138 Hygiene + neue Vorgaben:

1) endDate wird bei DRAFT-Vertraegen NICHT mehr gesetzt (Entwurf = nur
   Vorlage). Nur Status wurde vorher geschont, endDate zog trotzdem mit.

2) Ueberweisungsbelege (credit-note-receipts) sind jetzt reine
   Mitarbeiter/Admin-Downloads: neuer FileOwner-kind 'contract-staff'
   blockt Portal-Kunden im fileDownload-Controller. Das generierte
   Gutschrift-PDF (credit-notes) bleibt vertragsbasiert -> der
   besitzende Kunde darf seine eigene Gutschrift laden.

Bereits vorher abgesichert (bestaetigt): Kunden koennen keine
Gutschriften anlegen (blockPortal) und keine Belege hochladen
(Portal-403 im Upload).

Verifiziert: DRAFT haelt endDate; Beleg-Owner=contract-staff,
PDF-Owner=contract.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../controllers/fileDownload.controller.ts    |  7 ++++++
 .../contractStatusScheduler.service.ts        |  8 ++++++-
 backend/src/services/fileDownload.service.ts  | 23 +++++++++++++------
 3 files changed, 30 insertions(+), 8 deletions(-)

diff --git a/backend/src/controllers/fileDownload.controller.ts b/backend/src/controllers/fileDownload.controller.ts
index e595635d..3a2d854c 100644
--- a/backend/src/controllers/fileDownload.controller.ts
+++ b/backend/src/controllers/fileDownload.controller.ts
@@ -48,6 +48,13 @@ export async function downloadFile(req: AuthRequest, res: Response): Promise<voi
     if (!(await canAccessCustomer(req, res, owner.customerId))) return;
   } else if (owner.kind === 'contract') {
     if (!(await canAccessContract(req, res, owner.contractId))) return;
+  } else if (owner.kind === 'contract-staff') {
+    // Nur Mitarbeiter/Admin – Portal-Kunden nie (z.B. Gutschrift-
+    // Überweisungsbelege). Staff dürfen wie bei allen Vertrags-Files.
+    if (req.user?.isCustomerPortal) {
+      res.status(403).json({ success: false, error: 'Keine Berechtigung' });
+      return;
+    }
   } else if (owner.kind === 'admin') {
     // PDF-Vorlagen: nur Mitarbeiter mit settings:read
     const perms = req.user?.permissions || [];
diff --git a/backend/src/services/contractStatusScheduler.service.ts b/backend/src/services/contractStatusScheduler.service.ts
index e2f9ae49..e11223b0 100644
--- a/backend/src/services/contractStatusScheduler.service.ts
+++ b/backend/src/services/contractStatusScheduler.service.ts
@@ -201,7 +201,13 @@ export async function maybeCancelOnCancellationConfirmation(
   }
 
   // Vertragsende = Kündigungsdatum (Bestätigungsdatum), falls vorhanden.
-  if (c.cancellationConfirmationDate && asDay(c.endDate) !== asDay(c.cancellationConfirmationDate)) {
+  // NICHT bei DRAFT: ein Entwurf ist nur eine Vorlage und bekommt kein
+  // berechnetes Vertragsende (Pentest R138, Hygiene-Punkt).
+  if (
+    c.status !== 'DRAFT' &&
+    c.cancellationConfirmationDate &&
+    asDay(c.endDate) !== asDay(c.cancellationConfirmationDate)
+  ) {
     updateData.endDate = c.cancellationConfirmationDate;
     changes.endDate = { vorher: asDay(c.endDate), nachher: asDay(c.cancellationConfirmationDate) };
   }
diff --git a/backend/src/services/fileDownload.service.ts b/backend/src/services/fileDownload.service.ts
index 4033e6c5..ed3ad846 100644
--- a/backend/src/services/fileDownload.service.ts
+++ b/backend/src/services/fileDownload.service.ts
@@ -15,6 +15,8 @@ import prisma from '../lib/prisma.js';
 export type FileOwner =
   | { kind: 'customer'; customerId: number }
   | { kind: 'contract'; contractId: number }
+  // Wie 'contract', aber Portal-Kunden ausgeschlossen (nur Mitarbeiter/Admin).
+  | { kind: 'contract-staff'; contractId: number }
   | { kind: 'admin' }
   | { kind: 'gdpr-admin' };
 
@@ -93,14 +95,21 @@ export async function findUploadOwner(uploadPath: string): Promise<FileOwner | n
       return r?.contractId ? { kind: 'contract', contractId: r.contractId } : null;
     }
 
-    case 'credit-note-receipts':
-    case 'credit-notes': {
-      // Überweisungsbeleg bzw. generiertes Gutschrift-PDF – Owner ist der
-      // Vertrag der Gutschrift.
+    case 'credit-note-receipts': {
+      // Überweisungsbeleg: NUR Mitarbeiter/Admin. Portal-Kunden dürfen Belege
+      // nicht herunterladen (nur ihre Gutschrift selbst).
       const r = await prisma.creditNote.findFirst({
-        where: subDir === 'credit-note-receipts'
-          ? { receiptPath: uploadPath }
-          : { pdfPath: uploadPath },
+        where: { receiptPath: uploadPath },
+        select: { contractId: true },
+      });
+      return r ? { kind: 'contract-staff', contractId: r.contractId } : null;
+    }
+
+    case 'credit-notes': {
+      // Generiertes Gutschrift-PDF: Owner ist der Vertrag der Gutschrift –
+      // der besitzende Kunde darf seine eigene Gutschrift laden.
+      const r = await prisma.creditNote.findFirst({
+        where: { pdfPath: uploadPath },
         select: { contractId: true },
       });
       return r ? { kind: 'contract', contractId: r.contractId } : null;