Update-Hinweis wird nie angezeigt: Konstruktor-Parameter untypisiert und ungenutzt #2

Closed
opened 2026-09-09 20:12:42 +00:00 by Aceless · 1 comment
Owner
const RecipePlanerApp({super.key, needsUpdate = false, updateUri = ""});

needsUpdate und updateUri sind weder typisiert noch Feldern zugewiesen (kein this.) und werden im build nirgends gelesen. Die komplette Update-Logik in main() laeuft also ins Leere, der Nutzer sieht nie einen Update-Hinweis.

Vorschlag: Als final bool needsUpdate / final String updateUri deklarieren und im build einen Dialog oder ein Banner mit Link auf updateUri zeigen. Siehe auch das Issue zum JSON-Casing in AppInfoService - aktuell wuerde der Flag ohnehin nie gesetzt.

Datei: app/lib/main.dart:34-49

```dart const RecipePlanerApp({super.key, needsUpdate = false, updateUri = ""}); ``` `needsUpdate` und `updateUri` sind weder typisiert noch Feldern zugewiesen (kein `this.`) und werden im `build` nirgends gelesen. Die komplette Update-Logik in `main()` laeuft also ins Leere, der Nutzer sieht nie einen Update-Hinweis. **Vorschlag:** Als `final bool needsUpdate` / `final String updateUri` deklarieren und im `build` einen Dialog oder ein Banner mit Link auf `updateUri` zeigen. Siehe auch das Issue zum JSON-Casing in `AppInfoService` - aktuell wuerde der Flag ohnehin nie gesetzt. Datei: `app/lib/main.dart:34-49`
Author
Owner

Behoben auf Branch frontend-improvements.

needsUpdate/updateUri ersetzt durch appUpdateProvider plus Banner, das den Link per url_launcher oeffnet.

Commits
  • 47edeeb Added Claude
  • 9b16a6e Einstellungen mit Ladebalken, url_launcher-Plugin-Registrierung
  • 8ad68a4 Settings erreichbar, Secrets sichern, Theme-Tokens, toter Code weg, Tests
  • bf204d3 Rezeptliste, Detail, Vorschlaege und Spaces ueberarbeiten
  • d85c681 Rezept-Editor: Absturz, stiller Datenverlust und Controller-Leaks beheben
  • d970fbe Provider-State korrigieren, Fehler sichtbar machen, Wochenlogik zentralisieren
  • 71e557d Start entkoppeln, API-Client haerten, Theme auf Tokens umstellen

flutter analyze: 0 Issues. flutter test: 22 Tests gruen.

Behoben auf Branch `frontend-improvements`. needsUpdate/updateUri ersetzt durch appUpdateProvider plus Banner, das den Link per url_launcher oeffnet. <details><summary>Commits</summary> - 47edeeb Added Claude - 9b16a6e Einstellungen mit Ladebalken, url_launcher-Plugin-Registrierung - 8ad68a4 Settings erreichbar, Secrets sichern, Theme-Tokens, toter Code weg, Tests - bf204d3 Rezeptliste, Detail, Vorschlaege und Spaces ueberarbeiten - d85c681 Rezept-Editor: Absturz, stiller Datenverlust und Controller-Leaks beheben - d970fbe Provider-State korrigieren, Fehler sichtbar machen, Wochenlogik zentralisieren - 71e557d Start entkoppeln, API-Client haerten, Theme auf Tokens umstellen </details> `flutter analyze`: 0 Issues. `flutter test`: 22 Tests gruen.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Aceless/RecipePlanner#2
No description provided.