„An alle" bei „An wen" – und kein Creator mehr auf der Team-Seite
=== screen6: „An alle" === Filipe: „bei an ween fehlt noch die option alle neben den namen allen." Der Knopf steht VORN, nicht hinten: Wer etwas an das ganze Team geben will, soll nicht erst an sechs Namen vorbeilesen. Er traegt die Anzahl der Leute und faerbt sich wie die Stufe „Alle" -- beide bedeuten „keine Einschraenkung", und zwei Farben dafuer waeren zwei Aussagen fuer einen Gedanken. Ab zwei Personen; bei einer waere „alle" derselbe Handgriff wie ihr Name. EINE ANFRAGE, NICHT SECHS. Der Browser koennte fuer jede Person einzeln fragen -- und beim dritten von sechs Aufrufen die Verbindung verlieren. Dann haette die Haelfte des Teams die Aufgabe und die andere nicht, und niemand saehe, wo es abgebrochen ist. DER DUPLIKAT-SCHUTZ WANDERTE IN DIE SCHLEIFE. Er fragt, was EINE Person schon liegen hat; stuende er davor, bekaeme nur die erste ihre Pruefung und alle anderen die Vorlage doppelt -- genau so entstanden am 22.09. aus zwei Klicks 112 Aufgaben. Gemessen: erster Klick 36 Aufgaben (12 mal 3 Modis), zweiter Klick 0 angelegt und 36 uebersprungen. Ein gesperrter Modi bekommt nichts, ein Creator auch nicht, und wer gar nicht verteilen darf, kommt ueber „alle" ebenfalls nicht weiter. === screen4: kein anderer Creator === Filipe: „in dieser app gibt es keinen und wird es niemals einen anderen creator geben wie mich." Auf der Aufgabenseite war eine Filterreihe mit „Alle Creator" beschriftet -- und darunter standen Modis. Auf der Team-Adresse gibt es keine Creator und wird es nie geben. Sie heisst dort jetzt „Für wen". DAS HAUS KOMMT VOM SERVER. `/api/ich` schickt jetzt `haus`; es stand schon an der Sitzung und wurde nie mitgeschickt. Eine Rollenliste im Browser waere die naechste zweite Wahrheit -- und falsch fuer DogFather, der in beiden Haeusern arbeitet. NEUN WEITERE „BEFUNDE" WAREN MEINE MESSFEHLER. Meine erste Messung lief ueber 127.0.0.1 und meldete unter anderem „CREATOR WORKSPACE" in der Kopfleiste JEDER Seite. Auf der echten Adresse steht dort „Team Dogi" -- `person.haus` wird aus dem Host abgeleitet, und auf 127.0.0.1 ist das Haus nun einmal die Agentur. Nachgemessen mit gesetztem Host: crew. -> haus="crew", marke="Team Dogi". Die Uebersichtsseite mit ihren Creator-Texten steht auf crew. gar nicht erst in den Kacheln. Gemessen: pruef-an-alle, 19 Pruefungen, 0 Fehler. Gruen: pruef-aufgabenbrett, pruef-aufgaben-vorlagen, pruef-vorlagen, pruef-entwicklung (48), pruef-css-klassen. Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
@@ -1903,16 +1903,44 @@ vorlagenRouter.post("/workspace/api/vorlagen/uebernehmen", gleicheHerkunft,
|
||||
if (!darfAufgabenVerteilen(req.person)) {
|
||||
return res.status(403).json({ fehler: "erst_bewerben" });
|
||||
}
|
||||
const zielId = Number(req.body.person_id) || null;
|
||||
if (!zielId) return res.status(400).json({ fehler: "Bitte zuerst eine Person wählen." });
|
||||
/* ==== „AN ALLE" (24.09.2026) ================================
|
||||
|
||||
Filipe: „bei an ween fehlt noch die option alle neben den
|
||||
namen allen."
|
||||
|
||||
EINE ANFRAGE, NICHT SECHS. Der Browser koennte fuer jede
|
||||
Person einzeln anfragen -- und beim dritten von sechs
|
||||
Aufrufen die Verbindung verlieren. Dann haette die Haelfte
|
||||
des Teams die Aufgabe und die andere nicht, und niemand
|
||||
saehe, wo es abgebrochen ist. Hier laeuft alles in einem
|
||||
Durchgang.
|
||||
|
||||
DER DUPLIKAT-SCHUTZ GILT JE PERSON. Er steht weiter unten
|
||||
und fragt, was diese eine Person schon hat -- bei „alle"
|
||||
wird er deshalb pro Person neu gestellt. Wer die Vorlage
|
||||
schon liegen hat, bekommt sie nicht doppelt; genau so
|
||||
entstanden am 22.09. die 112 Aufgaben aus zwei Klicks. */
|
||||
const anAlle = String(req.body.person_id) === "alle";
|
||||
const zielId = anAlle ? null : (Number(req.body.person_id) || null);
|
||||
if (!anAlle && !zielId) {
|
||||
return res.status(400).json({ fehler: "Bitte zuerst eine Person wählen." });
|
||||
}
|
||||
/* Nur an jemanden, fuer den die Kategorien ueberhaupt gelten --
|
||||
sonst landete eine Modi-Aufgabe samt Kategorie an einem
|
||||
Creator, und die Trennung waere von innen aufgeweicht. */
|
||||
const zielPerson = db().prepare(
|
||||
"SELECT id, rolle FROM personen WHERE id = ? AND aktiv = 1").get(zielId);
|
||||
if (!zielPerson || zielPerson.rolle !== "modi") {
|
||||
const zielListe = anAlle
|
||||
? db().prepare(
|
||||
"SELECT id, rolle FROM personen WHERE aktiv = 1 AND rolle = 'modi' ORDER BY id").all()
|
||||
: [db().prepare(
|
||||
"SELECT id, rolle FROM personen WHERE id = ? AND aktiv = 1").get(zielId)]
|
||||
.filter(Boolean);
|
||||
if (!zielListe.length || zielListe.some((z) => z.rolle !== "modi")) {
|
||||
return res.status(404).json({ fehler: "nicht_gefunden" });
|
||||
}
|
||||
/* Nur noch als Beleg, dass die Liste nicht leer ist -- gelesen
|
||||
wird sie in der Schleife weiter unten. Ein ungenutzter Wert
|
||||
waere Rauschen; dieser sagt etwas. */
|
||||
if (!zielListe[0]) return res.status(404).json({ fehler: "nicht_gefunden" });
|
||||
|
||||
/* AUS DEM KATALOG, DEN DIE SEITE AUCH BEKOMMEN HAT (16.09.2026).
|
||||
|
||||
@@ -1961,19 +1989,30 @@ vorlagenRouter.post("/workspace/api/vorlagen/uebernehmen", gleicheHerkunft,
|
||||
Dieselbe Sperre steht seit dem 10.09. im Termin-Zweig weiter
|
||||
unten. Dass sie hier fehlte, ist nicht aufgefallen, weil
|
||||
niemand zweimal drueckt -- bis es jemand tat. */
|
||||
const liegtSchon = art === "katalog_alle"
|
||||
? new Set(db().prepare(
|
||||
`SELECT vorlage FROM aufgaben
|
||||
WHERE verantwortlich_id = ?
|
||||
AND vorlage IS NOT NULL
|
||||
AND status NOT IN ('erledigt', 'abgebrochen')`)
|
||||
.all(zielPerson.id).map((z) => z.vorlage))
|
||||
: new Set();
|
||||
/* JE PERSON EINMAL (24.09.2026, fuer „An alle").
|
||||
|
||||
Die Duplikatsperre fragt, was DIESE Person schon liegen
|
||||
hat -- deshalb steht sie INNERHALB der Schleife. Waere sie
|
||||
davor, bekaeme bei „alle" nur die erste Person ihre
|
||||
Pruefung, und alle anderen haetten die Vorlage doppelt.
|
||||
|
||||
Bei einer einzelnen Person aendert sich nichts: Die
|
||||
Schleife laeuft dann genau einmal. */
|
||||
const lagSchonDa = [];
|
||||
for (const v of auswahl) {
|
||||
if (liegtSchon.has(v.schluessel)) { lagSchonDa.push(v.titel); continue; }
|
||||
katalogAufgabeAnlegen(v, zielPerson.id, req.person.id);
|
||||
angelegt.push(v.titel);
|
||||
for (const ziel of zielListe) {
|
||||
const liegtSchon = art === "katalog_alle"
|
||||
? new Set(db().prepare(
|
||||
`SELECT vorlage FROM aufgaben
|
||||
WHERE verantwortlich_id = ?
|
||||
AND vorlage IS NOT NULL
|
||||
AND status NOT IN ('erledigt', 'abgebrochen')`)
|
||||
.all(ziel.id).map((z) => z.vorlage))
|
||||
: new Set();
|
||||
for (const v of auswahl) {
|
||||
if (liegtSchon.has(v.schluessel)) { lagSchonDa.push(v.titel); continue; }
|
||||
katalogAufgabeAnlegen(v, ziel.id, req.person.id);
|
||||
angelegt.push(v.titel);
|
||||
}
|
||||
}
|
||||
/* DIE ZAHL GEHOERT IN DIE ANTWORT, nicht nur in die Datenbank.
|
||||
Ein Knopf, der "12 uebernehmen" heisst und dann schweigend
|
||||
|
||||
Reference in New Issue
Block a user