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

Fehler bei Match-Erstellung #104

Merged
flixcoo merged 5 commits from bug/102-fehler-bei-match-erstellung into development 2025-12-30 20:17:19 +00:00
Conversation 1 Commits 5 Files Changed 6 +112 -16
flixcoo commented 2025-12-30 17:42:35 +00:00
Owner
Copy Link

Fehler bei Match-Erstellung

Zugehörige Issue(s):
Closes #102

Beschreibung

Es wurde ein Bug behoben, der dazu geführt hat, dass neu erstellte Matches alle Player-Match Verbindungen löschen, welche die Spieler des neu erstellten Matches enthalten.

Änderungen

  • Insertmodes der DAO Funktion korrigiert
  • addPlayer() und addGroup() Funktionen aus der addMatch() Funktion entfernt.

Zusätzliche Anmerkungen

Die Funktion addMatch() wird in der CreateMatchView genutzt, um das Match an dieser Stelle zu erstellen. Dies bedeutet, dass alle Spieler und Gruppen zu diesem Zeitpunkt schon existieren, weshalb die Funktionen addPlayer() und addGroup() an dieser Stelle unnötig sind und halt auch den Fehler auslösten. Der Grund war, dass durch den InsertMode InsertMode.insertOrReplace die entsprechenden Datensätze an Spielern/Gruppen gelöscht und dann neu erstellt werden. Das Löschen führte aber dazu, dass alle vorher existierenden Player-Match bzw. Group-Match assoziationen gelöscht wurden (wegen onDeleteCascade).
In der Methode addMatchAsList() ist das aber anders. Hier werden Gruppen und Spieler erstellt, da diese Methode beim Import verwendet wird, und hier davon ausgegangen werden muss, dass die entsprechenden Spieler und Gruppen nicht existieren.

# Fehler bei Match-Erstellung **Zugehörige Issue(s):** Closes #102 ## Beschreibung Es wurde ein Bug behoben, der dazu geführt hat, dass neu erstellte Matches alle `Player`-`Match` Verbindungen löschen, welche die Spieler des neu erstellten Matches enthalten. ## Änderungen - [x] Insertmodes der DAO Funktion korrigiert - [x] `addPlayer()` und `addGroup()` Funktionen aus der `addMatch()` Funktion entfernt. ## Zusätzliche Anmerkungen Die Funktion `addMatch()` wird in der `CreateMatchView` genutzt, um das Match an dieser Stelle zu erstellen. Dies bedeutet, dass alle Spieler und Gruppen zu diesem Zeitpunkt schon existieren, weshalb die Funktionen `addPlayer()` und `addGroup()` an dieser Stelle unnötig sind und halt auch den Fehler auslösten. Der Grund war, dass durch den InsertMode `InsertMode.insertOrReplace` die entsprechenden Datensätze an Spielern/Gruppen gelöscht und dann neu erstellt werden. Das Löschen führte aber dazu, dass alle vorher existierenden `Player`-`Match` bzw. `Group`-`Match` assoziationen gelöscht wurden (wegen `onDeleteCascade`). In der Methode `addMatchAsList()` ist das aber anders. Hier werden Gruppen und Spieler erstellt, da diese Methode beim Import verwendet wird, und hier davon ausgegangen werden muss, dass die entsprechenden Spieler und Gruppen nicht existieren.
flixcoo added this to the Minimum Viable Product (MVP) milestone 2025-12-30 17:42:35 +00:00
flixcoo added the
PR
Ready for Review
2
 label 2025-12-30 17:42:35 +00:00
flixcoo self-assigned this 2025-12-30 17:42:35 +00:00
flixcoo added 2 commits 2025-12-30 17:42:35 +00:00
Updated insert mode to fix problem with replacing existing player match connections 356cb1fb43
Removed lines that added the group and players to the db again in addMatch()
Some checks failed
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
Details
Pull Request Pipeline / test (pull_request) Failing after 2m19s
Details
b8b65c4ca1
flixcoo requested review from gelbeinhalb 2025-12-30 17:42:35 +00:00
flixcoo requested review from sneeex 2025-12-30 17:42:36 +00:00
flixcoo added 1 commit 2025-12-30 17:45:32 +00:00
Updated comments
Some checks failed
Pull Request Pipeline / test (pull_request) Failing after 2m9s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m9s
Details
9cb35dc4a1
flixcoo added 1 commit 2025-12-30 17:52:02 +00:00
Fixed tests with adding players and groups to database on setup
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m11s
Details
e19f696714
flixcoo added 1 commit 2025-12-30 19:57:07 +00:00
Added tests for bug causing behaviour
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m6s
Details
Pull Request Pipeline / lint (pull_request) Successful in 2m8s
Details
c1789458e0
sneeex approved these changes 2025-12-30 20:16:36 +00:00
sneeex left a comment
Owner
Copy Link

sieht scheise aus, hab aber mitleid, also approved

sieht scheise aus, hab aber mitleid, also approved
flixcoo merged commit 27916456db into development 2025-12-30 20:17:19 +00:00
flixcoo deleted branch bug/102-fehler-bei-match-erstellung 2025-12-30 20:17:19 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
gelbeinhalb
sneeex
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
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
Refactoring
Task
Setup
Tasks related to project configuration, tooling, or infrastructure.
Task
Testing
Issue or pull request related to testing
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
PR
Ready for Review
2
Milestone
No items
No Milestone Minimum Viable Product (MVP)
Assignees
Clear assignees
flixcoo gelbeinhalb sneeex
No Assignees flixcoo
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: liquid-development/game-tracker#104
Delete Branch "bug/102-fehler-bei-match-erstellung"

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?