Merge pull request 'Spieler erstellen in CreateGroupView' (#43) from feature/42-spieler-erstellen-in-create-group-view-implementieren into development

Reviewed-on: #43
Reviewed-by: Felix Kirchner <felix.kirchner.fk@gmail.com>

lib/presentation/views/main_menu/create_group_view.dart

@@ -37,6 +37,24 @@ class _CreateGroupViewState extends State<CreateGroupView> {
   void initState() {
     super.initState();
     db = Provider.of<AppDatabase>(context, listen: false);
+    _searchBarController.addListener(() {
+      setState(() {});
+    });
+    _groupNameController.addListener(() {
+      setState(() {});
+    });
+    loadPlayerList();
+  }
+
+  @override
+  void dispose() {
+    _groupNameController.dispose();
+    _searchBarController
+        .dispose(); // Listener entfernen und Controller aufräumen
+    super.dispose();
+  }
+
+  void loadPlayerList() {
     _allPlayersFuture = db.playerDao.getAllPlayers();
     _allPlayersFuture.then((loadedPlayers) {
       setState(() {
@@ -99,6 +117,19 @@ class _CreateGroupViewState extends State<CreateGroupView> {
                         minHeight: 45,
                       ),
                       hintText: 'Search for players',
+                      trailingButtonShown: true,
+                      trailingButtonicon: Icons.add_circle,
+                      trailingButtonEnabled: _searchBarController.text
+                          .trim()
+                          .isNotEmpty,
+                      onTrailingButtonPressed: () async {
+                        addNewPlayerFromSearch(
+                          context: context,
+                          searchBarController: _searchBarController,
+                          db: db,
+                          loadPlayerList: loadPlayerList,
+                        );
+                      },
                       onChanged: (value) {
                         setState(() {
                           if (value.isEmpty) {
@@ -216,25 +247,16 @@ class _CreateGroupViewState extends State<CreateGroupView> {
                                       layoutBuilder:
                                           AnimatedSwitcher.defaultLayoutBuilder,
                                     ),
-                                child:
+                                child: Visibility(
+                                  visible:
                                       (suggestedPlayers.isEmpty &&
-                                        allPlayers.isNotEmpty)
+                                      allPlayers.isNotEmpty),
-                                    ? TopCenteredMessage(
+                                  replacement: ListView.builder(
-                                        icon: Icons.info,
-                                        title: 'Info',
-                                        message:
-                                            (selectedPlayers.length ==
-                                                allPlayers.length)
-                                            ? 'No more players to add.'
-                                            : 'No players found with that name.',
-                                      )
-                                    : ListView.builder(
                                     itemCount: suggestedPlayers.length,
                                     itemBuilder:
                                         (BuildContext context, int index) {
                                           return TextIconListTile(
-                                                text: suggestedPlayers[index]
+                                            text: suggestedPlayers[index].name,
-                                                    .name,
                                             onPressed: () {
                                               setState(() {
                                                 if (!selectedPlayers.contains(
@@ -244,8 +266,9 @@ class _CreateGroupViewState extends State<CreateGroupView> {
                                                     suggestedPlayers[index],
                                                   );
                                                   selectedPlayers.sort(
-                                                        (a, b) => a.name
+                                                    (a, b) => a.name.compareTo(
-                                                            .compareTo(b.name),
+                                                      b.name,
+                                                    ),
                                                   );
                                                   suggestedPlayers.remove(
                                                     suggestedPlayers[index],
@@ -256,6 +279,16 @@ class _CreateGroupViewState extends State<CreateGroupView> {
                                           );
                                         },
                                   ),
+                                  child: TopCenteredMessage(
+                                    icon: Icons.info,
+                                    title: 'Info',
+                                    message:
+                                        (selectedPlayers.length ==
+                                            allPlayers.length)
+                                        ? 'No more players to add.'
+                                        : 'No players found with that name.',
+                                  ),
+                                ),
                               ),
                             );
                           },
@@ -274,7 +307,7 @@ class _CreateGroupViewState extends State<CreateGroupView> {
                   : () async {
                       bool success = await db.groupDao.addGroup(
                         group: Group(
-                          name: _groupNameController.text,
+                          name: _groupNameController.text.trim(),
                           members: selectedPlayers,
                         ),
                       );
@@ -307,3 +340,47 @@ class _CreateGroupViewState extends State<CreateGroupView> {
     );
   }
 }
+
+/// Adds a new player to the database from the search bar input.
+/// Shows a snackbar indicating success or failure.
+/// [context] - BuildContext to show the snackbar.
+/// [searchBarController] - TextEditingController of the search bar.
+/// [db] - AppDatabase instance to interact with the database.
+/// [loadPlayerList] - Function to reload the player list after adding.
+void addNewPlayerFromSearch({
+  required BuildContext context,
+  required TextEditingController searchBarController,
+  required AppDatabase db,
+  required Function loadPlayerList,
+}) async {
+  String playerName = searchBarController.text.trim();
+  bool success = await db.playerDao.addPlayer(player: Player(name: playerName));
+  if (!context.mounted) return;
+  if (success) {
+    loadPlayerList();
+    ScaffoldMessenger.of(context).showSnackBar(
+      SnackBar(
+        backgroundColor: CustomTheme.boxColor,
+        content: Center(
+          child: Text(
+            'Successfully added player $playerName.',
+            style: const TextStyle(color: Colors.white),
+          ),
+        ),
+      ),
+    );
+    searchBarController.clear();
+  } else {
+    ScaffoldMessenger.of(context).showSnackBar(
+      SnackBar(
+        backgroundColor: CustomTheme.boxColor,
+        content: Center(
+          child: Text(
+            'Could not add player $playerName.',
+            style: const TextStyle(color: Colors.white),
+          ),
+        ),
+      ),
+    );
+  }
+}

lib/presentation/widgets/custom_search_bar.dart

@@ -6,11 +6,19 @@ class CustomSearchBar extends StatelessWidget {
   final String hintText;
   final ValueChanged<String>? onChanged;
   final BoxConstraints? constraints;
+  final bool trailingButtonShown;
+  final bool trailingButtonEnabled;
+  final VoidCallback? onTrailingButtonPressed;
+  final IconData trailingButtonicon;
 
   const CustomSearchBar({
     super.key,
     required this.controller,
     required this.hintText,
+    this.trailingButtonShown = false,
+    this.trailingButtonicon = Icons.clear,
+    this.trailingButtonEnabled = true,
+    this.onTrailingButtonPressed,
     this.onChanged,
     this.constraints,
   });
@@ -22,9 +30,24 @@ class CustomSearchBar extends StatelessWidget {
       constraints:
           constraints ?? const BoxConstraints(maxHeight: 45, minHeight: 45),
       hintText: hintText,
-      onChanged: onChanged,
+      onChanged: trailingButtonEnabled ? onChanged : null,
       hintStyle: WidgetStateProperty.all(const TextStyle(fontSize: 16)),
       leading: const Icon(Icons.search),
+      trailing: [
+        Visibility(
+          visible: trailingButtonShown,
+          child: GestureDetector(
+            onTap: trailingButtonEnabled ? onTrailingButtonPressed : null,
+            child: Icon(
+              trailingButtonicon,
+              color: trailingButtonEnabled
+                  ? null
+                  : Colors.grey.withValues(alpha: 0.2),
+            ),
+          ),
+        ),
+        const SizedBox(width: 5),
+      ],
       backgroundColor: WidgetStateProperty.all(CustomTheme.boxColor),
       side: WidgetStateProperty.all(BorderSide(color: CustomTheme.boxBorder)),
       shape: WidgetStateProperty.all(