add error message for game deletion and implement search functionality

lib/l10n/arb/app_de.arb

@@ -34,6 +34,7 @@
   "edit_match": "Gruppe bearbeiten",
   "enter_results": "Ergebnisse eintragen",
   "error_creating_group": "Fehler beim Erstellen der Gruppe, bitte erneut versuchen",
+  "error_deleting_game": "Fehler beim Löschen der Spielvorlage, bitte erneut versuchen",
   "error_deleting_group": "Fehler beim Löschen der Gruppe, bitte erneut versuchen",
   "error_editing_group": "Fehler beim Bearbeiten der Gruppe, bitte erneut versuchen",
   "error_reading_file": "Fehler beim Lesen der Datei",

lib/l10n/arb/app_en.arb

@@ -104,6 +104,9 @@
   "@error_creating_group": {
     "description": "Error message when group creation fails"
   },
+  "@error_deleting_game": {
+    "description": "Error message when game deletion fails"
+  },
   "@error_deleting_group": {
       "description": "Error message when group deletion fails"
   },
@@ -376,6 +379,7 @@
   "edit_match": "Edit Match",
   "enter_results": "Enter Results",
   "error_creating_group": "Error while creating group, please try again",
+  "error_deleting_game": "Error while deleting game, please try again",
   "error_deleting_group": "Error while deleting group, please try again",
   "error_editing_group": "Error while editing group, please try again",
   "error_reading_file": "Error reading file",

lib/l10n/generated/app_localizations.dart

@@ -296,6 +296,12 @@ abstract class AppLocalizations {
   /// **'Error while creating group, please try again'**
   String get error_creating_group;
 
+  /// Error message when game deletion fails
+  ///
+  /// In en, this message translates to:
+  /// **'Error while deleting game, please try again'**
+  String get error_deleting_game;
+
   /// Error message when group deletion fails
   ///
   /// In en, this message translates to:

lib/l10n/generated/app_localizations_de.dart

@@ -112,6 +112,10 @@ class AppLocalizationsDe extends AppLocalizations {
   String get error_creating_group =>
       'Fehler beim Erstellen der Gruppe, bitte erneut versuchen';
 
+  @override
+  String get error_deleting_game =>
+      'Fehler beim Löschen der Spielvorlage, bitte erneut versuchen';
+
   @override
   String get error_deleting_group =>
       'Fehler beim Löschen der Gruppe, bitte erneut versuchen';

lib/l10n/generated/app_localizations_en.dart

@@ -112,6 +112,10 @@ class AppLocalizationsEn extends AppLocalizations {
   String get error_creating_group =>
       'Error while creating group, please try again';
 
+  @override
+  String get error_deleting_game =>
+      'Error while deleting game, please try again';
+
   @override
   String get error_deleting_group =>
       'Error while deleting group, please try again';

lib/presentation/views/main_menu/match_view/create_match/choose_game_view.dart

@@ -37,12 +37,18 @@ class _ChooseGameViewState extends State<ChooseGameView> {
   /// Controller for the search bar
   final TextEditingController searchBarController = TextEditingController();
 
-  /// Currently selected game index
+  /// Currently selected game id
   late String selectedGameId;
 
+  /// Games filtered according to the current search query
+  late List<Game> filteredGames;
+
   @override
   void initState() {
     selectedGameId = widget.initialSelectedGameId;
+
+    // Start with all games visible
+    filteredGames = List<Game>.from(widget.games);
     super.initState();
   }
 
@@ -63,7 +69,7 @@ class _ChooseGameViewState extends State<ChooseGameView> {
           IconButton(
             icon: const Icon(Icons.add),
             onPressed: () async {
-              Game? newGame = await Navigator.push(
+              final result = await Navigator.push(
                 context,
                 adaptivePageRoute(
                   builder: (context) => CreateGameView(
@@ -73,10 +79,11 @@ class _ChooseGameViewState extends State<ChooseGameView> {
                   ),
                 ),
               );
-              if (newGame != null) {
+              if (result != null && result.game != null) {
                 setState(() {
-                  widget.games.insert(0, newGame);
+                  widget.games.insert(0, result.game);
                 });
+                _refreshFromSource();
               }
             },
           ),
@@ -100,46 +107,61 @@ class _ChooseGameViewState extends State<ChooseGameView> {
               child: CustomSearchBar(
                 controller: searchBarController,
                 hintText: loc.game_name,
+                onChanged: (value) {
+                  _applySearchFilter(value);
+                },
               ),
             ),
             const SizedBox(height: 5),
             Expanded(
               child: ListView.builder(
-                itemCount: widget.games.length,
+                itemCount: filteredGames.length,
                 itemBuilder: (BuildContext context, int index) {
+                  final game = filteredGames[index];
                   return TitleDescriptionListTile(
-                    title: widget.games[index].name,
+                    title: game.name,
-                    description: widget.games[index].description,
+                    description: game.description,
-                    badgeText: translateRulesetToString(
+                    badgeText: translateRulesetToString(game.ruleset, context),
-                      widget.games[index].ruleset,
+                    isHighlighted: selectedGameId == game.id,
-                      context,
-                    ),
-                    isHighlighted: selectedGameId == widget.games[index].id,
                     onTap: () async {
                       setState(() {
-                        if (selectedGameId == widget.games[index].id) {
+                        if (selectedGameId == game.id) {
                           selectedGameId = '';
                         } else {
-                          selectedGameId = widget.games[index].id;
+                          selectedGameId = game.id;
                         }
                       });
                     },
                     onLongPress: () async {
-                      Game? newGame = await Navigator.push(
+                      final result = await Navigator.push(
                         context,
                         adaptivePageRoute(
                           builder: (context) => CreateGameView(
-                            gameToEdit: widget.games[index],
+                            gameToEdit: game,
                             onGameCreatedOrEdited: () {
                               widget.onGamesUpdated?.call();
                             },
                           ),
                         ),
                       );
-                      if (newGame != null) {
+                      if (result != null && result.game != null) {
+                        // Find the index in the original list to mutate
+                        final originalIndex = widget.games.indexWhere(
+                          (g) => g.id == game.id,
+                        );
+                        if (originalIndex == -1) {
+                          return;
+                        }
+                        if (result.delete) {
                           setState(() {
-                          widget.games[index] = newGame;
+                            widget.games.removeAt(originalIndex);
                           });
+                        } else {
+                          setState(() {
+                            widget.games[originalIndex] = result.game;
+                          });
+                        }
+                        _refreshFromSource();
                       }
                     },
                   );
@@ -151,4 +173,28 @@ class _ChooseGameViewState extends State<ChooseGameView> {
       ),
     );
   }
+
+  /// Applies the search filter to the games list based on [query].
+  void _applySearchFilter(String query) {
+    final q = query.toLowerCase().trim();
+    if (q.isEmpty) {
+      setState(() {
+        filteredGames = List<Game>.from(widget.games);
+      });
+      return;
+    }
+
+    setState(() {
+      filteredGames = widget.games.where((game) {
+        final name = game.name.toLowerCase();
+        final description = game.description.toLowerCase();
+        return name.contains(q) || description.contains(q);
+      }).toList();
+    });
+  }
+
+  /// Re-applies the current filter after the underlying games list changed.
+  void _refreshFromSource() {
+    _applySearchFilter(searchBarController.text);
+  }
 }

lib/presentation/views/main_menu/match_view/create_match/create_match_view.dart

@@ -18,9 +18,13 @@ import 'package:tallee/presentation/widgets/player_selection.dart';
 import 'package:tallee/presentation/widgets/text_input/text_input_field.dart';
 import 'package:tallee/presentation/widgets/tiles/choose_tile.dart';
 
+/// A stateful widget for creating or editing a match.
 class CreateMatchView extends StatefulWidget {
-  /// A view that allows creating a new match
+  /// Constructor for `CreateMatchView`.
-  /// [onWinnerChanged]: Optional callback invoked when the winner is changed
+  ///
+  /// [onWinnerChanged] is an optional callback invoked when the winner is changed.
+  /// [matchToEdit] is an optional match to prefill the fields.
+  /// [onMatchUpdated] is an optional callback invoked when the match is updated.
   const CreateMatchView({
     super.key,
     this.onWinnerChanged,
@@ -28,47 +32,49 @@ class CreateMatchView extends StatefulWidget {
     this.onMatchUpdated,
   });
 
-  /// Optional callback invoked when the winner is changed
+  /// Optional callback invoked when the winner is changed.
   final VoidCallback? onWinnerChanged;
 
-  /// Optional callback invoked when the match is updated
+  /// Optional callback invoked when the match is updated.
   final void Function(Match)? onMatchUpdated;
 
-  /// An optional match to prefill the fields
+  /// An optional match to prefill the fields.
   final Match? matchToEdit;
 
   @override
   State<CreateMatchView> createState() => _CreateMatchViewState();
 }
 
+/// The state class for `CreateMatchView`, managing the UI and logic for creating or editing a match.
 class _CreateMatchViewState extends State<CreateMatchView> {
+  /// The database instance for accessing match data.
   late final AppDatabase db;
 
-  /// Controller for the match name input field
+  /// Controller for the match name input field.
   final TextEditingController _matchNameController = TextEditingController();
 
-  /// Hint text for the match name input field
+  /// Hint text for the match name input field.
   String? hintText;
 
-  /// List of all groups from the database
+  /// List of all games from the database.
   List<Game> gamesList = [];
 
-  /// List of all groups from the database
+  /// List of all groups from the database.
   List<Group> groupsList = [];
 
-  /// List of all players from the database
+  /// List of all players from the database.
   List<Player> playerList = [];
 
-  /// The currently selected group
+  /// The currently selected group.
   Group? selectedGroup;
 
-  /// The currently selected game
+  /// The currently selected game.
   Game? selectedGame;
 
-  /// The currently selected players
+  /// The currently selected players.
   List<Player> selectedPlayers = [];
 
-  /// GlobalKey for ScaffoldMessenger to show snackbars
+  /// GlobalKey for ScaffoldMessenger to show snackbars.
   final _scaffoldMessengerKey = GlobalKey<ScaffoldMessengerState>();
 
   @override
@@ -80,6 +86,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
 
     db = Provider.of<AppDatabase>(context, listen: false);
 
+    // Load games, groups, and players from the database.
     Future.wait([
       db.gameDao.getAllGames(),
       db.groupDao.getAllGroups(),
@@ -90,7 +97,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
       groupsList = result[1] as List<Group>;
       playerList = result[2] as List<Player>;
 
-      // If a match is provided, prefill the fields
+      // If a match is provided, prefill the fields.
       if (widget.matchToEdit != null) {
         prefillMatchDetails();
       }
@@ -130,6 +137,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
           child: Column(
             mainAxisAlignment: MainAxisAlignment.start,
             children: [
+              // Match name input field.
               Container(
                 margin: CustomTheme.tileMargin,
                 child: TextInputField(
@@ -138,6 +146,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
                   maxLength: Constants.MAX_MATCH_NAME_LENGTH,
                 ),
               ),
+              // Game selection tile.
               ChooseTile(
                 title: loc.game,
                 trailingText: selectedGame == null
@@ -172,15 +181,13 @@ class _CreateMatchViewState extends State<CreateMatchView> {
                   });
                 },
               ),
+              // Group selection tile.
               ChooseTile(
                 title: loc.group,
                 trailingText: selectedGroup == null
                     ? loc.none_group
                     : selectedGroup!.name,
                 onPressed: () async {
-                  // Remove all players from the previously selected group from
-                  // the selected players list, in case the user deselects the
-                  // group or selects a different group.
                   selectedPlayers.removeWhere(
                     (player) =>
                         selectedGroup?.members.any(
@@ -207,6 +214,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
                   });
                 },
               ),
+              // Player selection widget.
               Expanded(
                 child: PlayerSelection(
                   key: ValueKey(selectedGroup?.id ?? 'no_group'),
@@ -219,6 +227,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
                   },
                 ),
               ),
+              // Create or save button.
               CustomWidthButton(
                 text: buttonText,
                 sizeRelativeToWidth: 0.95,
@@ -240,16 +249,16 @@ class _CreateMatchViewState extends State<CreateMatchView> {
   ///
   /// Returns `true` if:
   /// - A ruleset is selected AND
-  /// - Either a group is selected OR at least 2 players are selected
+  /// - Either a group is selected OR at least 2 players are selected.
   bool _enableCreateGameButton() {
     return (selectedGroup != null ||
         (selectedPlayers.length > 1) && selectedGame != null);
   }
 
-  // If a match was provided to the view, it updates the match in the database
+  /// Handles navigation when the create or save button is pressed.
-  // and navigates back to the previous screen.
+  ///
-  // If no match was provided, it creates a new match in the database and
+  /// If a match is being edited, updates the match in the database.
-  // navigates to the MatchResultView for the newly created match.
+  /// Otherwise, creates a new match and navigates to the MatchResultView.
   void buttonNavigation(BuildContext context) async {
     if (widget.matchToEdit != null) {
       await updateMatch();
@@ -274,8 +283,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
     }
   }
 
-  /// Updates attributes of the existing match in the database based on the
+  /// Updates the existing match in the database.
-  /// changes made in the edit view.
   Future<void> updateMatch() async {
     final updatedMatch = Match(
       id: widget.matchToEdit!.id,
@@ -312,7 +320,6 @@ class _CreateMatchViewState extends State<CreateMatchView> {
       );
     }
 
-    // Add players who are in updatedMatch but not in the original match
     for (var player in updatedMatch.players) {
       if (!widget.matchToEdit!.players.any((p) => p.id == player.id)) {
         await db.playerMatchDao.addPlayerToMatch(
@@ -322,7 +329,6 @@ class _CreateMatchViewState extends State<CreateMatchView> {
       }
     }
 
-    // Remove players who are in the original match but not in updatedMatch
     for (var player in widget.matchToEdit!.players) {
       if (!updatedMatch.players.any((p) => p.id == player.id)) {
         await db.playerMatchDao.removePlayerFromMatch(
@@ -338,8 +344,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
     widget.onMatchUpdated?.call(updatedMatch);
   }
 
-  // Creates a new match and adds it to the database.
+  /// Creates a new match and adds it to the database.
-  // Returns the created match.
   Future<Match> createMatch() async {
     Match match = Match(
       name: _matchNameController.text.isEmpty
@@ -354,7 +359,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
     return match;
   }
 
-  // If a match was provided to the view, this method prefills the input fields
+  /// Prefills the input fields if a match was provided to the view.
   void prefillMatchDetails() {
     final match = widget.matchToEdit!;
     _matchNameController.text = match.name;
@@ -366,8 +371,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
     }
   }
 
-  // If none of the selected players are from the currently selected group,
+  /// Removes the group if none of its members are in the selected players list.
-  // the group is also deselected.
   Future<void> removeGroupWhenNoMemberLeft() async {
     if (selectedGroup == null) return;
 
@@ -381,7 +385,7 @@ class _CreateMatchViewState extends State<CreateMatchView> {
     }
   }
 
-  // Loads all games from the database and updates the gamesList.
+  /// Loads all games from the database and updates the state.
   Future<void> loadGames() async {
     final result = await db.gameDao.getAllGames();
     result.sort((a, b) => b.createdAt.compareTo(a.createdAt));

lib/presentation/views/main_menu/match_view/create_match/game_view/create_game_view.dart

@@ -7,13 +7,18 @@ import 'package:tallee/core/custom_theme.dart';
 import 'package:tallee/core/enums.dart';
 import 'package:tallee/data/db/database.dart';
 import 'package:tallee/data/dto/game.dart';
+import 'package:tallee/data/dto/group.dart';
 import 'package:tallee/l10n/generated/app_localizations.dart';
 import 'package:tallee/presentation/views/main_menu/match_view/create_match/game_view/choose_color_view.dart';
 import 'package:tallee/presentation/views/main_menu/match_view/create_match/game_view/choose_ruleset_view.dart';
 import 'package:tallee/presentation/widgets/buttons/custom_width_button.dart';
+import 'package:tallee/presentation/widgets/custom_alert_dialog.dart';
 import 'package:tallee/presentation/widgets/text_input/text_input_field.dart';
 import 'package:tallee/presentation/widgets/tiles/choose_tile.dart';
 
+/// A stateful widget for creating or editing a game.
+/// - [gameToEdit] An optional game to prefill the fields
+/// - [onGameCreatedOrEdited] Callback to invoke when the game is created or edited
 class CreateGameView extends StatefulWidget {
   const CreateGameView({
     super.key,
@@ -32,16 +37,39 @@ class CreateGameView extends StatefulWidget {
 }
 
 class _CreateGameViewState extends State<CreateGameView> {
+  /// GlobalKey for ScaffoldMessenger to show snackbars
+  final _scaffoldMessengerKey = GlobalKey<ScaffoldMessengerState>();
+
+  /// The database instance for accessing game data.
   late final AppDatabase db;
+
+  /// The currently selected ruleset for the game.
   Ruleset? selectedRuleset;
+
+  /// The index of the currently selected ruleset.
   int selectedRulesetIndex = -1;
+
+  /// A list of available rulesets and their localized names.
   late List<(Ruleset, String)> _rulesets;
 
+  /// The currently selected color for the game.
   GameColor? selectedColor;
 
+  /// Controller for the game name input field.
   final _gameNameController = TextEditingController();
+
+  /// Controller for the game description input field.
   final _descriptionController = TextEditingController();
 
+  /// The ID of the currently selected group.
+  late String selectedGroupId;
+
+  /// A controller for the search bar input field.
+  final TextEditingController controller = TextEditingController();
+
+  /// A list of groups filtered based on the search query.
+  late final List<Group> filteredGroups;
+
   @override
   void initState() {
     super.initState();
@@ -104,6 +132,51 @@ class _CreateGameViewState extends State<CreateGameView> {
         backgroundColor: CustomTheme.backgroundColor,
         appBar: AppBar(
           title: Text(isEditing ? loc.edit_game : loc.create_game),
+          actions: widget.gameToEdit == null
+              ? []
+              : [
+                  IconButton(
+                    icon: const Icon(Icons.delete),
+                    onPressed: () async {
+                      if (widget.gameToEdit != null) {
+                        showDialog<bool>(
+                          context: context,
+                          builder: (context) => CustomAlertDialog(
+                            title: loc.delete_game,
+                            content: loc.this_cannot_be_undone,
+                            actions: [
+                              TextButton(
+                                onPressed: () =>
+                                    Navigator.of(context).pop(false),
+                                child: Text(loc.cancel),
+                              ),
+                              TextButton(
+                                onPressed: () =>
+                                    Navigator.of(context).pop(true),
+                                child: Text(loc.delete),
+                              ),
+                            ],
+                          ),
+                        ).then((confirmed) async {
+                          if (confirmed == true && context.mounted) {
+                            bool success = await db.gameDao.deleteGame(
+                              gameId: widget.gameToEdit!.id,
+                            );
+                            if (!context.mounted) return;
+                            if (success) {
+                              Navigator.of(
+                                context,
+                              ).pop((game: widget.gameToEdit, delete: true));
+                            } else {
+                              if (!mounted) return;
+                              showSnackbar(message: loc.error_deleting_game);
+                            }
+                          }
+                        });
+                      }
+                    },
+                  ),
+                ],
         ),
         body: SafeArea(
           child: Column(
@@ -196,7 +269,9 @@ class _CreateGameViewState extends State<CreateGameView> {
                           }
                           widget.onGameCreatedOrEdited.call();
                           if (context.mounted) {
-                            Navigator.of(context).pop(newGame);
+                            Navigator.of(
+                              context,
+                            ).pop((game: newGame, delete: false));
                           }
                         }
                       : null,
@@ -209,6 +284,9 @@ class _CreateGameViewState extends State<CreateGameView> {
     );
   }
 
+  /// Handles updating an existing game in the database.
+  ///
+  /// [newGame] The updated game object.
   Future<void> handleGameUpdate(Game newGame) async {
     final oldGame = widget.gameToEdit!;
 
@@ -248,7 +326,26 @@ class _CreateGameViewState extends State<CreateGameView> {
     }
   }
 
+  /// Handles creating a new game in the database.
+  ///
+  /// [newGame] The game object to be created.
   Future<void> handleGameCreation(Game newGame) async {
     await db.gameDao.addGame(game: newGame);
   }
+
+  /// Displays a snackbar with the given message and optional action.
+  ///
+  /// [message] The message to display in the snackbar.
+  void showSnackbar({required String message}) {
+    final messenger = _scaffoldMessengerKey.currentState;
+    if (messenger != null) {
+      messenger.hideCurrentSnackBar();
+      messenger.showSnackBar(
+        SnackBar(
+          content: Text(message, style: const TextStyle(color: Colors.white)),
+          backgroundColor: CustomTheme.boxColor,
+        ),
+      );
+    }
+  }
 }

lib/presentation/views/main_menu/match_view/match_view.dart

@@ -19,7 +19,7 @@ import 'package:tallee/presentation/widgets/tiles/match_tile.dart';
 import 'package:tallee/presentation/widgets/top_centered_message.dart';
 
 class MatchView extends StatefulWidget {
-  /// A view that displays a list of matches
+  /// A view that displays a list of matches.
   const MatchView({super.key});
 
   @override
@@ -27,11 +27,14 @@ class MatchView extends StatefulWidget {
 }
 
 class _MatchViewState extends State<MatchView> {
+  /// Database instance used to access match data.
   late final AppDatabase db;
+
+  /// Indicates whether matches are currently being loaded.
   bool isLoading = true;
 
   /// Loaded matches from the database,
-  /// initially filled with skeleton matches
+  /// initially filled with skeleton matches.
   List<Match> matches = List.filled(
     4,
     Match(