Workspace: Rolle "Management" heisst jetzt "DogFather"
Auf Wunsch von Filipe. Betrifft ausschliesslich die Anzeige.
Der Rollenschluessel bleibt "admin". Er steckt in der CHECK-Regel der
Datenbank, in jeder bestehenden Sitzung und in jeder Rechteabfrage --
ihn umzubenennen haette alle drei gebrochen, und bestehende Anmeldungen
waeren ungueltig geworden. Umbenannt wird nur, was man LIEST.
Der Anzeigename steht jetzt an EINER Stelle (ROLLEN_NAME in
workspace.js) und kommt ueber /workspace/api/ich als rolle_name mit.
Stand er in elf Dateien, waere er beim naechsten Mal in zehn davon
geaendert.
Dabei aufgefallen: In der Kopfleiste stand auf JEDER Seite der rohe
Rollenschluessel -- "Chef · admin". Das war schon vorher unschoen, faellt
aber jetzt erst richtig auf. Alle elf Seiten zeigen jetzt "Chef ·
DogFather". Geprueft: kein rohes "admin" mehr in der Oberflaeche.
Geaendert: Rollenkachel und Untertitel auf der Anmeldeseite, Kopfleiste
aller Seiten, Rollentext auf dem Dashboard, Rollenmarken und Auswahl in
der Personenverwaltung, die Betreuungs-Auswahl ("nur DogFather"), der
Hinweis auf der Dateienseite, die Erklaerung zur internen Notiz im
Profil, die Hinweise fuer Scouts ohne Zuteilung -- und vier
Fehlermeldungen vom Server.
Schreibweise "DogFather" wie von Filipe geschrieben; das ist auch auf
der oeffentlichen Website die haeufigste Form (717 von 1216).
In den Code-Kommentaren der Fachmodule heisst die Rolle weiterhin "das
Management". Das bleibt bewusst so -- eine Massenaenderung an vierzig
Kommentaren waere reines Risiko ohne sichtbaren Nutzen. Ein Hinweis an
der ROLLEN_NAME-Zuordnung erklaert den Zusammenhang.
This commit is contained in:
@@ -129,7 +129,7 @@ function nameAusKopf(req) {
|
||||
function darfHochladen(req, res, next) {
|
||||
if (!DARF_HOCHLADEN.has(req.person.rolle)) {
|
||||
return res.status(403).json({
|
||||
fehler: "Dateien legen das Management und die Scouts ab.",
|
||||
fehler: "Dateien legen DogFather und die Scouts ab.",
|
||||
});
|
||||
}
|
||||
next();
|
||||
@@ -263,10 +263,10 @@ dateienRouter.patch("/workspace/api/dateien/:id", gleicheHerkunft, (req, res) =>
|
||||
/* "freigegeben" ist eine Abnahme -- die trifft laut Konzept das
|
||||
Management, nicht der, der die Datei hochgeladen hat. */
|
||||
if (req.body.status === "freigegeben" && req.person.rolle !== "admin") {
|
||||
return res.status(403).json({ fehler: "Freigeben darf nur das Management." });
|
||||
return res.status(403).json({ fehler: "Freigeben darf nur DogFather." });
|
||||
}
|
||||
if (datei.status === "freigegeben" && req.person.rolle !== "admin") {
|
||||
return res.status(403).json({ fehler: "Freigegebene Dateien ändert nur das Management." });
|
||||
return res.status(403).json({ fehler: "Freigegebene Dateien ändert nur DogFather." });
|
||||
}
|
||||
setz.push("status = ?"); daten.push(req.body.status);
|
||||
}
|
||||
@@ -309,7 +309,7 @@ dateienRouter.delete("/workspace/api/dateien/:id", gleicheHerkunft, (req, res) =
|
||||
return res.status(403).json({ fehler: "nicht_erlaubt" });
|
||||
}
|
||||
if (datei.status === "freigegeben") {
|
||||
return res.status(403).json({ fehler: "Freigegebene Dateien löscht nur das Management." });
|
||||
return res.status(403).json({ fehler: "Freigegebene Dateien löscht nur DogFather." });
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user