From 89bbed8679bab49ca43feaa6f3ec47bac6ec5ae4 Mon Sep 17 00:00:00 2001 From: Harlekin Date: Thu, 3 Sep 2026 19:16:17 +0200 Subject: [PATCH] =?UTF-8?q?Suche=20zeigt=20alle=20Arten,=20nur=20Arbeitsro?= =?UTF-8?q?llen=20=C3=B6ffnen=20die=20Akte?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Punkt 1 und 2 der Wünsche, plus zwei Lücken, die dabei sichtbar wurden. **Suche.** Wer nach Alvarez sucht, landet auf den Fallakten und erfährt nie, dass es dazu auch eine Personalakte gibt: die Suche fand sie, aber hinter einem Reiter, auf den nichts hindeutete. Die Reiter tragen jetzt ihre Trefferzahl, und ein leeres Ergebnis nennt ausdrücklich, wo etwas liegt. Dabei fiel auf, dass searchPersonnel das Recht personnel.read nicht prüfte. Die Kommandopalette tat es, die Aktenseite tat es, die Suche nicht - über /suche?kind=personnel kam jeder mit ausreichender Stufe an eine Liste aller Personalakten mit Namen, Dienstgrad und Etiketten. Die Prüfung steht jetzt dort, wo diese Akten herausgegeben werden. Die Trefferzahl einer Art, die es für den Nutzer nicht gibt, ist null und nicht 0: eine Null wäre die Auskunft „hier ist nichts", und die steht ihm nicht zu. **Täter und Mittäter** als Beteiligtenrollen - und damit wurde eine ältere Lücke offensichtlich: ein Beteiligter durfte die Akte lesen, gleich in welcher Rolle. Bei „Betroffener" fiel das nicht auf, bei „Täter" wäre es offensichtlich falsch. Der Beschuldigte hätte die Ermittlung gegen sich selbst mitgelesen. Die Beteiligtenliste beantwortet zwei Fragen: wer bearbeitet den Fall, und um wen geht es darin. Nur die erste ist eine Berechtigung. WORKING_ROLES nennt die drei, die die Akte öffnen; Zeuge, Betroffener, Täter und Mittäter öffnen nichts. Das gilt an allen vier Stellen, die auf Beteiligung prüften: Aktenzugriff, Dokumentzugriff über die besitzende Akte, „meine Fälle" und der Filter „nur meine". Zwei bestehende Tests hatten die alte Annahme festgeschrieben und benutzten WITNESS, um Zugriff zu belegen; sie prüfen jetzt mit INVESTIGATOR. --- .../migration.sql | 13 +++ packages/db/prisma/schema.prisma | 11 ++- web/app/(app)/suche/page.tsx | 73 +++++++++++---- web/lib/access/caseFilter.ts | 17 ++-- web/lib/access/personnelFilter.ts | 7 +- web/lib/cases/terms.ts | 27 +++++- web/lib/dashboard/load.ts | 5 +- web/lib/i18n/cases.ts | 8 +- web/lib/i18n/shell.ts | 2 + web/lib/search/service.ts | 89 +++++++++++++++++-- web/tests/suche.test.ts | 40 ++++++++- web/tests/verknuepfungen.test.ts | 6 +- web/tests/zugriff.test.ts | 55 +++++++++++- 13 files changed, 308 insertions(+), 45 deletions(-) create mode 100644 packages/db/prisma/migrations/20260904090000_taeter_und_mittaeter/migration.sql diff --git a/packages/db/prisma/migrations/20260904090000_taeter_und_mittaeter/migration.sql b/packages/db/prisma/migrations/20260904090000_taeter_und_mittaeter/migration.sql new file mode 100644 index 0000000..902c26b --- /dev/null +++ b/packages/db/prisma/migrations/20260904090000_taeter_und_mittaeter/migration.sql @@ -0,0 +1,13 @@ +-- Täter und Mittäter als Beteiligtenrollen. +-- +-- Damit wird eine Lücke sichtbar, die es schon vorher gab: ein Beteiligter darf +-- die Akte lesen, gleich in welcher Rolle. Bei „Betroffener" fiel das nicht auf, +-- bei „Täter" wäre es offensichtlich falsch — der Beschuldigte läse die +-- Ermittlung gegen sich selbst mit. +-- +-- Die Prüfung in lib/access/caseFilter.ts unterscheidet deshalb ab jetzt +-- Arbeitsrollen (LEAD, INVESTIGATOR, OBSERVER) von Angaben über den Fall +-- (WITNESS, SUBJECT, SUSPECT, ACCOMPLICE). Nur die ersten öffnen die Akte. + +ALTER TYPE "ParticipantRole" ADD VALUE IF NOT EXISTS 'SUSPECT'; +ALTER TYPE "ParticipantRole" ADD VALUE IF NOT EXISTS 'ACCOMPLICE'; diff --git a/packages/db/prisma/schema.prisma b/packages/db/prisma/schema.prisma index b8b5e02..f8e8285 100644 --- a/packages/db/prisma/schema.prisma +++ b/packages/db/prisma/schema.prisma @@ -410,12 +410,21 @@ model CaseParticipant { @@index([personnelFileId]) } +/// Rollen eines Beteiligten. +/// +/// Zwei Gruppen, und der Unterschied entscheidet ueber den Zugriff: LEAD, +/// INVESTIGATOR und OBSERVER **arbeiten** an der Akte und duerfen sie deshalb +/// lesen. WITNESS, SUBJECT, SUSPECT und ACCOMPLICE sind Angaben **ueber** den +/// Fall und verschaffen keinen Zugang — sonst laese der Beschuldigte die +/// Ermittlung gegen sich selbst mit. Siehe lib/cases/terms.ts. enum ParticipantRole { LEAD INVESTIGATOR + OBSERVER WITNESS SUBJECT - OBSERVER + SUSPECT + ACCOMPLICE } /// Schlaegt jede Freigabestufe. Deckt den Fall ab, dass gegen einen hochrangigen diff --git a/web/app/(app)/suche/page.tsx b/web/app/(app)/suche/page.tsx index 2798545..34899e2 100644 --- a/web/app/(app)/suche/page.tsx +++ b/web/app/(app)/suche/page.tsx @@ -3,7 +3,12 @@ import Link from 'next/link' import { db } from '@/lib/db' import { requireUser } from '@/lib/auth/current' import { parseFilters, toSearchParams, SEARCH_KINDS } from '@/lib/search/query' -import { searchCases, searchDocuments, searchPersonnel } from '@/lib/search/service' +import { + searchCases, + searchCounts, + searchDocuments, + searchPersonnel, +} from '@/lib/search/service' import { builtinViews } from '@/lib/search/views' import { de } from '@/lib/i18n/de' import { StampMark, classificationEdgeStyle } from '@/components/StampMark' @@ -44,12 +49,17 @@ export default async function SearchPage({ ]) const levelName = new Map(levels.map((l) => [l.level, l.name])) - const ergebnis = + const [ergebnis, zahlen] = await Promise.all([ filters.kind === 'document' - ? await searchDocuments(user.permissions, filters) + ? searchDocuments(user.permissions, filters) : filters.kind === 'personnel' - ? await searchPersonnel(user.permissions, filters) - : await searchCases(user.permissions, filters) + ? searchPersonnel(user.permissions, filters) + : searchCases(user.permissions, filters), + // Die Zahlen der anderen Arten. Ohne sie sucht man nach einem Namen, landet + // auf den Fallakten und erfährt nie, dass es dazu auch eine Personalakte + // gibt — der Reiter sieht leer aus wie jeder andere. + searchCounts(user.permissions, filters), + ]) const statusTags = filters.kind === 'personnel' ? await listStatusTags() : [] @@ -64,16 +74,26 @@ export default async function SearchPage({ -