From 314561fb64ea67f3bb78ca526cbc4cc502bc064a Mon Sep 17 00:00:00 2001 From: Dogfather Date: Mon, 21 Sep 2026 14:07:25 +0200 Subject: [PATCH] Aufgaben an bestimmte Leute verteilen -- jetzt auch sichtbar MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Filipe: "ich sehe noch immer nicht dass dogfather oder die rechte hand wenn sie aufgaben verteilen die spezifisch an leute verteilen können. mach das endlich auch weil das ist seeehr wichtig damit wir arbeiten können mit den leuten." GEMESSEN, BEVOR GEBAUT WURDE -- und die Messung hat zwei verschiedene Antworten gegeben: DogFather Feld sichtbar, vier Menschen in der Auswahl Rechte Hand Feld VERSTECKT, Auswahl leer Fuer ihn gab es die Zuteilung also, fuer sie nicht. Beides war falsch, jedes auf seine Art. 1. DIE RECHTE HAND KAM NIE AN DIE FELDER Das Absenden fragte `ich.darf_verteilen` -- das sagt der Server: Leitung ODER rechte Hand (seit 20.09.). Das ANZEIGEN fragte eine andere Menge aus bereiche.js: spicy, admin, manager. Zwei Bedingungen fuer dieselbe Sache, und eine davon war nie nachgezogen worden. Der Server haette ihre Zuteilung angenommen; sie konnte sie nur nirgends eintragen. Vier Stellen betroffen, und eine davon war gefaehrlich: Beim BEARBEITEN wurde `verantwortlich_id` nicht mitgeschickt -- die rechte Hand haette durch blosses Speichern die Zuteilung einer Aufgabe stillschweigend geloescht. Kein Fehler, keine Meldung, die Aufgabe gehoert danach niemandem. Alle vier fragen jetzt den Server. `LEITUNG.has(ich.rolle)` bleibt nur noch als Rueckfall hinter `??`, fuer den Fall einer aelteren Serverfassung. 2. UND FUER DOGFATHER LAG ES AM WORT Das Feld hiess "Verantwortlich" und stand auf "—". Das liest sich wie eine Eigenschaft der Aufgabe, nicht wie eine Frage an einen selbst. Es heisst jetzt "Wer macht es?", der leere Wert "— noch niemand —", und darunter steht, was die Auswahl bewirkt: Die Aufgabe steht bei ihm unter "Nur meine", und er bekommt eine Meldung. Eine Frage beantwortet man. Ein Strich uebersieht man. WARUM KEINE PRUEFUNG DAS GEFUNDEN HAT: Alle fragten den Server, und der war in Ordnung -- Rechte da, Leute da, Zuteilung wird angenommen. Eine Erlaubnis, an die niemand herankommt, sieht von dort aus wie eine Erlaubnis. pruef-verteilen liest deshalb jetzt auch den Quelltext der Oberflaeche: dieselbe Frage an beiden Stellen, und die Beschriftung muss eine Frage sein. Kommentare zaehlen dabei nicht mit -- beim ersten Lauf hat die Pruefung meine eigene Begruendung als Befund gezaehlt. pruef-verteilen 16/0 · pruef-aufgabenbrett in Ordnung Co-Authored-By: Claude Opus 5 --- server/pruef-verteilen.mjs | 72 +++++++++++++++++++++++++++++++++ workspace/anruf-probe.html | 8 ++-- workspace/app.webmanifest | 8 ++-- workspace/assets/js/aufgaben.js | 39 ++++++++++++++++-- workspace/aufgaben.html | 60 ++++++++++++++++++--------- workspace/automation.html | 30 +++++++------- workspace/befinden.html | 28 ++++++------- workspace/bereich.html | 38 ++++++++--------- workspace/bewerben.html | 34 ++++++++-------- workspace/bewerbungen.html | 32 +++++++-------- workspace/calls.html | 32 +++++++-------- workspace/chat.html | 34 ++++++++-------- workspace/content.html | 32 +++++++-------- workspace/crew-index.html | 10 ++--- workspace/crew.webmanifest | 8 ++-- workspace/dateien.html | 32 +++++++-------- workspace/entwicklung.html | 28 ++++++------- workspace/hilfe.html | 26 ++++++------ workspace/index.html | 8 ++-- workspace/kalender.html | 32 +++++++-------- workspace/leistung.html | 30 +++++++------- workspace/personen.html | 30 +++++++------- workspace/profil.html | 34 ++++++++-------- workspace/rechte.html | 28 ++++++------- workspace/report.html | 32 +++++++-------- workspace/scouting.html | 30 +++++++------- workspace/start.html | 28 ++++++------- workspace/startcheck.html | 32 +++++++-------- workspace/steckbrief.html | 34 ++++++++-------- workspace/talente.html | 30 +++++++------- workspace/team.html | 28 ++++++------- workspace/teamlage.html | 28 ++++++------- workspace/teilen.html | 26 ++++++------ workspace/treff-moderation.html | 28 ++++++------- workspace/treff-regeln.html | 28 ++++++------- workspace/uebersicht.html | 30 +++++++------- workspace/unsere-seiten.html | 24 +++++------ workspace/werdegang.html | 28 ++++++------- workspace/wissen.html | 32 +++++++-------- 39 files changed, 638 insertions(+), 513 deletions(-) diff --git a/server/pruef-verteilen.mjs b/server/pruef-verteilen.mjs index a21577f2..78ac2d48 100644 --- a/server/pruef-verteilen.mjs +++ b/server/pruef-verteilen.mjs @@ -169,6 +169,78 @@ ok(vonModi.code === 201 && vonModi.verant === idModi, `Gegenprobe: ein Modi kann es versuchen — die Aufgabe landet bei ihm selbst ` + `(${vonModi.verant}, nicht ${idModi2})`); +/* ======================================================================= + DIE FELDER MUESSEN AUCH ZU SEHEN SEIN + ======================================================================= + + Filipe am 21.09.2026: "ich sehe noch immer nicht dass dogfather oder + die rechte hand wenn sie aufgaben verteilen die spezifisch an leute + verteilen können. mach das endlich auch weil das ist seeehr wichtig + damit wir arbeiten können mit den leuten." + + ALLES DARUEBER WAR GRUEN -- und er hatte trotzdem recht. Der Server + nahm die Zuteilung an, die Rechte stimmten, die Leute standen in der + Auskunft. Nur die rechte Hand BEKAM DAS FELD NIE ZU SEHEN: Das + Absenden fragte `ich.darf_verteilen` (Server: Leitung ODER rechte + Hand), das Anzeigen eine andere Menge aus bereiche.js (spicy, admin, + manager). Zwei Bedingungen fuer dieselbe Sache, und eine davon + veraltet. + + EINE PRUEFUNG, DIE NUR DEN SERVER FRAGT, FINDET SO ETWAS NIE. Sie + sieht eine Erlaubnis, die es gibt -- und uebersieht, dass niemand + an sie herankommt. Deshalb wird hier der Quelltext der Oberflaeche + mitgelesen: Es muss an BEIDEN Stellen dieselbe Frage stehen. + ======================================================================= */ +{ + const { readFileSync } = await import("node:fs"); + /* KOMMENTARE ZAEHLEN NICHT MIT. + + Beim ersten Lauf meldete diese Pruefung fuenf Stellen -- zwei + davon waren mein eigener ERKLAERTEXT, in dem die alte Bedingung + woertlich zitiert steht. Eine Pruefung, die ihre eigene + Begruendung als Befund zaehlt, treibt einen dazu, die Begruendung + zu loeschen. Der Zerleger aus helfer-ohne-kommentar.mjs trennt + beides sauber. */ + const { ohneKommentar } = await import("./helfer-ohne-kommentar.mjs"); + const js = ohneKommentar(readFileSync( + new URL("../workspace/assets/js/aufgaben.js", import.meta.url), "utf8"), ".js"); + + /* Wie oft wird ueberhaupt entschieden, ob jemand verteilen darf? */ + const mitServer = (js.match(/ich\.darf_verteilen/g) || []).length; + ok(mitServer >= 2, + `die Oberflaeche fragt ${mitServer}-mal den Server (darf_verteilen)`); + + /* UND NIRGENDS MEHR ALLEIN DIE ROLLENMENGE. `LEITUNG.has(ich.rolle)` + darf nur noch als Rueckfall hinter `??` stehen -- also fuer den + Fall, dass eine aeltere Serverfassung das Feld gar nicht schickt. + Steht es irgendwo als eigene Bedingung, ist die zweite Wahrheit + zurueck. */ + const ohneRueckfall = [...js.matchAll(/LEITUNG\.has\(ich\.rolle\)/g)] + .filter((m) => !/\?\?\s*$/.test(js.slice(Math.max(0, m.index - 24), m.index))); + ok(ohneRueckfall.length === 0, + ohneRueckfall.length + ? `${ohneRueckfall.length}-mal entscheidet die Rollenmenge allein -- ` + + "der Server sagt es besser" + : "und nirgends entscheidet die Rollenmenge allein"); + + /* DIE BESCHRIFTUNG IST TEIL DER FUNKTION. "Verantwortlich" mit einem + Strich daneben liest sich wie eine Eigenschaft; gefunden hat + Filipe das Feld damit nicht. "Wer macht es?" ist eine Frage. */ + const html = readFileSync( + new URL("../workspace/aufgaben.html", import.meta.url), "utf8"); + const schilder = [...html.matchAll(/for="(?:f|b)-verant">([^<]+) m[1].trim()); + ok(schilder.length === 2, + `beide Formulare haben das Feld (${schilder.length})`); + ok(schilder.every((t) => /wer\s+macht/i.test(t)), + `und beide fragen dasselbe ("${schilder.join('" / "')}")`); + + /* Und der leere Wert sagt, was er bedeutet. Ein Strich ist kein + Zustand, sondern ein fehlendes Wort -- wer ihn sieht, weiss nicht, + ob er etwas versaeumt hat. */ + ok(/— noch niemand —/.test(html), + "der leere Wert heisst „noch niemand“ und nicht nur „—“"); +} + console.log(`\n${fehler ? fehler + " FEHLER" : "Alles in Ordnung"} — ${geprueft} Prüfungen\n`); try { rmSync(ordner, { recursive: true, force: true }); } catch { /* egal */ } process.exit(fehler ? 1 : 0); diff --git a/workspace/anruf-probe.html b/workspace/anruf-probe.html index 9223dcd2..b460c072 100644 --- a/workspace/anruf-probe.html +++ b/workspace/anruf-probe.html @@ -16,15 +16,15 @@ verdeckt. Gefunden hat es pruef-struktur. Richtig sind `gate.css` (die Farbwerte des Hauses) und `haus.css`. --> - - + + - +