confirmCycle: se il server rifiuta /cycle lo schermo mostra comunque il ciclo chiuso #29

Closed
opened 2026-09-21 20:14:50 +00:00 by Adriano · 3 comments
Owner

GRAVITÀ: media
FILE: src/frontend/flask_app/templates/measure/task_execute.html:1359-1384
COSA: cycleConfirmed=true prima della chiamata. Se postProduction torna null con produzione aperta, il codice cade nel ramo «nessuna produzione»: cycleCount++ locale, countdown locale, canLeaveTask true.
INPUT: produzione aperta, /cycle rifiutato (linea ferma, 5xx, rete giù).
VERIFICATO: lettura: if (run) {...return;} senza else, poi righe 1381-1384.

ORIGINE: revisione fable del 21/09/2026 su ramo V3.0.0, commit 8cc5e82. Documento completo: _output/2026-09-21-TieMeasureFlow-revisione.md su Wasabi.

GRAVITÀ: media FILE: src/frontend/flask_app/templates/measure/task_execute.html:1359-1384 COSA: `cycleConfirmed=true` prima della chiamata. Se `postProduction` torna null con produzione aperta, il codice cade nel ramo «nessuna produzione»: `cycleCount++` locale, countdown locale, `canLeaveTask` true. INPUT: produzione aperta, `/cycle` rifiutato (linea ferma, 5xx, rete giù). VERIFICATO: lettura: `if (run) {...return;}` senza else, poi righe 1381-1384. ORIGINE: revisione fable del 21/09/2026 su ramo `V3.0.0`, commit 8cc5e82. Documento completo: `_output/2026-09-21-TieMeasureFlow-revisione.md` su Wasabi.
Adriano added the bug label 2026-09-21 20:14:50 +00:00
Adriano added the assegnata:jeeg label 2026-09-21 20:23:14 +00:00
Author
Owner

⚡ jeeg — preso in carico 2026-09-21T20:23:14Z
ramo: lavoro/jeeg · capacità: docker, gpu, systemd

⚡ jeeg — **preso in carico** 2026-09-21T20:23:14Z ramo: `lavoro/jeeg` · capacità: docker, gpu, systemd
Author
Owner

ANALISI (passo 1) — ramo lavoro/jeeg, worktree ~/lavoro/TieMeasureFlow su jeeg, lotto B1

MODIFICHE: task_execute.html confirmCycle — se this.productionRun esiste e postProduction torna null: ripristina cycleConfirmed=false, riapri l'overlay di completamento, mostra l'errore del server, e return. Il ramo locale (cycleCount++, countdown) si esegue SOLO quando non c'è produzione aperta.
TEST: a lettura; passi manuali nel rapporto.

ANALISI (passo 1) — ramo `lavoro/jeeg`, worktree `~/lavoro/TieMeasureFlow` su jeeg, lotto B1 MODIFICHE: task_execute.html `confirmCycle` — se `this.productionRun` esiste e `postProduction` torna null: ripristina `cycleConfirmed=false`, riapri l'overlay di completamento, mostra l'errore del server, e `return`. Il ramo locale (cycleCount++, countdown) si esegue SOLO quando non c'è produzione aperta. TEST: a lettura; passi manuali nel rapporto.
Author
Owner

Soluzione

ESITO: fatto
COSA: confirmCycle: se la produzione è aperta e il server rifiuta, cycleConfirmed torna false, l'overlay si riapre con l'errore; il ramo locale vale solo senza produzione.
COMMIT (ramo lavoro/jeeg, unito su V3.0.0 a d483fa1): 31fa78b
PROVA: uv run --extra server --extra client --extra dev pytest -q → 622 passed, 4 skipped (erano 391); visione su Python 3.13 → 16 passed (erano 8).

**Soluzione** ESITO: fatto COSA: `confirmCycle`: se la produzione è aperta e il server rifiuta, `cycleConfirmed` torna false, l'overlay si riapre con l'errore; il ramo locale vale solo senza produzione. COMMIT (ramo lavoro/jeeg, unito su V3.0.0 a d483fa1): 31fa78b PROVA: `uv run --extra server --extra client --extra dev pytest -q` → 622 passed, 4 skipped (erano 391); visione su Python 3.13 → 16 passed (erano 8).
Adriano removed the assegnata:jeeg label 2026-09-21 21:56:35 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Adriano/TieMeasureFlow#29