Aufgaben: wer verantwortlich ist, hat auch eine Zuteilung
VanVan im Support: „Wenn ein Modi sich auf eine Aufgabe beworben hat
und die von uns angenommen wurde, dann steht beim Modi unter der
Aufgabe immer noch ich bewerbe mich."
GEMESSEN AN DEN ECHTEN DATEN (lesend, auf einer Kopie):
Aufgaben mit Verantwortlichem: 11
davon OHNE Zuteilungszeile: 8
Darunter Marinas zwei offene Vorlagen-Aufgaben -- genau die zwei
Karten aus ihrem Bildschirmfoto.
DIE URSACHE. `katalogAufgabeAnlegen()` schrieb eine Zeile in
`aufgaben` mit `verantwortlich_id` und KEINE in
`aufgaben_zuteilung`. Das Brett liest aber die Zuteilung, nicht den
Verantwortlichen: Es fand nichts, lieferte `meine_zuteilung: null`,
und die Bedingung im Browser beginnt mit `!mein` -- also bot sie an,
sich auf die eigene Aufgabe zu bewerben.
ES WAR NICHT NUR EIN FALSCHER KNOPF. „Ich fange an" und „Fertig"
haengen an derselben Zeile. Der Mensch bekam eine Karte, auf der er
das Falsche tun konnte und das Richtige nicht.
WARUM DIE VORHANDENE PRUEFUNG ES NICHT FAND: Sie geht den Weg ueber
das AUFGABENbrett -- dort war alles in Ordnung, nachgemessen steht
nach dem Annehmen richtig „Ich fange an | Fertig". VanVans Weg ist
der ueber das VORLAGENbrett, und der endet in einer neu angelegten
Aufgabe. Dieser Weg war nie geprueft.
DREI TEILE
1. `katalogAufgabeAnlegen` legt die Zuteilungszeile mit an. Mit
Zustand: `angenommen`, wenn die Person darum gebeten hat und die
Bitte angenommen wurde; `offen`, wenn die Leitung zutraegt -- dann
steht bei ihr Annehmen/Ablehnen, und das ist richtig, sie hat noch
nicht ja gesagt.
2. Eine Umstellung traegt nach, was schon dasteht. Wiederholbar
(`NOT EXISTS` + `INSERT OR IGNORE`), deshalb bei den Umstellungen
und nicht in einem Skript, das jemand vergisst. Auf einer KOPIE
der echten Datenbank durchgespielt: 8 Zeilen angelegt, danach 0
ohne Zuteilung, `integrity_check ok`. Marinas Karten stehen
danach auf `angenommen`.
3. Ein Riegel im Browser: Auf eine Aufgabe, die mir schon gehoert,
bewirbt man sich nicht. Er faengt jeden weiteren Weg ab, der eine
Aufgabe ohne Zuteilungszeile anlegt -- nicht alle acht kamen aus
der Vorlage.
GEPRUEFT
pruef-bewerbung-aufgaben 126/0 (15 neue: der ganze Vorlagenweg von
der Bewerbung bis zum Knopf auf ihrem Brett). NACHGESTELLT: Ohne die
Zuteilungszeile wird sie rot („und sie hat eine Zuteilungszeile
(undefined)"). pruef-zuteilung, pruef-aufgabenbrett, pruef-vorlagen
24/0, pruef-aufgaben-vorlagen, pruef-struktur, pruef-zwischenspeicher
34/0.
Ein Messfehler unterwegs, im Text festgehalten: Mein neuer Block
bewarb sich auf dieselbe Vorlage wie ein spaeterer Abschnitt und
nahm ihm damit seine -- zehn Pruefungen wurden rot, ohne dass am
Programm etwas falsch war.
Datenbank vor der Umstellung gesichert und geprueft
(integrity_check ok, 12 Aufgaben, 3 Zuteilungen).
Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
Claude Opus 5
@@ -580,6 +580,124 @@ melde("=== 6. Am echten Bildschirm ===");
|
||||
`und keinen Block zum Entscheiden (${b?.bewerbungen})`);
|
||||
} finally { await m2.ctx.close(); }
|
||||
|
||||
/* =================================================================
|
||||
UND JETZT WIRD ANGENOMMEN -- VanVans Meldung (30.09.2026)
|
||||
|
||||
„Wenn ein Modi sich auf eine Aufgabe beworben hat und die von uns
|
||||
angenommen wurde, dann steht beim Modi unter der Aufgabe immer
|
||||
noch ich bewerbe mich."
|
||||
|
||||
DIESER SCHRITT HAT HIER GEFEHLT. Die Pruefung ging den ganzen
|
||||
Weg bis zur Bewerbung und hoerte dann auf. Serverseitig ist das
|
||||
Annehmen weiter oben geprueft („Fridas Stand ist angenommen");
|
||||
was der Mensch danach auf seinem Brett LIEST, hat nie jemand
|
||||
nachgesehen.
|
||||
================================================================= */
|
||||
const annahme = await roh(`/workspace/api/aufgaben/${frischId}/bewerbung/annehmen`, {
|
||||
methode: "POST", keks: hand.keks,
|
||||
body: { person_id: idModi, kommentar: "Gern, mach das." } });
|
||||
ok(annahme.code === 200 && annahme.json?.zustand === "angenommen",
|
||||
`die rechte Hand nimmt Fridas Bewerbung an (${annahme.code}/${annahme.json?.zustand})`);
|
||||
|
||||
const m3 = await seiteAls("modi", "CODE-MODI-0001", 390);
|
||||
try {
|
||||
const b = await knoepfeAn(m3.s, "Fuer den Bildschirm");
|
||||
ok(!!b, "Frida sieht die Aufgabe weiterhin");
|
||||
/* DER BEFUND. Ein Knopf, der nach der Zusage weiter „Ich
|
||||
bewerbe mich" sagt, behauptet, es sei nichts entschieden --
|
||||
und wer ihn drueckt, bekommt eine Fehlermeldung auf etwas,
|
||||
das laengst geklappt hat. */
|
||||
ok(!!b && !b.knoepfe.some((t) => /bewerbe mich/i.test(t)),
|
||||
`und NICHT mehr „Ich bewerbe mich" (${b?.knoepfe.join(" | ")})`);
|
||||
ok(!!b && !b.knoepfe.some((t) => /zurückziehen/i.test(t)),
|
||||
"auch nichts mehr zum Zurueckziehen -- es gibt keine offene Bewerbung mehr");
|
||||
/* UND DAS EIGENTLICHE: Steht da, dass sie die Aufgabe HAT?
|
||||
„Kein falscher Knopf" allein waere eine leere Karte. */
|
||||
ok(!!b && /(du machst|gehört dir|angenommen|übernommen|deine aufgabe)/i.test(b.text),
|
||||
`sondern dass sie die Aufgabe hat ("${(b?.text || "").slice(0, 90)}…")`);
|
||||
} finally { await m3.ctx.close(); }
|
||||
|
||||
/* =================================================================
|
||||
DERSELBE WEG, ABER UEBER DAS VORLAGENBRETT -- VanVans Fall
|
||||
|
||||
Der Abschnitt darueber geht ueber das Aufgabenbrett
|
||||
(`/api/aufgaben/:id/bewerben`). Dort war alles in Ordnung.
|
||||
|
||||
VanVans Weg ist der andere: Sie bewirbt sich auf eine VORLAGE.
|
||||
Wird die angenommen, entsteht eine NEUE Aufgabe -- und die
|
||||
bekam bis zum 30.09.2026 keine Zeile in `aufgaben_zuteilung`.
|
||||
|
||||
WAS DAS BEDEUTET HAT. Das Brett liest die Zuteilung, nicht
|
||||
`verantwortlich_id`. Ohne Zeile sagt es „niemand zugeteilt" und
|
||||
bietet an, sich auf die eigene Aufgabe zu bewerben. Und weil
|
||||
„Ich fange an" und „Fertig" an derselben Zeile haengen, fehlten
|
||||
sie: eine Karte, auf der man das Falsche tun kann und das
|
||||
Richtige nicht.
|
||||
|
||||
Gemessen an den echten Daten hatten 8 von 11 Aufgaben mit einem
|
||||
Verantwortlichen keine Zuteilungszeile.
|
||||
================================================================= */
|
||||
const vorl = await roh("/workspace/api/vorlagen", { keks: modi.keks });
|
||||
/* DIE VORLAGEN STEHEN UNTER `katalog.aufgaben`. Beim ersten
|
||||
Anlauf habe ich auf `gruppen` geraten und einen leeren Treffer
|
||||
bekommen -- die Pruefung meldete daraufhin „konnte nicht
|
||||
nachsehen", und das war die richtige Antwort auf eine falsche
|
||||
Frage. */
|
||||
/* DIE LETZTE UND NICHT DIE ERSTE. Ein spaeterer Abschnitt (Zeile
|
||||
~999) bewirbt sich ebenfalls, und zwar auf die ERSTE Vorlage.
|
||||
Nahm ich sie auch, bekam er 409 („liegt schon bei dir") -- und
|
||||
zehn Pruefungen wurden rot, ohne dass am Programm etwas falsch
|
||||
war. Ein Messfehler, der wie ein Befund aussieht, und einer,
|
||||
den ich selbst eingebaut hatte. */
|
||||
const katalogAufgaben = vorl.json?.katalog?.aufgaben || [];
|
||||
const v = katalogAufgaben.at(-1);
|
||||
if (!v) {
|
||||
ok(false, "KONNTE NICHT NACHSEHEN: keine Vorlage gefunden "
|
||||
+ `(${Object.keys(vorl.json || {}).join(", ")})`);
|
||||
} else {
|
||||
const vb = await roh("/workspace/api/vorlagen/bewerben", {
|
||||
methode: "POST", keks: modi.keks,
|
||||
body: { vorlage: v.schluessel, text: "Das mache ich gern." } });
|
||||
ok(vb.code === 201 || vb.code === 200,
|
||||
`Frida bewirbt sich auf die Vorlage „${v.titel}" (${vb.code})`);
|
||||
|
||||
const offen2 = await roh("/workspace/api/vorlagen/bewerbungen", { keks: hand.keks });
|
||||
const bw = (offen2.json?.bewerbungen || []).find((x) => x.vorlage === v.schluessel);
|
||||
ok(!!bw, `die rechte Hand sieht die Bewerbung (${offen2.json?.bewerbungen?.length})`);
|
||||
|
||||
const vann = await roh("/workspace/api/vorlagen/bewerbung/annehmen", {
|
||||
methode: "POST", keks: hand.keks,
|
||||
body: { id: bw?.id, kommentar: "Gern, mach das." } });
|
||||
ok(vann.code === 200, `und nimmt sie an (${vann.code})`);
|
||||
|
||||
/* ---- DER KERN: was steht jetzt auf Fridas Brett? ---------- */
|
||||
const nach = await roh("/workspace/api/aufgaben", { keks: modi.keks });
|
||||
const liste = nach.json?.aufgaben || nach.json?.liste || [];
|
||||
const neue = liste.find((x) => x.vorlage === v.schluessel);
|
||||
ok(!!neue, `die Aufgabe steht auf ihrem Brett (${liste.length} insgesamt)`);
|
||||
ok(!!neue && !!neue.meine_zuteilung,
|
||||
`und sie hat eine Zuteilungszeile (${JSON.stringify(neue?.meine_zuteilung?.zustand)})`);
|
||||
ok(neue?.meine_zuteilung?.zustand === "angenommen",
|
||||
`im Zustand „angenommen" -- sie hat darum gebeten und sie bekommen `
|
||||
+ `(${neue?.meine_zuteilung?.zustand})`);
|
||||
ok(neue?.ich_beworben === false,
|
||||
`und es liegt keine offene Bewerbung mehr vor (${neue?.ich_beworben})`);
|
||||
|
||||
const m4 = await seiteAls("modi", "CODE-MODI-0001", 390);
|
||||
try {
|
||||
const b = await knoepfeAn(m4.s, v.titel);
|
||||
ok(!!b, "Frida sieht die neue Aufgabe mit ihrem Zuteilungsblock");
|
||||
/* VANVANS BEFUND. Ein Knopf, der nach der Zusage weiter „Ich
|
||||
bewerbe mich" sagt, behauptet, es sei nichts entschieden. */
|
||||
ok(!!b && !b.knoepfe.some((t) => /bewerbe mich/i.test(t)),
|
||||
`und NICHT „Ich bewerbe mich" (${b?.knoepfe.join(" | ")})`);
|
||||
/* UND DIE ANDERE HAELFTE: Sie kann auch wirklich etwas tun.
|
||||
„Kein falscher Knopf" allein waere eine Karte ohne Griff. */
|
||||
ok(!!b && b.knoepfe.some((t) => /fange an/i.test(t)),
|
||||
`sondern „Ich fange an" (${b?.knoepfe.join(" | ")})`);
|
||||
} finally { await m4.ctx.close(); }
|
||||
}
|
||||
|
||||
/* =================================================================
|
||||
7. AUFGABEN DORT ANLEGEN, WO DIE PERSON STEHT (A5 und B6)
|
||||
=================================================================
|
||||
|
||||
@@ -1509,7 +1509,34 @@ function gleicheHerkunft(req, res, next) {
|
||||
DIE KENNUNG DER VORLAGE WANDERT MIT. Nur so kann das Brett spaeter
|
||||
zeigen, was schon uebernommen ist -- der Titel taugt dafuer nicht,
|
||||
er darf geaendert werden. */
|
||||
function katalogAufgabeAnlegen(v, zielId, vonId) {
|
||||
/** Legt eine Aufgabe aus einer Vorlage an -- MIT Zuteilungszeile.
|
||||
*
|
||||
* ==== DIE ZUTEILUNG HAT GEFEHLT (berichtigt 30.09.2026) ==========
|
||||
*
|
||||
* VanVan im Support: „Wenn ein Modi sich auf eine Aufgabe beworben
|
||||
* hat und die von uns angenommen wurde, dann steht beim Modi unter
|
||||
* der Aufgabe immer noch ich bewerbe mich."
|
||||
*
|
||||
* Hier stand nur das `INSERT INTO aufgaben` mit `verantwortlich_id`.
|
||||
* Fuer das Aufgabenbrett war damit NIEMAND zugeteilt: Es liest
|
||||
* `aufgaben_zuteilung`, findet nichts, und liefert
|
||||
* `meine_zuteilung: null`. Die Bedingung im Browser beginnt mit
|
||||
* `!mein` -- also bot sie dem Menschen an, sich auf eine Aufgabe zu
|
||||
* bewerben, die ihm laengst gehoert.
|
||||
*
|
||||
* ES WAR NICHT NUR EIN FALSCHER KNOPF. „Ich fange an" und „Fertig"
|
||||
* haengen ebenfalls an `mein.zustand`. Ohne Zuteilungszeile konnte
|
||||
* er auf der Karte das Falsche tun und das Richtige nicht.
|
||||
*
|
||||
* GEMESSEN an den echten Daten: 8 von 11 Aufgaben mit einem
|
||||
* Verantwortlichen hatten keine Zuteilungszeile.
|
||||
*
|
||||
* @param {string} zustand `angenommen`, wenn die Person darum
|
||||
* gebeten hat und die Bitte angenommen wurde. `offen`, wenn die
|
||||
* Leitung sie zutraegt -- dann steht bei ihr Annehmen/Ablehnen,
|
||||
* und das ist richtig so: Sie hat noch nicht ja gesagt.
|
||||
*/
|
||||
function katalogAufgabeAnlegen(v, zielId, vonId, zustand = "offen") {
|
||||
const d = new Date(Date.now() + (v.tage || 7) * 86400_000);
|
||||
const frist = `${d.getFullYear()}-${String(d.getMonth() + 1).padStart(2, "0")}`
|
||||
+ `-${String(d.getDate()).padStart(2, "0")}`;
|
||||
@@ -1519,7 +1546,16 @@ function katalogAufgabeAnlegen(v, zielId, vonId) {
|
||||
VALUES (?,?,'offen',?,?,?,?,?,?,?)`)
|
||||
.run(v.titel, v.text, v.tage <= 1 ? "hoch" : "mittel",
|
||||
zielId, frist, v.kategorie, v.schluessel, jetzt(), vonId);
|
||||
return db().prepare("SELECT last_insert_rowid() AS id").get().id;
|
||||
const id = db().prepare("SELECT last_insert_rowid() AS id").get().id;
|
||||
const nun = jetzt();
|
||||
db().prepare(`INSERT INTO aufgaben_zuteilung
|
||||
(aufgabe_id, person_id, zustand, zugeteilt_am, zugeteilt_von, geantwortet_am)
|
||||
VALUES (?,?,?,?,?,?)`)
|
||||
.run(id, zielId, zustand, nun, vonId ?? null,
|
||||
/* „Wann geantwortet" nur, wenn schon geantwortet wurde. Bei
|
||||
`offen` steht die Frage noch aus. */
|
||||
zustand === "offen" ? null : nun);
|
||||
return id;
|
||||
}
|
||||
|
||||
/* Wie lang darf ein Wort zu einer Bewerbung sein? Dieselbe Groesse wie
|
||||
@@ -1776,7 +1812,10 @@ for (const [weg, ja] of [["annehmen", true], ["ablehnen", false]]) {
|
||||
let aufgabeId = null;
|
||||
|
||||
if (ja) {
|
||||
aufgabeId = katalogAufgabeAnlegen(v, wer.id, req.person.id);
|
||||
/* `angenommen`: Er hat darum gebeten, und die Bitte wurde
|
||||
angenommen. Ihn danach noch einmal „Annehmen" druecken zu
|
||||
lassen waere dieselbe Frage zweimal. */
|
||||
aufgabeId = katalogAufgabeAnlegen(v, wer.id, req.person.id, "angenommen");
|
||||
}
|
||||
d.prepare(`UPDATE vorlagen_bewerbungen
|
||||
SET zustand = ?, entscheid_text = ?, entschieden_von = ?,
|
||||
|
||||
@@ -4361,6 +4361,68 @@ function umstellungen(d) {
|
||||
}
|
||||
}
|
||||
|
||||
/* =====================================================================
|
||||
WER VERANTWORTLICH IST, BRAUCHT EINE ZUTEILUNGSZEILE (30.09.2026)
|
||||
=====================================================================
|
||||
|
||||
VanVan im Support: „Wenn ein Modi sich auf eine Aufgabe beworben
|
||||
hat und die von uns angenommen wurde, dann steht beim Modi unter
|
||||
der Aufgabe immer noch ich bewerbe mich."
|
||||
|
||||
GEMESSEN an den echten Daten: 11 Aufgaben hatten einen
|
||||
Verantwortlichen, 8 davon KEINE Zeile in `aufgaben_zuteilung`.
|
||||
Darunter beide Vorlagen-Aufgaben, die in ihrem Bildschirmfoto zu
|
||||
sehen sind.
|
||||
|
||||
Das Aufgabenbrett liest die Zuteilung, nicht `verantwortlich_id`.
|
||||
Ohne Zeile sagt es „niemand zugeteilt" -- und bietet dem Menschen
|
||||
an, sich auf seine eigene Aufgabe zu bewerben. „Ich fange an" und
|
||||
„Fertig" fehlen ihm dabei, denn die haengen an derselben Zeile.
|
||||
|
||||
Die Ursache ist behoben (`katalogAufgabeAnlegen` legt sie jetzt
|
||||
mit an). Dieser Block holt nach, was schon dasteht.
|
||||
|
||||
WELCHER ZUSTAND: Die Aufgabe GEHOERT der Person schon -- sie
|
||||
jetzt „Annehmen/Ablehnen" druecken zu lassen waere eine Frage zu
|
||||
etwas, das laengst entschieden ist. Also `angenommen`, und bei
|
||||
abgeschlossenen Aufgaben der passende Endzustand.
|
||||
|
||||
WIEDERHOLBAR: `NOT EXISTS` plus `INSERT OR IGNORE`. Beim zweiten
|
||||
Start ist alles da und nichts passiert -- derselbe Gedanke wie
|
||||
bei der Teilnehmer-Uebernahme darueber. */
|
||||
try {
|
||||
const spalten = d.prepare("PRAGMA table_info(aufgaben)").all().map((x) => x.name);
|
||||
if (spalten.includes("verantwortlich_id")) {
|
||||
const vorher = d.prepare("SELECT COUNT(*) AS n FROM aufgaben_zuteilung").get().n;
|
||||
d.exec(`INSERT OR IGNORE INTO aufgaben_zuteilung
|
||||
(aufgabe_id, person_id, zustand, zugeteilt_am, zugeteilt_von, geantwortet_am)
|
||||
SELECT a.id, a.verantwortlich_id,
|
||||
CASE a.status
|
||||
WHEN 'erledigt' THEN 'erledigt'
|
||||
WHEN 'review' THEN 'erledigt'
|
||||
WHEN 'abgebrochen' THEN 'abgelehnt'
|
||||
WHEN 'arbeit' THEN 'arbeit'
|
||||
ELSE 'angenommen'
|
||||
END,
|
||||
COALESCE(a.erstellt, datetime('now')),
|
||||
a.erstellt_von,
|
||||
COALESCE(a.erstellt, datetime('now'))
|
||||
FROM aufgaben a
|
||||
WHERE a.verantwortlich_id IS NOT NULL
|
||||
AND NOT EXISTS (
|
||||
SELECT 1 FROM aufgaben_zuteilung z
|
||||
WHERE z.aufgabe_id = a.id
|
||||
AND z.person_id = a.verantwortlich_id)`);
|
||||
const nachher = d.prepare("SELECT COUNT(*) AS n FROM aufgaben_zuteilung").get().n;
|
||||
if (nachher > vorher) {
|
||||
console.log(`[workspace] ${nachher - vorher} Aufgaben mit Verantwortlichem `
|
||||
+ "haben ihre fehlende Zuteilungszeile bekommen.");
|
||||
}
|
||||
}
|
||||
} catch (fehler) {
|
||||
console.error("[workspace] Zuteilung nachtragen:", fehler?.message);
|
||||
}
|
||||
|
||||
loeschspurenAufraeumen(d);
|
||||
|
||||
/* =====================================================================
|
||||
|
||||
Reference in New Issue
Block a user