bugs.md #10: Drei nicht funktionierende Alert-"Abbrechen"/"OK"-Buttons #22

Closed
opened 2026-09-17 21:10:06 +02:00 by kay · 0 comments
Owner

Quelle: bugs.md #10 (README-Milestone-Nachcheck, 2026-09-17)
Confidence: hoch (Logikfehler direkt im Code nachvollzogen, nicht live reproduziert)

Beim erneuten Review der Port-Konflikt-Prüfung (README-Milestone-Nachcheck) drei eigenständige Bugs derselben Klasse gefunden:

  1. ReviewApplyViews "Anwenden fehlgeschlagen"-Alert: OK-Button-Action leer, isPresented-Bindings-Setter ebenfalls No-Op — applyError wurde nie zurückgesetzt.
  2. ConnectViews "Unbekanntes Zertifikat"-Alert: "Abbrechen"-Button-Action komplett leer — connectionService.state blieb dauerhaft auf .needsCertificateConfirmation hängen.
  3. ConnectViews "Unbekannter SSH-Schlüssel"-Alert: "Abbrechen" funktionierte nur für einen von zwei möglichen Auslöse-Pfaden.

Fix: neue ConnectionService.cancelPendingTrustConfirmation(), neue SetupViewModel.dismissApplyError(). Alle drei Alerts korrekt verdrahtet (Button-Action + Bindings-Setter, deckt auch Tap-Outside/Esc ab). Totes dismissPendingSSHTrust() entfernt.

Verifikation: Build grün, 102 Unit-Tests grün (1 neuer Regressionstest). Noch nicht live gegenreproduziert.

**Quelle:** bugs.md #10 (README-Milestone-Nachcheck, 2026-09-17) **Confidence:** hoch (Logikfehler direkt im Code nachvollzogen, nicht live reproduziert) Beim erneuten Review der Port-Konflikt-Prüfung (README-Milestone-Nachcheck) drei eigenständige Bugs derselben Klasse gefunden: 1. `ReviewApplyView`s "Anwenden fehlgeschlagen"-Alert: OK-Button-Action leer, `isPresented`-Bindings-Setter ebenfalls No-Op — `applyError` wurde nie zurückgesetzt. 2. `ConnectView`s "Unbekanntes Zertifikat"-Alert: "Abbrechen"-Button-Action komplett leer — `connectionService.state` blieb dauerhaft auf `.needsCertificateConfirmation` hängen. 3. `ConnectView`s "Unbekannter SSH-Schlüssel"-Alert: "Abbrechen" funktionierte nur für einen von zwei möglichen Auslöse-Pfaden. **Fix:** neue `ConnectionService.cancelPendingTrustConfirmation()`, neue `SetupViewModel.dismissApplyError()`. Alle drei Alerts korrekt verdrahtet (Button-Action + Bindings-Setter, deckt auch Tap-Outside/Esc ab). Totes `dismissPendingSSHTrust()` entfernt. **Verifikation:** Build grün, 102 Unit-Tests grün (1 neuer Regressionstest). Noch nicht live gegenreproduziert.
kay closed this issue 2026-09-17 21:10:07 +02:00
Sign in to join this conversation.
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: kay/RouterOS#22