Bisher hatten Scouts mit der Creator-Betreuung nichts zu tun -- keine Profile, keine Betreuungsbereiche, keine Reports. Das aendert sich, mit zwei bewusst gesetzten Grenzen. GRENZE 1: nur zugeteilte Creator, keine Rollenregel. Neue Tabelle betreuung (creator_id PRIMARY KEY -> betreuer_id). Ein Creator hat genau EINE zustaendige Person, damit nie unklar ist, wer gefragt ist. Das Management sieht ohnehin alle und braucht keinen Eintrag. Wer nichts zugeteilt bekommt, sieht weiterhin nichts -- kein Recht entsteht automatisch aus der Rolle. Zugeteilt wird in "Personen & Zugaenge", direkt in der Personenzeile: Betreuung ist eine Eigenschaft der Person, kein eigener Vorgang. Nur das Management darf zuteilen -- koennte ein Scout sich selbst Creator geben, haette er die Rechtevergabe in der Hand, die ihn begrenzen soll. Zustaendig koennen nur aktive Scouts sein, kein Admin (der sieht alles) und kein anderer Creator. Bei der Uebergabe aus der Pipeline passiert die Zuteilung von selbst: Wer jemanden gefunden hat, betreut ihn weiter. Genau darum geht es bei "Creator-Onboarding starten". Umhaengen kann das Management jederzeit. GRENZE 2: betreuen, nicht verwalten. Profile, die fuenf Bereiche und Reports wie ein Manager. ABER: - keine Zugangscodes, kein Sperren von Personen (personen.html bleibt admin-only, unveraendert) - keine management-internen Felder. Der Scout bekommt admin_notiz, plan_start und naechster_review NICHT -- die Felder fehlen in der Antwort komplett, nicht nur in der Anzeige. Eine Notiz UEBER die Betreuung gehoert nicht in die Hand dessen, der betreut. Geprueft: Ein Scout, der admin_notiz mitschickt, aendert sie nicht. Die Regel steht an EINER Stelle (betreuteIds / betreutWo / darfCreator in workspace.js) und wird von sechs Modulen benutzt. Eine Rechteregel, die an sechs Stellen steht, ist eine Rechteregel, die irgendwann an fuenf Stellen stimmt. Genau das ist beim Bauen auch passiert: workspace-calls.js hatte eine wortgleiche Kopie der Kalender-Sichtbarkeit. Erweitert wurde nur der Kalender -- Scouts sahen die Termine ihrer Creator, dieselben Termine als Call aber nicht. Die Kopie ist jetzt weg, calls.js importiert die Regel aus workspace-kalender.js. Zwei Fehler, die die Aenderung selbst erzeugt haette, vorher gefunden: - Report-Entscheidung: ein Scout haette eine Aufgabe angelegt, deren "Creator" er selbst ist -- die waere in jeder Auswertung falsch mitgelaufen. Zeigt jetzt auf einen seiner Creator. - Bereichseintrag: derselbe Fehler. Ein Scout hat gar keinen eigenen Betreuungsbereich. Ein Eintrag ohne oder mit fremder Zuordnung landet beim ersten zugeteilten Creator, nie bei einem fremden. Geprueft: Mikas Bereich bleibt bei jedem Versuch unberuehrt.
201 lines
8.3 KiB
JavaScript
201 lines
8.3 KiB
JavaScript
/* =====================================================================
|
|
workspace-profil.js — Creator-Profile (Onboarding-Teil des Konzepts):
|
|
Stammdaten, Ziele, 90-Tage-Plan.
|
|
|
|
Sicherheitskern dieses Moduls ist das Feld `admin_notiz`. Das Konzept
|
|
fordert "private Admin-Notizen separat" -- diese Notiz darf ein Creator
|
|
unter keinen Umständen sehen, auch nicht als Nebeneffekt einer
|
|
Sammelabfrage. Deshalb wird sie nicht nur im Browser ausgeblendet,
|
|
sondern gar nicht erst mitgeschickt: Die Spaltenliste der Abfrage
|
|
haengt an der Rolle (siehe FELDER_OFFEN / FELDER_ADMIN).
|
|
===================================================================== */
|
|
|
|
import express from "express";
|
|
import {
|
|
db, protokolliere, echteIp, sitzungLesen, darfCreator, betreuteIds,
|
|
} from "./workspace.js";
|
|
|
|
export const profilRouter = express.Router();
|
|
|
|
/* Felder, die auch der Creator selbst sehen und ändern darf. */
|
|
const FELDER_OFFEN = [
|
|
"handles", "nische", "live_zeiten", "technik",
|
|
"ziel_live", "ziel_content", "ziel_community", "ziel_technik",
|
|
"plan_prio1", "plan_prio2", "plan_prio3",
|
|
];
|
|
/* Nur fürs Management. `admin_notiz` ist der eigentliche Grund für die
|
|
Trennung, die Termine stehen hier, weil sie zur Steuerung gehören. */
|
|
const FELDER_ADMIN = ["plan_start", "naechster_review", "admin_notiz"];
|
|
|
|
const DATUMSFELDER = new Set(["plan_start", "naechster_review"]);
|
|
const TEXT_MAX = 2000;
|
|
|
|
const jetzt = () => new Date().toISOString();
|
|
|
|
function angemeldet(req, res, next) {
|
|
const person = sitzungLesen(req);
|
|
if (!person) return res.status(401).json({ fehler: "nicht_angemeldet" });
|
|
req.person = person;
|
|
next();
|
|
}
|
|
|
|
function gleicheHerkunft(req, res, next) {
|
|
const herkunft = req.get("origin");
|
|
if (!herkunft) return next();
|
|
let erlaubt;
|
|
try { erlaubt = new URL(herkunft).host === req.get("host"); } catch { erlaubt = false; }
|
|
if (!erlaubt) return res.status(403).json({ fehler: "fremde_herkunft" });
|
|
next();
|
|
}
|
|
|
|
profilRouter.use("/workspace/api/profil", angemeldet);
|
|
|
|
/* Wer darf welches Profil?
|
|
|
|
"voll" = alle Felder, auch die management-internen (Plan-Start,
|
|
Review-Termin, interne Notiz). Nur das Management.
|
|
"betreuend"= ein Scout bei einem ihm zugeteilten Creator. Er arbeitet
|
|
mit dem Profil wie ein Manager, sieht aber die internen
|
|
Notizen NICHT -- eine Notiz UEBER die Betreuung gehoert
|
|
nicht in die Hand dessen, der betreut.
|
|
"eigen" = der Creator selbst.
|
|
null = 404. */
|
|
function darf(person, zielId) {
|
|
if (person.rolle === "admin") return "voll";
|
|
if (person.rolle === "creator") return person.id === zielId ? "eigen" : null;
|
|
if (person.rolle === "scout") return darfCreator(person, zielId) ? "betreuend" : null;
|
|
return null;
|
|
}
|
|
|
|
/* ---------- Lesen ------------------------------------------------------- */
|
|
|
|
profilRouter.get("/workspace/api/profil/:id", (req, res) => {
|
|
try {
|
|
const id = Number(req.params.id);
|
|
if (!Number.isInteger(id)) return res.status(400).json({ fehler: "ungueltig" });
|
|
|
|
const recht = darf(req.person, id);
|
|
/* 404 statt 403: Wer kein Recht hat, soll nicht erfahren, ob es die
|
|
Person überhaupt gibt. */
|
|
if (!recht) return res.status(404).json({ fehler: "nicht_gefunden" });
|
|
|
|
const person = db().prepare(
|
|
"SELECT id, name, rolle, aktiv, erstellt, letzter_login FROM personen WHERE id = ?").get(id);
|
|
if (!person || person.rolle !== "creator") {
|
|
return res.status(404).json({ fehler: "nicht_gefunden" });
|
|
}
|
|
|
|
const felder = recht === "voll" ? [...FELDER_OFFEN, ...FELDER_ADMIN] : FELDER_OFFEN;
|
|
const reihe = db().prepare(
|
|
`SELECT ${felder.join(", ")}, geaendert FROM profile WHERE person_id = ?`).get(id);
|
|
|
|
res.json({
|
|
person,
|
|
/* Kein Profil angelegt = leeres Objekt, kein Fehler. */
|
|
profil: reihe || Object.fromEntries(felder.map((f) => [f, null])),
|
|
darf_alles: recht === "voll",
|
|
});
|
|
} catch (fehler) {
|
|
console.error("[workspace] Profil lesen:", fehler?.message);
|
|
res.status(503).json({ fehler: "nicht_verfuegbar" });
|
|
}
|
|
});
|
|
|
|
/* Liste aller Creator -- fürs Management, zur Auswahl. */
|
|
profilRouter.get("/workspace/api/profil", (req, res) => {
|
|
try {
|
|
if (req.person.rolle === "creator") {
|
|
return res.json({ creator: [{ id: req.person.id, name: req.person.name }], eigen: true });
|
|
}
|
|
/* Ein Scout bekommt genau die Creator, die er betreut -- und ohne
|
|
Zuteilung eine leere Liste statt einer Fehlermeldung. */
|
|
let nur = "";
|
|
let werte = [];
|
|
if (req.person.rolle === "scout") {
|
|
const ids = betreuteIds(req.person);
|
|
if (!ids.length) return res.json({ creator: [], eigen: false, betreuend: true });
|
|
nur = ` AND p.id IN (${ids.map(() => "?").join(",")})`;
|
|
werte = ids;
|
|
} else if (req.person.rolle !== "admin") {
|
|
return res.status(404).json({ fehler: "nicht_gefunden" });
|
|
}
|
|
res.json({
|
|
creator: db().prepare(`
|
|
SELECT p.id, p.name, p.aktiv, p.letzter_login,
|
|
(SELECT COUNT(*) FROM aufgaben a
|
|
WHERE a.creator_id = p.id AND a.status <> 'erledigt') AS offene_aufgaben,
|
|
(SELECT naechster_review FROM profile f WHERE f.person_id = p.id) AS naechster_review,
|
|
(SELECT geaendert FROM profile f WHERE f.person_id = p.id) AS profil_geaendert
|
|
FROM personen p WHERE p.rolle = 'creator'${nur}
|
|
ORDER BY p.aktiv DESC, p.name`).all(...werte),
|
|
eigen: false,
|
|
betreuend: req.person.rolle === "scout",
|
|
});
|
|
} catch (fehler) {
|
|
console.error("[workspace] Creator-Liste:", fehler?.message);
|
|
res.status(503).json({ fehler: "nicht_verfuegbar" });
|
|
}
|
|
});
|
|
|
|
/* ---------- Speichern ---------------------------------------------------- */
|
|
|
|
profilRouter.put("/workspace/api/profil/:id", gleicheHerkunft, (req, res) => {
|
|
try {
|
|
const id = Number(req.params.id);
|
|
if (!Number.isInteger(id)) return res.status(400).json({ fehler: "ungueltig" });
|
|
|
|
const recht = darf(req.person, id);
|
|
if (!recht) return res.status(404).json({ fehler: "nicht_gefunden" });
|
|
|
|
const person = db().prepare("SELECT id, rolle FROM personen WHERE id = ?").get(id);
|
|
if (!person || person.rolle !== "creator") return res.status(404).json({ fehler: "nicht_gefunden" });
|
|
|
|
/* Ein Creator kann die Verwaltungsfelder nicht setzen -- sie stehen
|
|
gar nicht erst in der Liste der zulässigen Felder. Was er trotzdem
|
|
mitschickt, wird stillschweigend ignoriert. */
|
|
const erlaubt = recht === "voll" ? [...FELDER_OFFEN, ...FELDER_ADMIN] : FELDER_OFFEN;
|
|
|
|
const werte = {};
|
|
for (const feld of erlaubt) {
|
|
if (req.body?.[feld] === undefined) continue;
|
|
let wert = req.body[feld];
|
|
if (wert === null) { werte[feld] = null; continue; }
|
|
wert = String(wert).trim();
|
|
if (!wert) { werte[feld] = null; continue; }
|
|
if (DATUMSFELDER.has(feld)) {
|
|
if (!/^\d{4}-\d{2}-\d{2}$/.test(wert) || Number.isNaN(Date.parse(wert))) {
|
|
return res.status(400).json({ fehler: `Kein gültiges Datum: ${feld}` });
|
|
}
|
|
} else if (wert.length > TEXT_MAX) {
|
|
return res.status(400).json({ fehler: "Ein Feld ist zu lang." });
|
|
}
|
|
werte[feld] = wert;
|
|
}
|
|
if (!Object.keys(werte).length) return res.status(400).json({ fehler: "nichts_zu_speichern" });
|
|
|
|
const spalten = Object.keys(werte);
|
|
/* UPSERT: Beim ersten Speichern entsteht die Zeile, danach wird sie
|
|
aktualisiert -- ohne vorher prüfen zu müssen, ob es sie gibt. */
|
|
db().prepare(`
|
|
INSERT INTO profile (person_id, ${spalten.join(", ")}, geaendert, geaendert_von)
|
|
VALUES (?, ${spalten.map(() => "?").join(", ")}, ?, ?)
|
|
ON CONFLICT(person_id) DO UPDATE SET
|
|
${spalten.map((s) => `${s} = excluded.${s}`).join(", ")},
|
|
geaendert = excluded.geaendert,
|
|
geaendert_von = excluded.geaendert_von
|
|
`).run(id, ...spalten.map((s) => werte[s]), jetzt(), req.person.id);
|
|
|
|
protokolliere("profil_gespeichert", {
|
|
personId: req.person.id, rolle: req.person.rolle, ip: echteIp(req),
|
|
/* Nur die Feldnamen, nie die Inhalte -- im Profil stehen persönliche
|
|
Angaben, die nicht zusätzlich im Protokoll landen sollen. */
|
|
detail: `Creator #${id}: ${spalten.join(",")}`.slice(0, 120),
|
|
});
|
|
|
|
res.json({ ok: true });
|
|
} catch (fehler) {
|
|
console.error("[workspace] Profil speichern:", fehler?.message);
|
|
res.status(503).json({ fehler: "nicht_verfuegbar" });
|
|
}
|
|
});
|