Ranggefuege: Rollen, die aufeinander aufbauen
Prüfung / Lint, Typen, Tests (push) Canceled after 0s
Prüfung / Images bauen (pdf) (push) Canceled after 0s
Prüfung / Images bauen (web) (push) Canceled after 0s
Prüfung / Images bauen (worker) (push) Canceled after 0s
Images / bauen (pdf, docker/pdf.Dockerfile) (push) Canceled after 0s
Images / bauen (web, docker/web.Dockerfile) (push) Canceled after 0s
Images / bauen (worker, docker/worker.Dockerfile) (push) Canceled after 0s
Prüfung / Lint, Typen, Tests (push) Canceled after 0s
Prüfung / Images bauen (pdf) (push) Canceled after 0s
Prüfung / Images bauen (web) (push) Canceled after 0s
Prüfung / Images bauen (worker) (push) Canceled after 0s
Images / bauen (pdf, docker/pdf.Dockerfile) (push) Canceled after 0s
Images / bauen (web, docker/web.Dockerfile) (push) Canceled after 0s
Images / bauen (worker, docker/worker.Dockerfile) (push) Canceled after 0s
Bisher trug jede Rolle ihre Rechte vollstaendig selbst. Bei achtzehn Rollen und zweiundzwanzig Rechten heisst das, dass eine hoehere Rangrolle die ganze Liste der darunterliegenden noch einmal abhaken muss - beim Anlegen fehleranfaellig und bei jeder spaeteren Aenderung doppelt zu pflegen. Ein Gefuege ist eine benannte, geordnete Folge von Rollen. Jede Sprosse erbt alles von jeder Sprosse unter ihr, verwaltet wird nur noch das, was oben dazukommt. Geerbt wird nicht nur die Rechteliste, sondern auch Freigabestufe und Anmeldung: sonst waere eine Befoerderung unter Umstaenden eine Herabstufung, naemlich immer dann, wenn Discord beim Befoerdern die alte Rangrolle abnimmt. Die Ordnung ist bewusst eine Reihenfolge und kein Zeiger je Rolle - ueber ganzen Zahlen gibt es keinen Zyklus, den man abfangen muesste. Aufloesen eines Gefueges laesst die Rollen stehen und nimmt nur das Geerbte weg. Die Rechnung steht als reine Funktion in lib/auth/ladders.ts und wird an drei Stellen benutzt: bei der Rechteaufloesung, in der Verwaltung und in der Vorschau des Editors, die im Browser mitrechnet. Eine Vorschau, die den Stand vor dem Verschieben zeigt, waere schlimmer als keine. Die Rollenseite zeigt Geerbtes kursiv und gesperrt an, damit dort niemand blind Kaestchen setzt, die laengst gesetzt sind. Nebenbei: Aenderungen an der Rollenzuordnung landen jetzt im Protokoll, mit Vorher und Nachher. Wer Rollen aendert, aendert die Rechte anderer - genau dafuer ist es da.
This commit is contained in:
@@ -13,6 +13,11 @@ export type RoleRowData = {
|
||||
category: string
|
||||
permissions: string[]
|
||||
isPresent: boolean
|
||||
/** Rechte, die nur aus dem Ranggefüge kommen. Hier nicht abwählbar. */
|
||||
inherited: string[]
|
||||
ladderName: string | null
|
||||
/** Stufe einschliesslich Gefüge - kann über der eigenen liegen. */
|
||||
effectiveClearanceLevel: number
|
||||
}
|
||||
|
||||
const CATEGORIES = ['MEMBERSHIP', 'RANK', 'CLEARANCE', 'UNSORTED'] as const
|
||||
@@ -80,6 +85,11 @@ export function RoleRow({
|
||||
{roleAdminText.missingFromGuild}
|
||||
</span>
|
||||
) : null}
|
||||
{role.ladderName ? (
|
||||
<span className="font-condensed text-2xs text-text-dim">
|
||||
{roleAdminText.inLadder} {role.ladderName}
|
||||
</span>
|
||||
) : null}
|
||||
</div>
|
||||
|
||||
<div className="mt-3 flex flex-wrap gap-x-8 gap-y-3 text-xs">
|
||||
@@ -117,6 +127,11 @@ export function RoleRow({
|
||||
</option>
|
||||
))}
|
||||
</select>
|
||||
{role.effectiveClearanceLevel > clearanceLevel ? (
|
||||
<span className="font-condensed text-2xs text-stamp-3">
|
||||
{roleAdminText.clearanceRaised} {role.effectiveClearanceLevel}
|
||||
</span>
|
||||
) : null}
|
||||
</label>
|
||||
|
||||
<label className="flex items-center gap-2">
|
||||
@@ -133,22 +148,44 @@ export function RoleRow({
|
||||
</div>
|
||||
|
||||
<fieldset className="mt-4">
|
||||
<legend className="text-xs text-text-dim">{roleAdminText.permissions}</legend>
|
||||
<legend className="text-xs text-text-dim">
|
||||
{roleAdminText.permissions}
|
||||
{role.inherited.length > 0 ? (
|
||||
<span className="ml-2 text-text-dim">{roleAdminText.inheritedNote}</span>
|
||||
) : null}
|
||||
</legend>
|
||||
<div className="mt-2 grid grid-cols-[repeat(auto-fill,minmax(13rem,1fr))] gap-x-4 gap-y-1.5">
|
||||
{PERMISSIONS.map((permission) => {
|
||||
const optIn = OPT_IN_PERMISSIONS.includes(permission)
|
||||
// Geerbtes ist hier nicht abwählbar: das Kästchen gehört zur
|
||||
// niedrigeren Sprosse. Es abzublenden wäre die schlechtere Antwort -
|
||||
// dann sähe die Rolle so aus, als könnte sie weniger, als sie kann.
|
||||
const geerbt = role.inherited.includes(permission)
|
||||
return (
|
||||
<label
|
||||
key={permission}
|
||||
className="flex items-center gap-2 font-mono text-2xs"
|
||||
title={optIn ? roleAdminText.optIn : undefined}
|
||||
title={
|
||||
geerbt
|
||||
? `${roleAdminText.inheritedFrom} ${role.ladderName ?? ''}`.trim()
|
||||
: optIn
|
||||
? roleAdminText.optIn
|
||||
: undefined
|
||||
}
|
||||
>
|
||||
<input
|
||||
type="checkbox"
|
||||
checked={permissions.includes(permission)}
|
||||
checked={geerbt || permissions.includes(permission)}
|
||||
disabled={geerbt}
|
||||
onChange={(event) => toggle(permission, event.target.checked)}
|
||||
/>
|
||||
<span className={optIn ? 'text-stamp-3' : undefined}>{permission}</span>
|
||||
<span
|
||||
className={
|
||||
geerbt ? 'text-text-dim italic' : optIn ? 'text-stamp-3' : undefined
|
||||
}
|
||||
>
|
||||
{permission}
|
||||
</span>
|
||||
</label>
|
||||
)
|
||||
})}
|
||||
|
||||
@@ -4,6 +4,7 @@ import { revalidatePath } from 'next/cache'
|
||||
import { z } from 'zod'
|
||||
import { db } from '@/lib/db'
|
||||
import { requirePermissionForAction } from '@/lib/auth/current'
|
||||
import { writeAudit } from '@/lib/audit/write'
|
||||
import { PERMISSIONS } from '@/lib/auth/permissions'
|
||||
import { invalidateAll } from '@/lib/auth/effective'
|
||||
import { de } from '@/lib/i18n/de'
|
||||
@@ -29,7 +30,7 @@ export type SaveResult = { ok: true } | { ok: false; error: string }
|
||||
* eigenen Prozess sofort ab, ohne auf die Runde über die Datenbank zu warten.
|
||||
*/
|
||||
export async function saveRoleMapping(input: unknown): Promise<SaveResult> {
|
||||
await requirePermissionForAction('roles.manage')
|
||||
const user = await requirePermissionForAction('roles.manage')
|
||||
|
||||
const parsed = schema.safeParse(input)
|
||||
if (!parsed.success) {
|
||||
@@ -41,14 +42,32 @@ export async function saveRoleMapping(input: unknown): Promise<SaveResult> {
|
||||
const existing = await db.roleMapping.findUnique({ where: { discordRoleId } })
|
||||
if (!existing) return { ok: false, error: de.roleAdmin.roleGone }
|
||||
|
||||
await db.roleMapping.update({
|
||||
where: { discordRoleId },
|
||||
data: {
|
||||
clearanceLevel,
|
||||
grantsMembership,
|
||||
category,
|
||||
permissions: [...new Set(permissions)].sort(),
|
||||
},
|
||||
const neue = [...new Set(permissions)].sort()
|
||||
|
||||
// Wer Rollen ändert, ändert die Rechte anderer. Genau dafür gibt es das
|
||||
// Protokoll (harte Regel 3) - mit dem Vorher, sonst steht dort nur, dass
|
||||
// jemand etwas geändert hat, aber nicht was.
|
||||
await db.$transaction(async (tx) => {
|
||||
await tx.roleMapping.update({
|
||||
where: { discordRoleId },
|
||||
data: { clearanceLevel, grantsMembership, category, permissions: neue },
|
||||
})
|
||||
await writeAudit(tx, {
|
||||
actorId: user.id,
|
||||
action: 'roles.update',
|
||||
targetType: 'RoleMapping',
|
||||
targetId: discordRoleId,
|
||||
clearanceAtTime: user.permissions.clearanceLevel,
|
||||
meta: {
|
||||
label: existing.label,
|
||||
from: {
|
||||
clearanceLevel: existing.clearanceLevel,
|
||||
grantsMembership: existing.grantsMembership,
|
||||
permissions: existing.permissions,
|
||||
},
|
||||
to: { clearanceLevel, grantsMembership, permissions: neue },
|
||||
},
|
||||
})
|
||||
})
|
||||
|
||||
invalidateAll()
|
||||
|
||||
@@ -2,6 +2,7 @@ import type { Metadata } from 'next'
|
||||
import { db } from '@/lib/db'
|
||||
import { requirePermission } from '@/lib/auth/current'
|
||||
import { de } from '@/lib/i18n/de'
|
||||
import { resolveAll, type LadderRung } from '@/lib/auth/ladders'
|
||||
import { RoleRow } from './RoleRow'
|
||||
import { ClearancePreview } from './ClearancePreview'
|
||||
|
||||
@@ -21,11 +22,18 @@ export default async function RoleAdminPage() {
|
||||
// Ohne roles.manage gibt es 404, nicht 403.
|
||||
await requirePermission('roles.manage')
|
||||
|
||||
const [mappings, levels] = await Promise.all([
|
||||
const [mappings, levels, ladders] = await Promise.all([
|
||||
db.roleMapping.findMany({ orderBy: [{ sortOrder: 'desc' }, { label: 'asc' }] }),
|
||||
db.clearanceLevelConfig.findMany({ orderBy: { level: 'asc' } }),
|
||||
db.roleLadder.findMany({ select: { id: true, name: true } }),
|
||||
])
|
||||
|
||||
// Was jede Rolle **tatsächlich** gewährt, einschliesslich ihres Ranggefüges.
|
||||
// Ohne das zeigte diese Seite die eigene Zeile und die Vorschau eine Stufe,
|
||||
// die für niemanden gilt - siehe lib/auth/ladders.ts.
|
||||
const aufgeloest = resolveAll(mappings as LadderRung[])
|
||||
const ladderName = new Map(ladders.map((leiter) => [leiter.id, leiter.name]))
|
||||
|
||||
return (
|
||||
<section>
|
||||
<h2 className="font-condensed">{de.roleAdmin.heading}</h2>
|
||||
@@ -39,7 +47,7 @@ export default async function RoleAdminPage() {
|
||||
levels={levels}
|
||||
mappings={mappings.map((m) => ({
|
||||
label: m.label,
|
||||
clearanceLevel: m.clearanceLevel,
|
||||
clearanceLevel: aufgeloest.get(m.discordRoleId)?.clearanceLevel ?? m.clearanceLevel,
|
||||
}))}
|
||||
/>
|
||||
|
||||
@@ -63,6 +71,15 @@ export default async function RoleAdminPage() {
|
||||
category: role.category,
|
||||
permissions: role.permissions,
|
||||
isPresent: role.isPresent,
|
||||
inherited: [
|
||||
...(aufgeloest.get(role.discordRoleId)?.inherited ?? []),
|
||||
].sort(),
|
||||
ladderName: role.ladderId
|
||||
? (ladderName.get(role.ladderId) ?? null)
|
||||
: null,
|
||||
effectiveClearanceLevel:
|
||||
aufgeloest.get(role.discordRoleId)?.clearanceLevel ??
|
||||
role.clearanceLevel,
|
||||
}}
|
||||
levels={levels.map((l) => ({ level: l.level, name: l.name }))}
|
||||
/>
|
||||
|
||||
Reference in New Issue
Block a user