Der Modi bekommt seine eigene Seite -- und sein Brett zurueck

DER WICHTIGSTE FUND, und er kam nicht aus dem Nachdenken: Ein Modi sah
auf dem Aufgabenbrett GAR NICHTS -- nicht einmal seine eigenen Aufgaben.
sichtbar() endet mit `default: 0=1`, und 'modi' stand nicht darin.

Genau dieser Fehler ist am 01.09.2026 schon dem Manager passiert; der
Kommentar zwei Zeilen darueber warnt woertlich davor ("Ein leeres Brett
sieht aus wie 'nichts zu tun', nicht wie ein Fehler; deshalb ist das
vermutlich lange niemandem aufgefallen"). Der Rollen-Rundgang meldete
fuer den Modi trotzdem brav "aufgaben.html ok" -- die Seite laedt ja,
sie war nur leer. Gefunden hat es erst eine Pruefung, die eine Aufgabe
ANLEGT und sie danach wiederzufinden versucht.

Ein Modi sieht jetzt die Aufgaben des ganzen Modi-Teams (Filipes
Entscheidung "sie sind untereinander ein Team"), aendern darf er
weiterhin nur seine eigenen. Die Nummern werden bei jeder Abfrage frisch
gelesen -- eine beim Serverstart gebaute Liste waere ab dem naechsten
neuen Modi falsch, und niemand wuesste warum.

DIE STARTSEITE. Ein Modi hatte keine einzige Kachel: Jede traegt eine
feste Rollenliste, und 'modi' darf dort nicht stehen -- bereiche.js
bekommt jeder ausgeliefert, der die Seite oeffnet. Die Kacheln kommen
deshalb vom Server (MODI_BEREICHE), samt Beschriftung. Nur die Ziele zu
schicken haette nicht gereicht: Unter "Dashboard" stuende sonst "Alle
Creator auf einen Blick" -- fuer jemanden ohne Creator. Die Worte
gehoeren zum Empfaenger, nicht zum Ziel.

Neun Kacheln in drei Gruppen: Aufgaben, Chat, Kalender, Dateien /
Live-Ablauf, Community, Technik / Profil, Wissen. Nichts aus der
Agentur -- diese Seiten drehen sich um betreute Creator oder um Rechte.

`null` heisst "nimm deine eigene Liste", eine LEERE Liste hiesse "keine
Kacheln". Verwechselte man die beiden, haetten die fuenf bekannten
Rollen ab sofort eine leere Startseite.

ZWEI DINGE HAT DAS BILDSCHIRMFOTO GEZEIGT, NICHT DER CODE:

  * Ueber der Modi-Startseite stand "Spicy Media" -- die Marke einer
    Agentur, mit der er nichts zu tun hat. Jetzt "Team Dogi", wie auf
    der oeffentlichen Seite. Ersetzt wird nur der Textknoten: In dem
    Element sitzen zwei Zierrauten, ein textContent haette sie lautlos
    geloescht.
  * Auf seinem Aufgabenbrett stand das Creator-Vorlagenbrett, 80
    Aufgaben fuer den Aufbau eines Kanals. Fuer einen Moderator ist
    davon nichts gedacht. Ausgeblendet, bis sein Katalog aus Teil 2 des
    Anforderungsdokuments da ist -- nichts ist ehrlicher als etwas
    Fremdes.

Dazu: "0 betreut" stand dauerhaft auf seiner Startseite, eine Zahl, die
nie etwas anderes sagen kann. Jetzt zaehlt sie, wie viele im Modi-Team
sind. Und der Satz unter der Begruessung war nur das Wort "Modi", neben
fuenf Rollen mit einem ganzen Satz -- das sah nicht verborgen aus,
sondern unfertig.

KATEGORIEN (Kapitel 6.1), nach Filipes Entscheidung nur bei den Modis.
Acht Stueck; hier steht, wohin die dreizehn aus Teil 2 fallen
(Branding/Team/Kommunikation -> Planung, Wachstum -> Community).

Der heikelste Fall ist nicht das Setzen, sondern das SCHICKEN durch
jemanden, der es nicht darf: Eine Absage ("Unbekannte Kategorie") waere
die Auskunft, dass es das Feld gibt. Also faellt der Wert lautlos weg
und die Aufgabe entsteht ganz normal. Wer die Kategorien benutzen darf,
bekommt bei einem Tippfehler dagegen sehr wohl eine Absage.

Das Feld erscheint nur, wenn die Aufgabe wirklich zu einem Modi gehoert
-- bei DogFather also erst, wenn er einen als Person auswaehlt. Sonst
stuende es auch an jeder Creator-Aufgabe. Verborgen heisst dabei auch
"nichts mitschicken": Ein Wert in einem unsichtbaren Feld wandert sonst
beim naechsten Speichern mit.

KEINE NEUE CSS-KLASSE fuer die Kategorie auf der Karte. Sie muesste in
sieben gleichlautenden Kopien der Modulliste gepflegt werden -- sieben
Gelegenheiten fuer einen Unterschied, fuer eine Zeile Text.

AUSSERDEM BERICHTIGT, UND ES WAR SCHON VORHER ROT: pruef-start-ansicht
erwartete drei Kachelgruppen. Seit b45de94 gibt es vier ("Rund um das
Team"). Bevor ich die Zahl angefasst habe, habe ich meine Aenderungen
beiseitegelegt und den Lauf wiederholt -- schon auf dem unveraenderten
Stand rot, also nicht von mir. Geprueft werden jetzt die NAMEN: Vier
Gruppen koennten auch drei richtige und eine doppelte sein.

GEPRUEFT: pruef-modi-verborgen (57), pruef-modi-kategorien (22, neu),
pruef-rollen (113 statt 97 -- der Modi laeuft jetzt ueber jede der 16
Seiten), pruef-start-ansicht, pruef-aufgabenbrett, pruef-sicht,
pruef-verborgen, pruef-personen-formular (24), pruef-personen-liste,
pruef-css-klassen, pruef-spicy (60).

Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
2026-09-10 00:52:06 +02:00
co-authored by Claude Opus 5
parent 7be488e0ca
commit 114a00eed5
30 changed files with 1196 additions and 268 deletions
+83 -7
View File
@@ -12,6 +12,7 @@ import express from "express";
import {
db, protokolliere, echteIp, sitzungLesen, betreutWo, istLeitung, ohneDogFather, heuteLokal, ROLLEN_SORTIERUNG, betreuteIds,
externPruefen, externSql, sichtbarePersonenIds, sichtbareCreatorIds,
MODI_KATEGORIEN, kategorienFuer,
} from "./workspace.js";
export const aufgabenRouter = express.Router();
@@ -99,6 +100,46 @@ export function sichtbar(person) {
? { wo: `(${eigen} OR ${b.wo})`, werte: [person.id, person.id, ...b.werte] }
: { wo: eigen, werte: [person.id, person.id] };
}
/* =================================================================
MODI (10.09.2026) -- UND FAST WAERE DERSELBE FEHLER PASSIERT WIE
AM 01.09.2026 BEIM MANAGER.
Er stand hier nicht, fiel in den Default und sah damit auf dem
Aufgabenbrett GAR NICHTS -- eigene Aufgaben eingeschlossen. Und
wie der Kommentar oben schon sagt: Ein leeres Brett sieht aus wie
"nichts zu tun", nicht wie ein Fehler. Der Rollen-Rundgang
(pruef-rollen) meldete fuer den Modi brav "aufgaben.html ok",
denn die Seite laedt ja, sie ist nur leer. Gefunden hat es erst
eine Pruefung, die eine Aufgabe ANLEGT und sie danach
wiederzufinden versucht.
WAS ER SIEHT: die Aufgaben des ganzen Modi-Teams. Filipes
Entscheidung vom 09.09.2026 -- "ja, sie sind untereinander ein
Team". Ohne das koennte niemand eine Schicht tauschen oder eine
Eskalation weiterreichen, ohne dass DogFather sie einzeln
durchstellt.
AENDERN DARF ER TROTZDEM NUR SEINE EIGENEN. Das entscheidet
darfAendern() weiter unten, und es bleibt dabei: Sehen, um sich
abzustimmen, ist etwas anderes als in fremder Arbeit schreiben.
WARUM DIE NUMMERN FRISCH GELESEN WERDEN und nicht einmal beim
Start: Kommt ein Modi dazu, muessen ihn die anderen sofort sehen.
Eine Liste, die beim Serverstart entsteht, waere bis zum naechsten
Neustart falsch -- und niemand wuesste, warum. */
case "modi": {
const modis = db().prepare(
"SELECT id FROM personen WHERE rolle = 'modi'").all().map((z) => z.id);
/* Kann nicht leer sein -- diese Person IST ein Modi. Die Abfrage
faende sonst `IN ()`, und das ist in SQLite ein Syntaxfehler. */
if (!modis.length) return { wo: "a.verantwortlich_id = ?", werte: [person.id] };
const platz = modis.map(() => "?").join(",");
return {
wo: `(a.creator_id IN (${platz}) OR a.verantwortlich_id IN (${platz})`
+ ` OR a.erstellt_von IN (${platz}))`,
werte: [...modis, ...modis, ...modis],
};
}
default:
return { wo: "0=1", werte: [] }; // unbekannte Rolle sieht nichts
}
@@ -123,6 +164,13 @@ const SPALTEN = `
/* Die Vorlage, aus der sie entstanden ist (09.09.2026) -- damit das
Vorlagenbrett zeigen kann, was schon geholt wurde. */
a.vorlage,
/* Die Kategorie (10.09.2026, Kapitel 6.1 des Anforderungsdokuments).
Sie kommt fuer JEDE Aufgabe mit -- wer sie nicht benutzen darf,
hat ohnehin keine Aufgabe mit einem Wert darin. Sie hier je nach
Rolle wegzulassen waere eine zweite Sichtbarkeitsregel neben der,
die es schon gibt, und damit eine zweite Gelegenheit, sie falsch
zu ziehen. */
a.kategorie,
pa.name AS abbruch_von_name`;
const VERBUND = `
@@ -163,7 +211,15 @@ aufgabenRouter.get("/workspace/api/aufgaben", (req, res) => {
CASE WHEN a.frist IS NULL THEN 1 ELSE 0 END, a.frist,
CASE a.prioritaet WHEN 'hoch' THEN 0 WHEN 'mittel' THEN 1 ELSE 2 END,
a.id DESC`).all(...werte, heuteLokal());
res.json({ aufgaben: reihen });
res.json({
aufgaben: reihen,
/* Die Kategorien kommen mit der Liste, nicht ueber einen eigenen
Aufruf: Das Brett braucht beides im selben Augenblick, und ein
zweiter Aufruf waere ein zweiter Weg, der einzeln scheitern
kann. Wer sie nicht benutzen darf, bekommt eine leere Liste --
und die Oberflaeche zeigt dann gar kein Feld. */
kategorien: kategorienFuer(req.person),
});
} catch (fehler) {
console.error("[workspace] Aufgaben lesen:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
@@ -278,7 +334,9 @@ aufgabenRouter.get("/workspace/api/personen", (req, res) => {
/* ---------- Anlegen und Ändern ------------------------------------------ */
function pruefeFelder(körper, { neu }) {
const KATEGORIEN = new Set(MODI_KATEGORIEN.map((k) => k.wert));
function pruefeFelder(körper, { neu, person }) {
const fehler = [];
const aus = {};
@@ -316,6 +374,24 @@ function pruefeFelder(körper, { neu }) {
else aus[feld] = zahl;
}
/* DIE KATEGORIE (10.09.2026).
STILLSCHWEIGEND FALLEN GELASSEN, NICHT ABGELEHNT, wenn jemand sie
schickt, der sie nicht benutzen darf. Eine Fehlermeldung
("Unbekannte Kategorie") waere die Auskunft, dass es das Feld gibt
-- und genau die soll niemand bekommen. Wer sie benutzen darf,
bekommt bei einem falschen Wert dagegen sehr wohl eine Absage:
Fuer ihn ist ein Tippfehler ein Fehler und kein Geheimnis.
Leer heisst ausdruecklich `null` und nicht "unveraendert" -- sonst
liesse sich eine einmal gesetzte Kategorie nie wieder entfernen. */
if (körper.kategorie !== undefined && kategorienFuer(person).length) {
const k = String(körper.kategorie ?? "").trim();
if (!k) aus.kategorie = null;
else if (!KATEGORIEN.has(k)) fehler.push("Unbekannte Kategorie.");
else aus.kategorie = k;
}
/* Frei eingetragene Namen statt einer Person -- eine Agentur, eine
Marke, jemand ohne Konto. Sie schlagen die Auswahl: Wer tippt,
meint das Getippte, und die Verknuepfung faellt weg. */
@@ -496,7 +572,7 @@ aufgabenRouter.delete("/workspace/api/notizen/:id", gleicheHerkunft, (req, res)
aufgabenRouter.post("/workspace/api/aufgaben", gleicheHerkunft, (req, res) => {
try {
const { aus, fehler } = pruefeFelder(req.body || {}, { neu: true });
const { aus, fehler } = pruefeFelder(req.body || {}, { neu: true, person: req.person });
if (fehler.length) return res.status(400).json({ fehler: fehler.join(" ") });
/* Wer nicht Management ist, darf ausschließlich für sich selbst
@@ -515,12 +591,12 @@ aufgabenRouter.post("/workspace/api/aufgaben", gleicheHerkunft, (req, res) => {
const { lastInsertRowid } = db().prepare(`
INSERT INTO aufgaben
(titel, beschreibung, status, prioritaet, creator_id, verantwortlich_id,
creator_extern, verantwortlich_extern, frist, erstellt, erstellt_von)
VALUES (?,?,?,?,?,?,?,?,?,?,?)`).run(
creator_extern, verantwortlich_extern, frist, kategorie, erstellt, erstellt_von)
VALUES (?,?,?,?,?,?,?,?,?,?,?,?)`).run(
aus.titel, aus.beschreibung ?? null, aus.status ?? "offen",
aus.prioritaet ?? "mittel", aus.creator_id ?? null, aus.verantwortlich_id ?? null,
aus.creator_extern ?? null, aus.verantwortlich_extern ?? null,
aus.frist ?? null, jetzt(), req.person.id);
aus.frist ?? null, aus.kategorie ?? null, jetzt(), req.person.id);
protokolliere("aufgabe_angelegt", {
personId: req.person.id, rolle: req.person.rolle, ip: echteIp(req),
@@ -547,7 +623,7 @@ aufgabenRouter.patch("/workspace/api/aufgaben/:id", gleicheHerkunft, (req, res)
if (!aufgabe) return res.status(404).json({ fehler: "nicht_gefunden" });
if (!darfAendern(req.person, aufgabe)) return res.status(403).json({ fehler: "nicht_erlaubt" });
const { aus, fehler } = pruefeFelder(req.body || {}, { neu: false });
const { aus, fehler } = pruefeFelder(req.body || {}, { neu: false, person: req.person });
if (fehler.length) return res.status(400).json({ fehler: fehler.join(" ") });
/* Zuordnungen darf nur das Management verschieben. */