Ein Rollenname im ausgelieferten Code, die Seite zu breit, und drei Pruefungen, die den Umbau verschlafen hatten
Filipe: "falls es noch was gibt was fehlt oder nicht richtig funktioniert,
auf dem handy oder auf dem pc, oder als installiert, ich will dass du das
alles abcheckst und machst dass jede seite reibungslos klappt."
Gemessen wurde der ganze Bestand: 26 Pruefdateien einzeln, dazu drei neue
Messungen fuer Dinge, die keine Pruefung ansieht. Von 37 roten Meldungen
aus dem Nachtlauf sind nach dieser Runde die haelfte erledigt -- und die
Haelfte davon war gar nicht kaputt.
=== ECHTE FEHLER ===
1. EIN ROLLENNAME STAND IM AUSGELIEFERTEN QUELLTEXT.
`aufgaben.js` sagte "Modis moechten das uebernehmen -- du
entscheidest". Diese Datei bekommt JEDER, der die Seite oeffnet:
jeder Manager, jeder Scout, jeder Creator im anderen Haus. Der
verborgene Zugang haelt genau so lange, wie der Name dort nicht
steht. Er stammte aus der Bewerbungsspalte von gestern -- einen Tag
alt, gefunden von pruef-modi-wortleck. Jetzt: "Jemand moechte das
uebernehmen". Wer es ist, steht ohnehin mit Namen auf den Karten
darunter, und die sieht nur, wer sie sehen darf.
2. DIE SUPPORT-SEITE WAR AUF JEDER BREITE UNTER 1440 px ZU BREIT --
13 px auf dem Handy, 41 px auf dem Tablet quer, 30 px auf dem
Laptop. Schuld war das weiche Licht, das ich in der Nacht eingebaut
habe: `inset: -8% -4% 0`. Die vier Prozent rechts sahen nach einer
Kleinigkeit aus; ein absolut gesetztes Element zaehlt aber zur
Scrollbreite, auch mit `pointer-events: none` und `z-index: -1`.
`pruef-breiten` hat es gemeldet und konnte nicht sagen, WAS es ist
("13px Ueberstand []") -- ein Pseudoelement hat keine Box im DOM.
Gefunden hat es eine neue Messung, die JEDES echte Element abfragt:
keines ragte heraus, und genau das war der Hinweis.
Jetzt: 0 px auf allen neun Breiten, 38 Seiten, 84 252 Elemente.
3. EIN AUFGABENLINK IM REPORT WAR 27 x 18 px GROSS. Mit der Maus gut,
mit dem Daumen nicht -- eine Fingerkuppe ist rund 45 px breit.
Jetzt fuellt der Link die Zeile und ist 44 px hoch, aber nur auf
schmalen Bildschirmen und an groben Zeigegeraeten. Die zweite
Bedingung habe ich beim ersten Anlauf vergessen, und der Link blieb
27 x 21 -- eine Regel, die nur unter idealen Bedingungen greift,
hilft niemandem auf dem halben Weg dorthin.
4. EINE SEITE LUD DAS FALSCHE MANIFEST. `anruf-probe.html` verwies
fest auf `crew.webmanifest`; die Seite ist aber fuer ALLE Rollen
offen und damit auf beiden Adressen erreichbar. Wer sie im
Agenturhaus oeffnet und die App installiert, bekam "DogFather
Universe" mit dem Crew-Symbol auf den Startbildschirm. Alle anderen
38 Seiten machen es richtig: `app.webmanifest`, und die Adresse
biegt es um.
5. DER KNOPF "+ GIF HINZUFUEGEN" WAR 40 px HOCH, und der Kommentar
daneben nannte das "die Hausgroesse". Die Hausgroesse sind 44 --
fuenfmal in derselben Datei so begruendet. Die erste Reparatur
griff nicht: Die Fingerregel stand 400 Zeilen VOR der Grundregel,
und bei gleicher Staerke gewinnt die spaetere. Jetzt steht sie
direkt dahinter, und der Knopf misst 133 x 44 am Finger, 133 x 40
an der Maus.
KEINE PRUEFUNG KONNTE DAS FINDEN: Die GIF-Tafel ist zu, solange
niemand sie aufmacht, und alle Rundgaenge messen, was auf dem
Bildschirm steht. Dafuer gibt es jetzt server/mess-gifs-handy.mjs.
6. ZWEI KLEINIGKEITEN AUS DER NACHT: eine Fehlerkennung ohne Satz
(`unbekannter_punkt`) und eine Stelle in support.js, die den
Serverfehler roh anzeigte statt durch `fehlerText` -- die zwei
anderen Stellen derselben Datei machen es richtig. So entsteht eine
Ausnahme: nicht aus Absicht, sondern weil man die Hausregel beim
Neuschreiben nicht danebenliegen hatte.
7. TOTES CSS (.e-leerwahl, vier Regeln). Der Leerkasten ist am
24.09. auf Filipes Wunsch wieder verschwunden, sein Stil blieb
einen Tag laenger stehen.
=== ROT, ABER NICHT KAPUTT ===
Drei Pruefungen haben den Umbau vom 24.09. nicht mitbekommen:
pruef-anruf meldete 31 Fehler und "ein Gespraech entsteht (403)".
Sie meldete DogFather auf der AGENTUR-Adresse an und liess ihn dann
die rechte Hand anrufen -- die es dort seit der Haustrennung nicht
gibt. Der Server hatte recht. Jetzt telefoniert Team Dogi auf crew.,
und der Creator bleibt auf workspace. -- seine Gegenprobe ist damit
sogar schaerfer als vorher (ein Fremder aus dem ANDEREN Haus).
96 -> 127 Pruefungen, 0 Fehler.
pruef-crew-wand-bild erwartete vier Rollen auf der Zugangswand. Es
sind fuenf, seit die linke Hand am 21.09. dazukam. Die Zeile stand
unter einem Kommentar, der wortwoertlich vor festen Namen in
Pruefungen warnt ("eine Zeitbombe mit Datum") -- und war selbst
einer. Jetzt leitet sie die Rollen aus `rollenImHaus("crew")` ab,
derselben Quelle, aus der der Server die Wand baut.
pruef-entwicklung-kacheln rechnete noch mit "x von 68". Seit
Filipes Wunsch ("es sollen nur die menge angezeigt werden die wir
zutragen") ist das Ganze das, was jemandem zugetragen ist. Sie baut
jetzt BEIDE Faelle -- eine Person mit vier zugetragenen Punkten und
eine ohne -- und misst Ring, Bogen, Zeile und Vorleseschild gegen
die Zuteilung. Dazu eine Gegenprobe mit einer Einschaetzung auf
einem NICHT zugetragenen Punkt: zaehlt die Uebersicht sie mit,
stuende dort 2 statt 1. 31 -> 34 Pruefungen, 0 Fehler.
=== WAS DIE BILDSCHAU ANGEHT ===
Meine eigene Messung meldete erst "Mitte 174 von 640" -- die Schau sah
kaputt aus. Sie war es nicht: `querySelector("img")` nahm das
Husky-Zeichen in der Kopfzeile statt des Bildes. Nachgemessen am
richtigen Element steht es auf 195 von 195 (Handy) und 640 von 640
(Rechner), und ein GIF oeffnet sich als GIF. Ein Messfehler, der wie
ein Befund aussieht, kostet mehr Zeit als gar keine Messung -- deshalb
steht die Begruendung jetzt im Quelltext der Messung.
GEMESSEN: pruef-breiten (9 Breiten, 38 Seiten, 84 252 Elemente),
pruef-support 63/0, pruef-gifs 17/0, pruef-chat 63/0, pruef-chat-optik
62/0, pruef-tippziele 11/0, pruef-css-klassen, pruef-struktur,
pruef-meldungen, pruef-modi-wortleck, pruef-crew-adresse,
pruef-installieren, pruef-aufgabenbrett, pruef-entwicklung -- alle 0
Fehler.
Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
Claude Opus 5
@@ -124,7 +124,39 @@ neueAufgabe("Clips vom Samstag schneiden", idDiene, GESTERN, "offen");
|
||||
neueAufgabe("Plan für nächste Woche", idDiene, IN_DREI, "arbeit");
|
||||
neueAufgabe("Ideen sammeln", idDiene, null, "offen");
|
||||
neueAufgabe("Längst abgehakt", idDiene, GESTERN, "erledigt");
|
||||
|
||||
/* =====================================================================
|
||||
ZUGETRAGENE PUNKTE (25.09.2026) -- der Nenner der Kachel
|
||||
=====================================================================
|
||||
|
||||
Filipe: „es sollen nur die menge angezeigt werden die wir zutragen
|
||||
und erledigte dan auch nur die die erledigt wurden von den
|
||||
zugetragenen."
|
||||
|
||||
BIS HEUTE stand auf jeder Kachel „x von 68" -- 68 war der ganze
|
||||
Katalog. Jetzt ist das Ganze das, was dieser Mensch bekommen hat.
|
||||
Wer nichts bekommen hat, hat keinen Nenner, und dort steht ein Satz
|
||||
statt einer Null.
|
||||
|
||||
BEIDE FAELLE WERDEN GEBAUT, und das ist der Punkt: Diene bekommt
|
||||
vier Punkte zugetragen und bei einem davon eine Einschaetzung
|
||||
(also 1 von 4 = 25 %), VanVan bekommt nichts. Eine Pruefung, die nur
|
||||
den einen Fall kennt, haette den Umbau nicht bemerkt -- genau das
|
||||
ist heute frueh passiert: Sie war rot mit „noch nichts zugetragen,
|
||||
noch nichts zugetragen" und konnte nicht sagen, ob das richtig ist.
|
||||
|
||||
DIE FUENFTE EINSCHAETZUNG IST DIE GEGENPROBE. Sie steht auf einem
|
||||
Punkt, den Diene NICHT zugetragen bekommen hat. Zaehlte die
|
||||
Uebersicht sie mit, stuende dort „2 von 4" -- und in der Karte
|
||||
daneben „1 von 4". Zwei Zahlen fuer dieselbe Sache. */
|
||||
const { ENTWICKLUNG_FREMD } = await import("./workspace-entwicklungs-punkte.js");
|
||||
const VIER = ENTWICKLUNG_FREMD.slice(0, 4).map((x) => x.schluessel);
|
||||
const FUENFTER = ENTWICKLUNG_FREMD[4].schluessel;
|
||||
d.close();
|
||||
/* Eingetragen wird weiter unten ueber die ECHTEN Routen -- die
|
||||
Tabellen des Entwicklungsmoduls entstehen erst beim ersten Zugriff
|
||||
darauf, vorher gibt es sie gar nicht. Und ueber die Route geprueft
|
||||
zu werden ist ohnehin mehr wert als ein INSERT von der Seite. */
|
||||
|
||||
const an = await fetch(BASIS + "/workspace/api/anmelden", {
|
||||
method: "POST", headers: { "Content-Type": "application/json" },
|
||||
@@ -132,6 +164,25 @@ const an = await fetch(BASIS + "/workspace/api/anmelden", {
|
||||
});
|
||||
const keks = (an.headers.getSetCookie?.() || []).map((z) => z.split(";")[0]).join("; ");
|
||||
|
||||
/* ---- Jetzt die Zuteilung, ueber die Wege, die es wirklich gibt ----
|
||||
Vier Punkte fuer Diene, eine Einschaetzung auf einem davon und eine
|
||||
auf einem FUENFTEN, den sie nicht bekommen hat (die Gegenprobe:
|
||||
zaehlt die Uebersicht sie mit, stuende dort 2 statt 1). */
|
||||
const mitKeks = (pfad, art, koerper) => fetch(BASIS + pfad, {
|
||||
method: art,
|
||||
headers: { "Content-Type": "application/json", Cookie: keks,
|
||||
Origin: BASIS, Referer: BASIS + "/workspace/entwicklung.html" },
|
||||
body: koerper ? JSON.stringify(koerper) : undefined,
|
||||
});
|
||||
const zut = await mitKeks(`/workspace/api/entwicklung/person/${idDiene}/punkte`,
|
||||
"PUT", { punkte: VIER });
|
||||
melde(` -- Zuteilung: HTTP ${zut.status}`);
|
||||
for (const s of [VIER[0], FUENFTER]) {
|
||||
const r = await mitKeks(`/workspace/api/entwicklung/person/${idDiene}/${s}`,
|
||||
"PUT", { stufe: "laeuft" });
|
||||
if (r.status >= 400) melde(` -- Einschaetzung ${s}: HTTP ${r.status}`);
|
||||
}
|
||||
|
||||
const { chromium } = await import(
|
||||
"file:///C:/Users/qciga/Documents/Obelix/Analyse/node_modules/playwright/index.mjs");
|
||||
const browser = await chromium.launch();
|
||||
@@ -235,17 +286,46 @@ try {
|
||||
};
|
||||
}));
|
||||
|
||||
const gesamt = lage?.gesamt_fremd || 0;
|
||||
const sollAnteil = (p) => (gesamt > 0
|
||||
? Math.round((Number(p?.gesetzt) || 0) / gesamt * 100) : 0);
|
||||
/* ==== DAS GANZE IST, WAS ZUGETRAGEN IST (25.09.2026) ==========
|
||||
|
||||
Hier stand `gesamt_fremd` -- die Zahl aller Punkte im Katalog
|
||||
(68). Die gibt es weiterhin, sie ist aber nur noch die Groesse
|
||||
der AUSWAHL, nicht der Nenner der Kachel. Der Nenner steht seit
|
||||
heute je Person in `zugeteilt`.
|
||||
|
||||
WER NICHTS ZUGETRAGEN BEKOMMEN HAT, hat keinen Nenner: Dort
|
||||
steht „–" im Ring und ein Satz in der Zeile, keine Null. „0 von
|
||||
0 erledigt" liest sich wie ein Versaeumnis; es ist aber keins,
|
||||
sondern eine offene Aufgabe der Leitung. */
|
||||
const katalog = lage?.gesamt_fremd || 0;
|
||||
const ganzes = (p) => Number(p?.zugeteilt) || 0;
|
||||
const sollAnteil = (p) => (ganzes(p) > 0
|
||||
? Math.round((Number(p?.gesetzt) || 0) / ganzes(p) * 100) : 0);
|
||||
const nachNr = new Map((lage?.personen || []).map((p) => [Number(p.id), p]));
|
||||
|
||||
/* Dass die Testdaten ueberhaupt angekommen sind, wird gemessen und
|
||||
nicht vorausgesetzt: Eine Pruefung, deren Voraussetzung fehlt,
|
||||
darf nicht still gruen werden. */
|
||||
const mitZuteilung = [...nachNr.values()].filter((x) => ganzes(x) > 0);
|
||||
const ohneZuteilung = [...nachNr.values()].filter((x) => ganzes(x) === 0);
|
||||
ok(mitZuteilung.length >= 1 && ohneZuteilung.length >= 1,
|
||||
`beide Faelle liegen vor: ${mitZuteilung.length} mit Zuteilung, `
|
||||
+ `${ohneZuteilung.length} ohne`);
|
||||
ok(mitZuteilung.every((x) => ganzes(x) === 4 && Number(x.gesetzt) === 1),
|
||||
`die zugetragenen Punkte zaehlen, der fuenfte nicht `
|
||||
+ `(${mitZuteilung.map((x) => `${x.name}: ${x.gesetzt} von ${ganzes(x)}`).join(", ")})`);
|
||||
ok(katalog > 4,
|
||||
`und der Katalog ist weiterhin groesser als das Zugetragene (${katalog})`);
|
||||
|
||||
ok(z.length > 0 && z.every((k) => nachNr.has(k.nr)),
|
||||
`jede Kachel gehoert zu einer Person aus der Auskunft (${z.map((k) => k.nr).join(", ")})`);
|
||||
|
||||
const ringOk = z.every((k) => k.ringZahl === String(sollAnteil(nachNr.get(k.nr))));
|
||||
const ringOk = z.every((k) => {
|
||||
const pp = nachNr.get(k.nr);
|
||||
return k.ringZahl === (ganzes(pp) > 0 ? String(sollAnteil(pp)) : "\u2013");
|
||||
});
|
||||
ok(ringOk, `die Ziffer im Ring ist der gerechnete Anteil `
|
||||
+ `(${z.map((k) => `${k.name}: ${k.ringZahl}%`).join(", ")})`);
|
||||
+ `(${z.map((k) => `${k.name}: ${k.ringZahl}`).join(", ")})`);
|
||||
|
||||
/* Der BOGEN ist das, was man sieht. Umfang 100 -- die erste Zahl
|
||||
im stroke-dasharray ist damit buchstaeblich der Prozentwert. */
|
||||
@@ -257,23 +337,28 @@ try {
|
||||
/* DIE ZEILE DARUNTER nennt beide Zahlen, aus denen der Anteil
|
||||
entsteht -- ohne sie waere "68 %" eine Behauptung ohne Beleg. */
|
||||
const standOk = z.every((k) => {
|
||||
const p = nachNr.get(k.nr);
|
||||
return k.standZahl === String(Number(p?.gesetzt) || 0)
|
||||
&& k.standRest === ` von ${gesamt} angesehen`;
|
||||
const pp = nachNr.get(k.nr);
|
||||
return ganzes(pp) > 0
|
||||
? (k.standZahl === String(Number(pp?.gesetzt) || 0)
|
||||
&& k.standRest === ` von ${ganzes(pp)} erledigt`)
|
||||
: (k.standZahl === "" && k.standRest === "noch nichts zugetragen");
|
||||
});
|
||||
ok(standOk, `"x von ${gesamt} angesehen" stimmt bei jeder Kachel `
|
||||
+ `(${z.map((k) => k.standZahl + k.standRest).join(", ")})`);
|
||||
ok(standOk, `"x von n erledigt" bzw. der Satz stimmt bei jeder Kachel `
|
||||
+ `(${z.map((k) => k.standZahl + k.standRest).join(" | ")})`);
|
||||
|
||||
/* UND DAS VORLESEPROGRAMM HOERT DASSELBE. Ein Ring ist ein Bild;
|
||||
ohne Schild ist er fuer ein Vorleseprogramm gar nicht da. */
|
||||
ok(z.every((k) => k.rolle === "img"), "der Ring gibt sich als Bild zu erkennen");
|
||||
const schildOk = z.every((k) => {
|
||||
const p = nachNr.get(k.nr);
|
||||
const soll = `${Number(p?.gesetzt) || 0} von ${gesamt} angesehen `
|
||||
+ `(${sollAnteil(p)} Prozent)`;
|
||||
const pp = nachNr.get(k.nr);
|
||||
const soll = ganzes(pp) > 0
|
||||
? `${Number(pp?.gesetzt) || 0} von ${ganzes(pp)} erledigt `
|
||||
+ `(${sollAnteil(pp)} Prozent)`
|
||||
: "noch nichts zugetragen";
|
||||
return k.schild === soll;
|
||||
});
|
||||
ok(schildOk, `und sagt dasselbe wie das Bild ("${z[0]?.schild}")`);
|
||||
ok(schildOk, `und sagt dasselbe wie das Bild `
|
||||
+ `(${z.map((k) => `"${k.schild}"`).join(" | ")})`);
|
||||
|
||||
/* EINE NULL IST KEINE WARNUNG: "verschieden gesehen" steht nur da,
|
||||
wenn es etwas gibt. Im Testbestand hat niemand etwas anders
|
||||
|
||||
Reference in New Issue
Block a user