All checks were successful
Container-Image bauen und veröffentlichen / build-and-push (push) Successful in 1m23s
Die öffentliche API gab bei ?admin=true alle Datensätze heraus, auch die nicht veröffentlichten: von 117 Erklärungen waren 112 Entwürfe, die jeder Besucher abrufen konnte. Der Parameter setzt jetzt eine Admin-Sitzung voraus, sonst kommt die öffentliche Liste. Der Einzelabruf behandelt Entwürfe wie nicht vorhanden, damit sich über die fortlaufende ID nicht abklopfen lässt, welche Datensätze existieren. Markdown wurde mit sanitize: false gerendert und per dangerouslySetInnerHTML eingesetzt. Das Standard-Schema lässt jetzt Überschriften, Listen, Tabellen und http(s)/mailto-Links durch und entfernt Skripte, Event-Attribute, eingebettete Rahmen und javascript:-Ziele. Der Titel in der Druckfassung war an drei Stellen unmaskiert. Standardtexte - Zehn Bausteine, die für alle Erklärungen gelten, werden einmal zentral gepflegt. In der Erklärung steht nur eine Abweichung; ein leeres Feld bedeutet "folgt dem Standard". Ein Wechsel in der Amtsleitung wirkt damit sofort auf alle Erklärungen, die nicht bewusst abweichen. - Die Migration übernimmt den häufigsten Bestandswert als Standard und setzt übereinstimmende Felder auf NULL. Von 117 Erklärungen folgen danach 116 dem Standard, eine weicht ab (Online-Verfahren mit ZIT-SH als zusätzlichem Verantwortlichen). Pflichtangaben nach Art. 13/14 DSGVO - Neue Felder für Drittlandübermittlung (Art. 13 Abs. 1 lit. f), automatisierte Entscheidungsfindung (Art. 13 Abs. 2 lit. f), Datenkategorien (Art. 14 Abs. 1 lit. d) und berechtigte Interessen (Art. 13 Abs. 1 lit. d). Auf die ersten beiden muss auch dann hingewiesen werden, wenn es sie nicht gibt, deshalb sind sie Standardtexte mit ausdrücklichem "findet nicht statt". - Der Hinweis auf das Widerspruchsrecht steht nach Art. 21 Abs. 4 in einem eigenen, abgesetzten Kasten vor allen Abschnitten, nicht als Zeile im Fließtext der Betroffenenrechte. - Zweck, Rechtsgrundlage und Speicherdauer sind Pflicht. Geprüft wird beim Veröffentlichen, nicht beim Speichern: Ein Entwurf darf unvollständig sein, eine öffentlich sichtbare Erklärung nicht. So bleiben Bestandsdaten bearbeitbar, ohne dass die Anwendung Angaben erfindet, die nur die Fachabteilung kennt. Die Admin-Übersicht markiert unvollständige Erklärungen und nennt die fehlenden Angaben. Öffentliche Seiten - Jede Erklärung hat eine dauerhafte Adresse /erklaerung/<bezeichner>, serverseitig gerendert, zum Verlinken aus Antragsformularen. Der Bezeichner wird aus dem Titel abgeleitet und bleibt beim Bearbeiten erhalten, solange der Titel gleich bleibt. Entwürfe liefern 404. - Impressum, Datenschutzerklärung des Portals und Erklärung zur Barrierefreiheit werden im Admin als Markdown gepflegt und in der Fußzeile verlinkt. Die Fußzeile rendert serverseitig, weil das Impressum ohne JavaScript erreichbar sein muss. Ein leerer Text nimmt die Seite vom Netz, statt eine leere Seite auszuliefern. - Die Übersicht nutzt echte Links statt Klick-Handler und wird serverseitig vorbefüllt. APP_COMPANY_NAME ersetzt den Namen der Anwendung in Kopf- und Fußzeile, Seitentiteln, Anmeldeseite, Druckfassung und Startmeldung des Containers. Der Wert wird zur Laufzeit gelesen, ein Neubau des Images ist zum Umbenennen nicht nötig. Getestet gegen die 117 Bestandsdatensätze: Migrationen angewendet, der angezeigte Text blieb bei allen 117 x 7 Baustein-Feldern identisch. Alle 117 Bezeichner sind eindeutig und stimmen mit lib/slug.ts überein (SQLites LOWER() kennt nur ASCII, Großumlaute werden deshalb vorher ersetzt). ?admin=true liefert ohne Anmeldung 5 statt 117 Datensätze, Entwürfe 404. Über die API mit Anmeldung geprüft: unvollständig als Entwurf wird angelegt, dieselben Angaben mit isPublic abgelehnt, ebenso fehlende Datenkategorien bei angegebener Herkunft. Ohne APP_COMPANY_NAME erscheint "Syncova Policies", mit gesetztem Wert kein einziges Vorkommen mehr. Die Änderungen liegen in einem Commit, weil Schema, API-Routen und Formulare von allen Teilen berührt werden und eine thematische Aufteilung nicht lauffähige Zwischenstände ergäbe. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
248 lines
8.0 KiB
TypeScript
248 lines
8.0 KiB
TypeScript
"use client"
|
||
|
||
import { useEffect, useState } from "react"
|
||
import { AlertCircle } from "lucide-react"
|
||
import { toast } from "sonner"
|
||
|
||
import { Button } from "@/components/ui/button"
|
||
import {
|
||
Dialog,
|
||
DialogBody,
|
||
DialogContent,
|
||
DialogDescription,
|
||
DialogFooter,
|
||
DialogHeader,
|
||
DialogTitle,
|
||
} from "@/components/ui/dialog"
|
||
import { Tabs, TabsContent, TabsList, TabsTrigger } from "@/components/ui/tabs"
|
||
import { MarkdownInput } from "@/components/markdown-input"
|
||
import { updatePolicyDefaults, usePolicyDefaults } from "@/hooks/use-policy-defaults"
|
||
import { ApiError } from "@/lib/api"
|
||
import {
|
||
DEFAULTED_FIELDS,
|
||
DEFAULTED_FIELD_LABELS,
|
||
DefaultedField,
|
||
REQUIRED_DEFAULTED_FIELDS,
|
||
} from "@/lib/policy-defaults"
|
||
import { PolicyDefaults } from "@/types/privacy-policy"
|
||
|
||
/** Eingabewerte des Formulars – wie PolicyDefaults, aber ohne `updatedAt`. */
|
||
type DefaultsFormData = Omit<PolicyDefaults, "updatedAt">
|
||
|
||
const emptyForm: DefaultsFormData = {
|
||
responsible: "",
|
||
contactDPO: "",
|
||
dataSubjectsRights: "",
|
||
complaintRight: "",
|
||
withdrawalRight: "",
|
||
objectionRight: "",
|
||
thirdCountryTransfer: "",
|
||
automatedDecision: "",
|
||
provisionObligation: "",
|
||
provisionConsequences: "",
|
||
}
|
||
|
||
type TabId = "base" | "rights" | "processing"
|
||
|
||
/** Aufteilung der Bausteine auf die Reiter des Dialogs. */
|
||
const tabFields: Record<TabId, DefaultedField[]> = {
|
||
base: ["responsible", "contactDPO"],
|
||
processing: ["thirdCountryTransfer", "automatedDecision"],
|
||
rights: [
|
||
"dataSubjectsRights",
|
||
"complaintRight",
|
||
"withdrawalRight",
|
||
"objectionRight",
|
||
"provisionObligation",
|
||
"provisionConsequences",
|
||
],
|
||
}
|
||
|
||
/** Zeilenhöhe der Textfelder – die Rechtetexte sind deutlich länger. */
|
||
const fieldRows: Partial<Record<DefaultedField, number>> = {
|
||
dataSubjectsRights: 12,
|
||
complaintRight: 8,
|
||
withdrawalRight: 6,
|
||
objectionRight: 8,
|
||
thirdCountryTransfer: 6,
|
||
automatedDecision: 6,
|
||
provisionObligation: 4,
|
||
provisionConsequences: 4,
|
||
}
|
||
|
||
interface PolicyDefaultsFormProps {
|
||
open: boolean
|
||
onOpenChange: (open: boolean) => void
|
||
}
|
||
|
||
/**
|
||
* Pflegt die Standardtexte, die für alle Erklärungen gelten. Eine einzelne
|
||
* Erklärung kann jeden dieser Bausteine im Sonderfall überschreiben.
|
||
*/
|
||
export function PolicyDefaultsForm({ open, onOpenChange }: PolicyDefaultsFormProps) {
|
||
const { defaults, isLoading } = usePolicyDefaults()
|
||
|
||
const [formData, setFormData] = useState<DefaultsFormData>(emptyForm)
|
||
const [errors, setErrors] = useState<string[]>([])
|
||
const [invalidFields, setInvalidFields] = useState<Set<string>>(new Set())
|
||
const [isSubmitting, setIsSubmitting] = useState(false)
|
||
const [tab, setTab] = useState<TabId>("base")
|
||
|
||
// Beim Öffnen den gespeicherten Stand laden
|
||
useEffect(() => {
|
||
if (!open || isLoading) return
|
||
|
||
setErrors([])
|
||
setInvalidFields(new Set())
|
||
setTab("base")
|
||
// Über die Feldliste statt Feld für Feld: Ein neuer Baustein im Schema
|
||
// landet damit automatisch auch im Formular.
|
||
setFormData(
|
||
Object.fromEntries(
|
||
DEFAULTED_FIELDS.map((key) => [key, defaults[key]])
|
||
) as DefaultsFormData
|
||
)
|
||
// `defaults` bewusst nicht als Abhängigkeit: sonst würden Eingaben beim
|
||
// Hintergrund-Refresh von SWR überschrieben.
|
||
// eslint-disable-next-line react-hooks/exhaustive-deps
|
||
}, [open, isLoading])
|
||
|
||
/** Setzt ein Feld und entfernt dessen Fehlermarkierung. */
|
||
const set = (key: DefaultedField, value: string) => {
|
||
setFormData((prev) => ({ ...prev, [key]: value }))
|
||
setInvalidFields((prev) => {
|
||
if (!prev.has(key)) return prev
|
||
const next = new Set(prev)
|
||
next.delete(key)
|
||
return next
|
||
})
|
||
}
|
||
|
||
const handleSubmit = async (e: React.FormEvent) => {
|
||
e.preventDefault()
|
||
|
||
// Pflicht-Bausteine dürfen zentral nicht leer sein
|
||
const missing = REQUIRED_DEFAULTED_FIELDS.filter(
|
||
(field) => !formData[field].trim()
|
||
)
|
||
|
||
if (missing.length > 0) {
|
||
setErrors(missing.map((field) => `${DEFAULTED_FIELD_LABELS[field]} ist erforderlich`))
|
||
setInvalidFields(new Set(missing))
|
||
const reiter = (Object.keys(tabFields) as TabId[]).find((id) =>
|
||
tabFields[id].includes(missing[0])
|
||
)
|
||
setTab(reiter ?? "base")
|
||
return
|
||
}
|
||
|
||
setErrors([])
|
||
setInvalidFields(new Set())
|
||
setIsSubmitting(true)
|
||
|
||
try {
|
||
await updatePolicyDefaults(formData)
|
||
toast.success("Standardtexte gespeichert")
|
||
onOpenChange(false)
|
||
} catch (error) {
|
||
const details =
|
||
error instanceof ApiError && error.details.length > 0
|
||
? error.details
|
||
: [error instanceof Error ? error.message : "Fehler beim Speichern"]
|
||
setErrors(details)
|
||
} finally {
|
||
setIsSubmitting(false)
|
||
}
|
||
}
|
||
|
||
/** Rendert ein Baustein-Feld mit Beschriftung und Markdown-Vorschau. */
|
||
const field = (name: DefaultedField) => (
|
||
<MarkdownInput
|
||
key={name}
|
||
id={`default-${name}`}
|
||
label={DEFAULTED_FIELD_LABELS[name]}
|
||
value={formData[name]}
|
||
onChange={(value) => set(name, value)}
|
||
rows={fieldRows[name] ?? 6}
|
||
required={REQUIRED_DEFAULTED_FIELDS.includes(name)}
|
||
invalid={invalidFields.has(name)}
|
||
/>
|
||
)
|
||
|
||
return (
|
||
<Dialog open={open} onOpenChange={onOpenChange}>
|
||
<DialogContent className="max-h-[88vh] sm:max-w-4xl">
|
||
<DialogHeader>
|
||
<DialogTitle>Standardtexte</DialogTitle>
|
||
<DialogDescription>
|
||
Diese Texte gelten für alle Datenschutzerklärungen, die keinen
|
||
eigenen Text hinterlegt haben. Eine Änderung wirkt sich sofort auf
|
||
alle diese Erklärungen aus.
|
||
</DialogDescription>
|
||
</DialogHeader>
|
||
|
||
<form onSubmit={handleSubmit} className="flex min-h-0 flex-1 flex-col gap-4">
|
||
<Tabs
|
||
value={tab}
|
||
onValueChange={(value) => setTab(value as TabId)}
|
||
className="min-h-0 flex-1 gap-3"
|
||
>
|
||
<TabsList>
|
||
<TabsTrigger value="base">Stammdaten</TabsTrigger>
|
||
<TabsTrigger value="processing">Verarbeitung</TabsTrigger>
|
||
<TabsTrigger value="rights">Ihre Rechte</TabsTrigger>
|
||
</TabsList>
|
||
|
||
{errors.length > 0 && (
|
||
<div className="flex gap-2.5 rounded-lg border border-destructive/30 bg-destructive/5 p-3 text-sm text-destructive">
|
||
<AlertCircle className="mt-0.5 size-4 shrink-0" />
|
||
<ul className="space-y-1">
|
||
{errors.map((error, index) => (
|
||
<li key={index}>{error}</li>
|
||
))}
|
||
</ul>
|
||
</div>
|
||
)}
|
||
|
||
<DialogBody className="pt-1">
|
||
<TabsContent value="base" className="grid gap-5 md:grid-cols-2">
|
||
{tabFields.base.map(field)}
|
||
</TabsContent>
|
||
|
||
<TabsContent value="processing" className="space-y-4">
|
||
<p className="text-sm text-muted-foreground">
|
||
Auf beides muss ausdrücklich hingewiesen werden – auch dann,
|
||
wenn es nicht stattfindet. Ein klares „findet nicht statt“ ist
|
||
die richtige Antwort, kein leeres Feld.
|
||
</p>
|
||
<div className="grid gap-5 md:grid-cols-2">
|
||
{tabFields.processing.map(field)}
|
||
</div>
|
||
</TabsContent>
|
||
|
||
<TabsContent value="rights" className="grid gap-5 md:grid-cols-2">
|
||
<div className="space-y-5">{tabFields.rights.slice(0, 2).map(field)}</div>
|
||
<div className="space-y-5">{tabFields.rights.slice(2).map(field)}</div>
|
||
</TabsContent>
|
||
</DialogBody>
|
||
</Tabs>
|
||
|
||
<DialogFooter className="border-t pt-4">
|
||
<Button
|
||
type="button"
|
||
variant="outline"
|
||
onClick={() => onOpenChange(false)}
|
||
disabled={isSubmitting}
|
||
>
|
||
Abbrechen
|
||
</Button>
|
||
<Button type="submit" disabled={isSubmitting || isLoading}>
|
||
{isSubmitting ? "Speichern…" : "Standardtexte speichern"}
|
||
</Button>
|
||
</DialogFooter>
|
||
</form>
|
||
</DialogContent>
|
||
</Dialog>
|
||
)
|
||
}
|