Skip to content

Lastenheft_SQLite_ViewQuery_Bugfix.md

Latest commit

 

History

History
118 lines (79 loc) · 6.72 KB

Lastenheft_SQLite_ViewQuery_Bugfix.md

File metadata and controls

118 lines (79 loc) · 6.72 KB

Lastenheft: Bugfix fehlerhafte View-Abfragen im SqliteDbService

Dokument-Status: Entwurf Erstellt: 2026-04-18 Betrifft: InventarWorkerCommon/Services/Database/SqliteDbService.cs Prioritaet: Mittel (funktionaler Bug, liefert aktuell ungefilterte Daten statt View-Ergebnisse) Herkunft: Review des Lastenhefts "Vervollstaendigung der PostgreSQL-Implementierung", Punkt 1 Reihenfolge: 2 von 5

Umsetzungsreihenfolge aller Lastenhefte

Nr. Lastenheft Abhaengigkeit
1 Lastenheft_PostgreSQL_Implementation.md Keine
2 Lastenheft_SQLite_ViewQuery_Bugfix.md (dieses Dokument) Keine (unabhaengig, aber nach Nr. 1 geplant)
3 Lastenheft_IDbService_Interface.md Setzt Nr. 1 voraus
4 Lastenheft_Statistik_View_Lesemethoden.md Setzt Nr. 1 voraus
5 Lastenheft_MongoDB_Paritaet.md Keine direkte, logisch nach Nr. 1

Ausgangslage

Im SqliteDbService enthalten zwei Methoden fehlerhaftes SQL. Statt die definierten Views AllDeprovisionedMachinesView und AllDisabledMachinesView abzufragen, wird die Tabelle Machines mit einem Alias gelesen. Das fuehrt dazu, dass die WHERE-Bedingungen der Views nicht greifen und alle Maschinen zurueckgegeben werden.

Methode Zeile Fehlerhaftes SQL Erwartetes SQL
GetAllDeprovisionedMachinesAsync 496 SELECT * FROM Machines AllDeprovisionedMachinesView ORDER BY Name SELECT * FROM AllDeprovisionedMachinesView ORDER BY Name
GetAllDisabledMachinesAsync 511 SELECT * FROM Machines AllDisabledMachinesView ORDER BY Name SELECT * FROM AllDisabledMachinesView ORDER BY Name

Zum Vergleich: Die Methoden GetAllActiveMachinesAsync (Zeile 464) und GetAllActiveMachinesWithNetworkInfoAsync (Zeile 482) verwenden korrekt die jeweilige View.

Anforderungen

R-SQLBUG-01: Korrektur der View-Abfrage in GetAllDeprovisionedMachinesAsync

Die SQL-Abfrage in SqliteDbService.GetAllDeprovisionedMachinesAsync() muss korrigiert werden:

  • Ist: SELECT * FROM Machines AllDeprovisionedMachinesView ORDER BY Name
  • Soll: SELECT * FROM AllDeprovisionedMachinesView ORDER BY Name

R-SQLBUG-02: Korrektur der View-Abfrage in GetAllDisabledMachinesAsync

Die SQL-Abfrage in SqliteDbService.GetAllDisabledMachinesAsync() muss korrigiert werden:

  • Ist: SELECT * FROM Machines AllDisabledMachinesView ORDER BY Name
  • Soll: SELECT * FROM AllDisabledMachinesView ORDER BY Name

R-SQLBUG-03: Testabdeckung

Fuer beide korrigierten Methoden muessen Unit-Tests existieren, die sicherstellen, dass:

  • Nur Maschinen mit Disabled = 1 AND Deprovisioned = 1 von GetAllDeprovisionedMachinesAsync zurueckgegeben werden.
  • Nur Maschinen mit Disabled = 1 AND Deprovisioned = 0 von GetAllDisabledMachinesAsync zurueckgegeben werden.
  • Maschinen mit abweichenden Flag-Kombinationen nicht im Ergebnis enthalten sind.

Nicht im Scope

  • Aenderungen an der PostgreSQL-Implementierung (wird separat im Lastenheft "PostgreSQL-Implementierung" behandelt).
  • Aenderungen an den View-Definitionen selbst (diese sind korrekt).
  • Refactoring oder Optimierung anderer Methoden im SqliteDbService.

Akzeptanzkriterien

ID Kriterium
AK-SQLBUG-01 GetAllDeprovisionedMachinesAsync gibt ausschliesslich Maschinen mit Disabled = 1 AND Deprovisioned = 1 zurueck.
AK-SQLBUG-02 GetAllDisabledMachinesAsync gibt ausschliesslich Maschinen mit Disabled = 1 AND Deprovisioned = 0 zurueck.
AK-SQLBUG-03 Unit-Tests fuer beide Methoden sind vorhanden und gruen.
AK-SQLBUG-04 dotnet build laeuft ohne Warnungen durch.

Hinweis fuer Lernende

Deutsch: Dieser Bug zeigt ein haeufiges SQL-Muster: SELECT * FROM TabelleA AliasB ist syntaktisch korrekt und setzt einen Tabellen-Alias. Das Ergebnis ist jedoch voellig anders als SELECT * FROM ViewB. Solche Fehler fallen oft erst auf, wenn Filterbedingungen geprueft werden -- der Query laeuft fehlerfrei, liefert aber falsche Daten.

English: This bug demonstrates a common SQL pattern: SELECT * FROM TableA AliasB is syntactically valid and sets a table alias. The result, however, is completely different from SELECT * FROM ViewB. Such errors often go unnoticed until filter conditions are verified -- the query runs without errors but returns incorrect data.

Spec-Kit-Intake-Reife / Spec Kit Intake Readiness

Dieses Lastenheft ist als Eingabedatei fuer einen spaeteren /speckit-specify-Lauf vorgesehen. Vor dem Start muss der aktuelle Repository-Stand geprueft werden, damit bereits erledigte oder ueberholte Punkte nicht erneut umgesetzt werden.

This requirements document is intended as input for a later /speckit-specify run. Before starting, check the current repository state so already completed or superseded items are not implemented again.

Der spaetere Lauf muss mindestens klassifizieren:

  • Applicable: gilt fuer diesen Lauf und braucht Umsetzung oder Evidenz.
  • AlreadySatisfied: ist im aktuellen Stand bereits nachweisbar erledigt.
  • N/A: gilt fuer diesen Lauf nicht und braucht eine kurze Begruendung.
  • Open: gilt, ist aber noch nicht ausreichend geklaert oder belegt.
  • FollowUp: fachlich relevant, aber nicht Teil dieses Laufs.

Kopierbarer /speckit-specify-Prompt / Copyable /speckit-specify Prompt

/speckit-specify Nutze Lastenheft_SQLite_ViewQuery_Bugfix.md als verbindliche Eingabedatei. Erstelle die Feature-Spezifikation fuer einen SQLite-Bugfix- und Datenzugriffslauf im Repository InventarWorkerService.

Ziel: Pruefe das Lastenheft gegen den aktuellen Repository-Stand und erstelle eine belastbare Spec-Kit-Spezifikation, die fuer Auszubildende, Entwickler*innen, Reviewer und KI-Agenten nachvollziehbar ist.

Pflichtpunkte:
- Lies dieses Lastenheft vollstaendig und uebernehme vorhandene Anforderungen, Scope-Grenzen, Reihenfolgehinweise und Akzeptanzkriterien.
- Pruefe, welche Punkte bereits umgesetzt, ueberholt oder noch offen sind.
- Klassifiziere Anforderungen als `Applicable`, `AlreadySatisfied`, `N/A`, `Open` oder `FollowUp`.
- Plane nur `Applicable`-Punkte fuer diesen Lauf.
- Dokumentiere fuer `N/A` und `FollowUp` jeweils eine kurze Begruendung.
- Beachte `constitution.md`, `.specify/memory/constitution.md`, AGENTS/CLAUDE/GEMINI/Copilot-Guidance, installierte Spec-Kit-Presets, Secure-Development-Basis, A11Y-Regeln, CEFR-B2-Verstaendlichkeit und didaktische Kommentar-Governance.
- Starte keinen weiteren Lastenheft-Lauf und kombiniere mehrere Lastenhefte nur, wenn die Kopplung fachlich begruendet und dokumentiert ist.

Erzeuge eine Spezifikation mit Scope, Nicht-Zielen, Anforderungen, Abhaengigkeiten, Akzeptanzkriterien, Risiken, Teststrategie, Evidenzpfaden und offenen Folgepunkten.