[Bug/Data] Keine Transaktionen bei mehrstufigen Schreibvorgaengen #50

New issue

Closed

opened 2026-09-09 20:17:01 +00:00 by Aceless · 1 comment

Aceless commented

2026-09-09 20:17:01 +00:00

Owner

Problem

Mehrere Operationen bestehen aus mehreren unabhaengigen Statements ohne Transaktion:

RecipeRepository.CreateRecipeInSpace: Rezept, Zutaten, Schritte, Tags, SuitableFor sind fuenf getrennte Aufrufe.
RecipeRepository.UpdateRecipeInSpace: erst DELETE, dann INSERT je Kindtabelle. Bricht es dazwischen ab, ist das Rezept ohne Zutaten oder Schritte.
SpaceRepository.CreateSpaceAsync: Space anlegen und Mitgliedschaft anlegen. Faellt der zweite Teil aus, existiert ein Space, den niemand sehen kann.

Da IDbConnection als Scoped registriert und von Dapper je Statement geoeffnet und geschlossen wird, laeuft jedes Statement im Autocommit.

Fix

Verbindung explizit oeffnen, BeginTransaction(), die Transaktion an jeden Dapper-Aufruf durchreichen und am Ende committen. Sinnvollerweise gekapselt in einer kleinen Hilfsmethode oder einem Unit-of-Work, damit die Repositories nicht jeweils eigene Verbindungsverwaltung bekommen.

## Problem Mehrere Operationen bestehen aus mehreren unabhaengigen Statements ohne Transaktion: - `RecipeRepository.CreateRecipeInSpace`: Rezept, Zutaten, Schritte, Tags, SuitableFor sind fuenf getrennte Aufrufe. - `RecipeRepository.UpdateRecipeInSpace`: erst DELETE, dann INSERT je Kindtabelle. Bricht es dazwischen ab, ist das Rezept ohne Zutaten oder Schritte. - `SpaceRepository.CreateSpaceAsync`: Space anlegen und Mitgliedschaft anlegen. Faellt der zweite Teil aus, existiert ein Space, den niemand sehen kann. Da `IDbConnection` als Scoped registriert und von Dapper je Statement geoeffnet und geschlossen wird, laeuft jedes Statement im Autocommit. ## Fix Verbindung explizit oeffnen, `BeginTransaction()`, die Transaktion an jeden Dapper-Aufruf durchreichen und am Ende committen. Sinnvollerweise gekapselt in einer kleinen Hilfsmethode oder einem Unit-of-Work, damit die Repositories nicht jeweils eigene Verbindungsverwaltung bekommen.

Aceless added the

bug

database

labels

2026-09-09 20:17:01 +00:00

Aceless referenced this issue from a commit

2026-09-09 21:18:12 +00:00

Infrastruktur fuer Transaktionen, Rechtepruefung und Bild-Uploads

Aceless referenced this issue from a commit

2026-09-09 21:18:12 +00:00

Spaces: defektes PATCH, NullReference und Rollenlogik

Aceless referenced this issue from a commit

2026-09-09 21:18:12 +00:00

Rezepte: Transaktionen, verwaiste Bilddateien, Batch-Laden

Aceless commented

2026-09-09 21:18:51 +00:00

Author

Owner

Erledigt.

Neu: Infrastructure/DbConnectionExtensions.BeginTransactionAsync, das die Verbindung bei Bedarf öffnet. Die Transaktion wird an jeden Dapper-Aufruf durchgereicht.

Umgestellt sind:

CreateRecipeInSpace: Rezept, Zutaten, Schritte, Tags und Mealtypes in einer Transaktion.
UpdateRecipeInSpace: DELETE und INSERT je Kindtabelle in einer Transaktion.
CreateSpaceAsync: Space und Mitgliedschaft in einer Transaktion.
DeleteSpaceAsync und DeleteRecipeInSpace: Bildpfade einsammeln und löschen in einer Transaktion.

IDbConnection kommt jetzt aus einer NpgsqlDataSource als Singleton, das Pooling bleibt erhalten.

Branch backend/issue-fixes, Commits 026544c, c59a727, 3957404.

Erledigt. Neu: `Infrastructure/DbConnectionExtensions.BeginTransactionAsync`, das die Verbindung bei Bedarf öffnet. Die Transaktion wird an jeden Dapper-Aufruf durchgereicht. Umgestellt sind: - `CreateRecipeInSpace`: Rezept, Zutaten, Schritte, Tags und Mealtypes in einer Transaktion. - `UpdateRecipeInSpace`: DELETE und INSERT je Kindtabelle in einer Transaktion. - `CreateSpaceAsync`: Space und Mitgliedschaft in einer Transaktion. - `DeleteSpaceAsync` und `DeleteRecipeInSpace`: Bildpfade einsammeln und löschen in einer Transaktion. `IDbConnection` kommt jetzt aus einer `NpgsqlDataSource` als Singleton, das Pooling bleibt erhalten. --- Branch `backend/issue-fixes`, Commits 026544c, c59a727, 3957404.

Aceless closed this issue

2026-09-09 21:18:51 +00:00