Runde Bedienelemente, Admin zusammengelegt, Kanäle entfallen
Vier der acht Änderungswünsche. **Knöpfe.** Rund vierzig der siebzig Knöpfe waren blosser Text in Akzentfarbe, von einem Link nicht zu unterscheiden. Jeder trägt jetzt Rahmen und Grundfläche, über die Klasse .btn an einer Stelle definiert statt siebzigmal als Klassenkette. Beim Drücken zieht der Knopf sich zusammen und bekommt einen inneren Ring — er wirkt kurz dicker, ohne dass die Rahmenbreite sich ändert und das Umfeld springt; die Rückmeldung setzt ohne Verzögerung ein und klingt langsam ab, sonst bliebe ein kurzer Klick unsichtbar. Dazu die neue Regel: Flächen bleiben eckig, alles Bedienbare ist rund. Damit ist die Rundung eine Auskunft und keine Verzierung. **Verwaltung.** Rollen, Vorlagen und Änderungsrichtlinie waren drei Einträge in der Kopfzeile für etwas, das man selten und dann zusammenhängend braucht. Jetzt ein Eintrag /admin mit Reiterleiste, dazu eine gemeinsame Tabs-Komponente, die auch der Fallakte zeigt, welcher Reiter offen ist. **Kanäle.** Ersatzlos entfernt, mitsamt Tabelle. Der Chat gehört dorthin, wo der Vorgang steht; ein eigener Reiter mit freien Räumen war der Rückweg nach Discord. Die Personalakte hat jetzt denselben Chat wie die Fallakte — ohne „als Vermerk übernehmen", denn dafür braucht es einen Verlauf, und Meldungen gehen nur an Erwähnte, nie in den allgemeinen Kanal: dass es zu jemandem eine Personalakte gibt, ist selbst eine Auskunft. Ein CHECK erzwingt jetzt, dass eine Nachricht zu genau einem Ort gehört. **Kopfzeile.** Suchbalken statt Suchlink, als gewöhnliches Formular ohne ein Byte im Bündel. Thema, Benachrichtigungen und Abmelden sitzen hinter dem Profilbild, das von Discord kommt; wer keins hat, bekommt seine Initialen. Nebenbei: Aktenzeichen in Tests waren nur auf vier Stellen eindeutig und kollidierten zwischen Läufen, weil Akten nicht löschbar sind.
This commit is contained in:
+52
-36
@@ -1,7 +1,7 @@
|
||||
// @vitest-environment node
|
||||
import { afterAll, beforeAll, describe, expect, it } from 'vitest'
|
||||
import { db } from '@/lib/db'
|
||||
import { visibleChannelWhere, visibleMessageWhere } from '@/lib/access/chatFilter'
|
||||
import { visibleMessageWhere } from '@/lib/access/chatFilter'
|
||||
import { adoptAsEntry, listMessages, sendMessage } from '@/lib/chat/service'
|
||||
import { resolveCasePreviews } from '@/lib/chat/preview'
|
||||
import { AccessError } from '@/lib/errors'
|
||||
@@ -37,8 +37,8 @@ suite('Aktenchat und Kanäle', () => {
|
||||
let geheimNummer = ''
|
||||
let offenId = ''
|
||||
let geheimId = ''
|
||||
let kanalOffen = ''
|
||||
let kanalGeheim = ''
|
||||
let personOffen = ''
|
||||
let personGeheim = ''
|
||||
|
||||
beforeAll(async () => {
|
||||
const vorlage = await db.documentTemplate.create({
|
||||
@@ -56,7 +56,7 @@ suite('Aktenchat und Kanäle', () => {
|
||||
const offen = await db.case.create({
|
||||
data: {
|
||||
...gemeinsam,
|
||||
caseNumber: `Chat-2026-${String(Date.now()).slice(-4)}`,
|
||||
caseNumber: `Chat-2026-${Date.now().toString(36)}${Math.random().toString(36).slice(2, 6)}`,
|
||||
title: 'Offener Vorgang',
|
||||
classification: 1,
|
||||
status: 'OPEN',
|
||||
@@ -65,7 +65,7 @@ suite('Aktenchat und Kanäle', () => {
|
||||
const geheim = await db.case.create({
|
||||
data: {
|
||||
...gemeinsam,
|
||||
caseNumber: `Gehm-2026-${String(Date.now()).slice(-4)}`,
|
||||
caseNumber: `Gehm-2026-${Date.now().toString(36)}${Math.random().toString(36).slice(2, 6)}`,
|
||||
title: 'Streng vertraulich',
|
||||
classification: 4,
|
||||
status: 'OPEN',
|
||||
@@ -76,24 +76,14 @@ suite('Aktenchat und Kanäle', () => {
|
||||
offenId = offen.id
|
||||
geheimId = geheim.id
|
||||
|
||||
const k1 = await db.channel.create({
|
||||
data: {
|
||||
key: `${marke}-allgemein`.slice(0, 40),
|
||||
name: 'Allgemein',
|
||||
minClearance: 0,
|
||||
createdById: nutzer.stufe4,
|
||||
},
|
||||
const p1 = await db.personnelFile.create({
|
||||
data: { displayName: `${marke}-offen`, classification: 1, createdById: nutzer.stufe4 },
|
||||
})
|
||||
const k2 = await db.channel.create({
|
||||
data: {
|
||||
key: `${marke}-leitung`.slice(0, 40),
|
||||
name: 'Leitung',
|
||||
minClearance: 4,
|
||||
createdById: nutzer.stufe4,
|
||||
},
|
||||
const p2 = await db.personnelFile.create({
|
||||
data: { displayName: `${marke}-geheim`, classification: 4, createdById: nutzer.stufe4 },
|
||||
})
|
||||
kanalOffen = k1.key
|
||||
kanalGeheim = k2.key
|
||||
personOffen = p1.id
|
||||
personGeheim = p2.id
|
||||
})
|
||||
|
||||
afterAll(async () => {
|
||||
@@ -119,17 +109,37 @@ suite('Aktenchat und Kanäle', () => {
|
||||
expect(sichtbar).toHaveLength(0)
|
||||
})
|
||||
|
||||
it('verbirgt einen Kanal über der eigenen Stufe', async () => {
|
||||
const kanaele = await db.channel.findMany({
|
||||
where: {
|
||||
AND: [
|
||||
visibleChannelWhere(perms(nutzer.stufe1, 1)),
|
||||
{ key: { in: [kanalOffen, kanalGeheim] } },
|
||||
],
|
||||
},
|
||||
select: { key: true },
|
||||
it('verbirgt den Chat einer zu hoch eingestuften Personalakte', async () => {
|
||||
const leiter = perms(nutzer.stufe4, 4, { permissions: ['personnel.read'] })
|
||||
await sendMessage(leiter, {
|
||||
id: `${marke}-person-geheim`,
|
||||
scope: { kind: 'personnel', fileId: personGeheim },
|
||||
body: 'Bewertung liegt vor',
|
||||
})
|
||||
expect(kanaele.map((k) => k.key)).toEqual([kanalOffen])
|
||||
|
||||
// Zu niedrige Stufe: den Ort gibt es für ihn nicht.
|
||||
await expect(
|
||||
listMessages(perms(nutzer.stufe1, 1, { permissions: ['personnel.read'] }), {
|
||||
kind: 'personnel',
|
||||
fileId: personGeheim,
|
||||
}),
|
||||
).rejects.toThrow(AccessError)
|
||||
})
|
||||
|
||||
it('lässt ohne das Recht auf Personalakten auch deren Chat nicht zu', async () => {
|
||||
// Die Stufe allein reicht nicht: Personalakten setzen personnel.read
|
||||
// zusätzlich voraus, und das gilt für ihren Chat genauso.
|
||||
await expect(
|
||||
listMessages(perms(nutzer.stufe4, 4), { kind: 'personnel', fileId: personOffen }),
|
||||
).rejects.toThrow(AccessError)
|
||||
|
||||
const ohneRecht = await db.chatMessage.findMany({
|
||||
where: {
|
||||
AND: [visibleMessageWhere(perms(nutzer.stufe4, 4)), { personnelFileId: personGeheim }],
|
||||
},
|
||||
select: { id: true },
|
||||
})
|
||||
expect(ohneRecht).toHaveLength(0)
|
||||
})
|
||||
|
||||
it('verrät über die Vorschaukarte nichts über eine verborgene Akte', async () => {
|
||||
@@ -214,12 +224,18 @@ suite('Aktenchat und Kanäle', () => {
|
||||
expect((await adoptAsEntry(p, id)).entryId).toBe(entryId)
|
||||
})
|
||||
|
||||
it('macht aus einer Kanalnachricht keinen Vermerk', async () => {
|
||||
const id = `${marke}-kanal`
|
||||
const p = perms(nutzer.stufe4, 4, { permissions: ['entry.create', 'case.edit'] })
|
||||
await sendMessage(p, { id, scope: { kind: 'channel', key: kanalOffen }, body: 'Absprache' })
|
||||
it('macht aus einer Nachricht der Personalakte keinen Vermerk', async () => {
|
||||
const id = `${marke}-person`
|
||||
const p = perms(nutzer.stufe4, 4, {
|
||||
permissions: ['entry.create', 'case.edit', 'personnel.read'],
|
||||
})
|
||||
await sendMessage(p, {
|
||||
id,
|
||||
scope: { kind: 'personnel', fileId: personOffen },
|
||||
body: 'Absprache',
|
||||
})
|
||||
|
||||
// Ein Vermerk gehört in eine Akte. Ein Kanal hat keine.
|
||||
// Ein Vermerk gehört in den Verlauf, und den führt nur die Fallakte.
|
||||
await expect(adoptAsEntry(p, id)).rejects.toThrow(AccessError)
|
||||
})
|
||||
})
|
||||
|
||||
Reference in New Issue
Block a user