liquid-development/game-tracker
3
1
Fork 0
Code Issues 7 Pull Requests 4 Actions Releases 1 Activity

Datenbank onDelete verhalten überprüfen #195

New Issue
Open
opened 2026-03-09 14:10:06 +00:00 by sneeex · 2 comments
sneeex commented 2026-03-09 14:10:06 +00:00
Owner
Copy Link

Beschreibung

In der Match-Tabelle war die groupId mit onDelete: KeyAction.cascade gesetzt. Das hätte bedeutet, dass beim Löschen einer Gruppe auch alle assoziierten Matches gelöscht werden. Diesen Fall habe ich jetzt auf setNull geändert. Das heißt: Wenn eine Gruppe gelöscht wird, wird die groupId in den verknüpften Matches auf null gesetzt.

Mir geht es vor allem darum, dass möglicherweise auch in anderen Tabellen ähnliche Fehler vorhanden sind. Wenn man so etwas nicht im Kopf hat, kann das zu sehr seltsamem Verhalten führen, das man dann unter Umständen wieder jahrelang debuggt.

Also sollten einmal alle Tabellen auf so welche Probleme überprüft werden.

Schritte zur Reproduktion

/

Umgebung

No response

Lösungsidee

/

Verwandte Issues

/

### Beschreibung In der Match-Tabelle war die groupId mit `onDelete: KeyAction.cascade` gesetzt. Das hätte bedeutet, dass beim Löschen einer Gruppe auch alle assoziierten Matches gelöscht werden. Diesen Fall habe ich jetzt auf `setNull` geändert. Das heißt: Wenn eine Gruppe gelöscht wird, wird die `groupId` in den verknüpften Matches auf `null` gesetzt. Mir geht es vor allem darum, dass möglicherweise auch in anderen Tabellen ähnliche Fehler vorhanden sind. Wenn man so etwas nicht im Kopf hat, kann das zu sehr seltsamem Verhalten führen, das man dann unter Umständen wieder jahrelang debuggt. Also sollten einmal alle Tabellen auf so welche Probleme überprüft werden. ### Schritte zur Reproduktion / ### Umgebung _No response_ ### Lösungsidee / ### Verwandte Issues /
sneeex added the
Task
Bug
 label 2026-03-09 14:10:06 +00:00
flixcoo added the
Layer
DB
Priority
High
2
 labels 2026-03-09 19:03:10 +00:00
flixcoo changed time estimate to 2h 2026-03-09 19:52:05 +00:00
sneeex commented 2026-03-09 20:16:13 +00:00
Author
Owner
Copy Link
  • sachen wie nullable überprüfen, bspw. war match name nullable, was ja keinen sinn macht
+ sachen wie nullable überprüfen, bspw. war match name nullable, was ja keinen sinn macht
flixcoo added this to the Alpha milestone 2026-03-09 20:29:14 +00:00
flixcoo self-assigned this 2026-03-09 20:29:16 +00:00
gelbeinhalb self-assigned this 2026-04-13 14:22:36 +00:00
flixcoo removed their assignment 2026-04-13 21:17:49 +00:00
flixcoo added the Technical Debt label 2026-04-16 12:27:32 +00:00
gelbeinhalb started working 2026-04-19 10:36:35 +00:00
gelbeinhalb added reference bug/195-datenbank-onDelete-ueberpruefen 2026-04-19 10:39:47 +00:00
gelbeinhalb worked for 35 minutes 2026-04-19 11:11:40 +00:00
gelbeinhalb added spent time 30 minutes 2026-04-30 09:01:05 +00:00
gelbeinhalb started working 2026-04-30 09:01:06 +00:00
gelbeinhalb worked for 55 minutes 2026-04-30 09:56:32 +00:00
gelbeinhalb started working 2026-05-12 18:01:16 +00:00
gelbeinhalb commented 2026-05-12 19:05:27 +00:00
Owner
Copy Link

Ich hab die sachen mit Nullable und dem cascade delete überprüft und keine weiteren Fehler gefunden.
Momentan ergeben alle usages von onDelete: KeyAction.cascade einen Sinn.

Ich hab die sachen mit Nullable und dem cascade delete überprüft und keine weiteren Fehler gefunden. Momentan ergeben alle usages von `onDelete: KeyAction.cascade` einen Sinn.
gelbeinhalb worked for 1 hour 9 minutes 2026-05-12 19:10:31 +00:00
gelbeinhalb started working 2026-05-12 19:12:14 +00:00
gelbeinhalb worked for 54 minutes 2026-05-12 20:07:00 +00:00
sneeex added a new dependency 2026-05-15 13:15:13 +00:00
#180 Spielerprofile implementieren
flixcoo added the
Workflow
In Progress
3
 label 2026-05-16 15:50:08 +00:00
Sign in to join this conversation.
bug/195-datenbank-onDelete-ueberpruefen
Branches Tags
Labels
Clear labels
Duplicate
This issue or pull request already exists
Layer
BE
Issues/PRs related to the backend
Layer
DB
Issues/PRs related to the database
Layer
FE
Issues/PRs related to the frontend
Layer
Full Stack
Issues/PRs spanning multiple layers (frontend, backend, database)
PR
In Review
1
Pull Request that is reviewed
PR
Ready for Review
2
Pull Requests which are ready to be reviewed
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Proposal
This issue contains a proposal for a new feature, change, or improvement. It is open for discussion and feedback before any decision is made.
Task
Bug
Reports of unexpected behavior or defects in existing functionality.
Task
Documentation
Tasks related to improving or adding documentation (READMEs, code comments, guides).
Task
Enhancement
Improvements or optimizations to existing features (non-breaking changes).
Task
Feature
Requests for new functionality or significant additions to the project.
Task
Hotfix
Task
Refactoring
Tasks that only involve refactoring and do not add any new functionality
Task
Setup
Tasks related to project configuration, tooling, or infrastructure.
Task
Testing
Issue or pull request related to testing
Technical Debt
Issues related to code that needs to be refactored, cleaned up, or improved due to shortcuts or outdated implementations and to keep the codebase maintainable.
Workflow
Abandoned
4
Somebody has started to work on this but abandoned work
Workflow
Backlog
4
Issues/PRs that are acknowledged but not being actively worked on yet.
Workflow
In Progress
3
Issues/PRs currently being actively worked on.
Workflow
On Hold
3
Issues/PRs temporarily being paused or blocked.
Workflow
Won't Fix
5
This issue won't be fixed
No Label
Layer
DB
Priority
High
2
Task
Bug
 Technical Debt
Workflow
In Progress
3
Milestone
No items
No Milestone Alpha
Assignees
Clear assignees
actions flixcoo gelbeinhalb sneeex
No Assignees gelbeinhalb
2 Participants
Notifications
Total Time Spent: 4 hours 4 minutes
gelbeinhalb
4 hours 4 minutes
Due Date
No due date set.
Blocks
#180 Spielerprofile implementieren
liquid-development/game-tracker
Reference: liquid-development/game-tracker#195
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?