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

Bearbeiten und Löschen von Gruppen #148

Merged
sneeex merged 20 commits from feature/118-bearbeiten-und-löschen-von-gruppen into development 2026-03-09 20:30:38 +00:00
Conversation 16 Commits 20 Files Changed 11 +28 -3
2 changed files with 28 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a304d9adf7 - Show all commits

29
lib/data/dao/group_dao.dart
View File

@@ -1,6 +1,7 @@
import 'package:drift/drift.dart';
import 'package:tallee/data/db/database.dart';
import 'package:tallee/data/db/tables/group_table.dart';
import 'package:tallee/data/db/tables/match_table.dart';
import 'package:tallee/data/db/tables/player_group_table.dart';
import 'package:tallee/data/dto/group.dart';
import 'package:tallee/data/dto/match.dart';
@@ -8,7 +9,7 @@ import 'package:tallee/data/dto/player.dart';
part 'group_dao.g.dart';
@DriftAccessor(tables: [GroupTable, PlayerGroupTable])
@DriftAccessor(tables: [GroupTable, PlayerGroupTable, MatchTable])
class GroupDao extends DatabaseAccessor<AppDatabase> with _$GroupDaoMixin {
GroupDao(super.db);
@@ -173,9 +174,31 @@ class GroupDao extends DatabaseAccessor<AppDatabase> with _$GroupDaoMixin {
}
/// Retrieves all matches associated with the given [groupId].
sneeex marked this conversation as resolved Outdated
flixcoo commented 2026-03-09 20:03:16 +00:00
Outdated
Review
Copy Link

Du sollst direkt die Datenbankanfrage so gestalten, dass du nur die Matches bekommst, welche die Gruppe enthalten. Das war mein Punkt

Du sollst direkt die Datenbankanfrage so gestalten, dass du nur die Matches bekommst, welche die Gruppe enthalten. Das war mein Punkt
sneeex commented 2026-03-09 20:05:15 +00:00
Outdated
Review
Copy Link

so wenig effort wie möglich

so wenig effort wie möglich
flixcoo commented 2026-03-09 20:05:54 +00:00
Outdated
Review
Copy Link

ja schlimm

ja schlimm
sneeex commented 2026-03-09 20:09:58 +00:00
Outdated
Review
Copy Link

match name soll nicht nullable sein oder?

match name soll nicht nullable sein oder?
sneeex commented 2026-03-09 20:10:10 +00:00
Outdated
Review
Copy Link
grafik.png
<img width="602" alt="grafik.png" src="attachments/052438c1-2d54-4df8-8450-db3bedd8f495">
grafik.png
176 KiB
flixcoo commented 2026-03-09 20:11:06 +00:00
Outdated
Review
Copy Link

Eigentlich nicht, das ist schon wieder Fusch am Bau

Eigentlich nicht, das ist schon wieder Fusch am Bau
/// Queries the database directly, filtering by [groupId].
Future<List<Match>> getGroupMatches({required String groupId}) async {
final matches = await db.matchDao.getAllMatches();
return matches.where((match) => match.group?.id == groupId).toList();
final query = select(matchTable)..where((m) => m.groupId.equals(groupId));
final rows = await query.get();
return Future.wait(
rows.map((row) async {
final game = await db.gameDao.getGameById(gameId: row.gameId);
final group = await db.groupDao.getGroupById(groupId: groupId);
final players =
await db.playerMatchDao.getPlayersOfMatch(matchId: row.id) ?? [];
final winner = await db.matchDao.getWinner(matchId: row.id);
return Match(
id: row.id,
name: row.name ?? '',
game: game,
group: group,
players: players,
notes: row.notes ?? '',
createdAt: row.createdAt,
endedAt: row.endedAt,
winner: winner,
);
}),
);
}
/// Deletes the group with the given [id] from the database.

2
lib/data/dao/group_dao.g.dart
View File

@@ -8,4 +8,6 @@ mixin _$GroupDaoMixin on DatabaseAccessor<AppDatabase> {
$PlayerTableTable get playerTable => attachedDatabase.playerTable;
$PlayerGroupTableTable get playerGroupTable =>
attachedDatabase.playerGroupTable;
$GameTableTable get gameTable => attachedDatabase.gameTable;
$MatchTableTable get matchTable => attachedDatabase.matchTable;
}