refactor: clean up code formatting and improve dialog handling in playlist features

Author: gokadzev

lib/screens/playlist_page.dart

@@ -372,7 +372,7 @@ class _PlaylistPageState extends State<PlaylistPage> {
       onPressed: _handleSyncPlaylist,
     );
   }
-  
+
   Widget _buildAddToPlaylistButton() {
     return IconButton.filledTonal(
       icon: const Icon(FluentIcons.album_add_24_regular),

lib/services/playlists_manager.dart

@@ -213,7 +213,7 @@ String addSongsInCustomPlaylist(
       );
       return context.l10n!.addedSuccess;
     } else {
-      return context.l10n!.songAlreadyInPlaylist; // Or some message saying all songs already exist
+      return context.l10n!.songAlreadyInPlaylist;
     }
   } else {
     logger.log('Custom playlist not found for ytid: $playlistId');

lib/utilities/playlist_dialogs.dart

@@ -26,8 +26,11 @@ import 'package:musify/services/playlists_manager.dart';
 import 'package:musify/utilities/flutter_toast.dart';
 import 'package:musify/utilities/playlist_image_picker.dart';
 
-void showCreatePlaylistDialog(BuildContext context,
-    {dynamic songToAdd, List<dynamic>? songsToAdd}) {
+void showCreatePlaylistDialog(
+  BuildContext context, {
+  dynamic songToAdd,
+  List<dynamic>? songsToAdd,
+}) {
   var id = '';
   var customPlaylistName = '';
   var isYouTubeMode = songToAdd == null && songsToAdd == null;
@@ -381,12 +384,10 @@ void showAddToPlaylistDialog(
                 ),
         ),
         actionsAlignment: MainAxisAlignment.end,
-        actions: <Widget>[
+        actions: [
           TextButton(
             child: Text(context.l10n!.cancel),
-            onPressed: () {
-              Navigator.of(context).pop();
-            },
+            onPressed: () => Navigator.pop(context),
           ),
           FilledButton.icon(
             onPressed: () {

lib/widgets/playlist_bar.dart

@@ -195,7 +195,7 @@ class PlaylistBar extends StatelessWidget {
             _handleEdit(context);
             break;
           case 'add_to_playlist':
-            _handleAddPlaylistToPlaylist(context, colorScheme);
+            _handleAddPlaylistToPlaylist(context);
             break;
         }
       },
@@ -422,56 +422,42 @@ class PlaylistBar extends StatelessWidget {
     }
   }
 
-  Future<void> _handleAddPlaylistToPlaylist(
-    BuildContext context,
-    ColorScheme colorScheme,
-  ) async {
+  Future<void> _handleAddPlaylistToPlaylist(BuildContext context) async {
     if (_resolvedPlaylistId == null) {
       showToast(context, context.l10n!.error);
       return;
     }
 
-    final currentContext = NavigationManager().context;
+    final navContext = NavigationManager().context;
     unawaited(
       showDialog(
-        context: currentContext,
+        context: navContext,
         barrierDismissible: false,
-        builder: (BuildContext context) {
-          return const Center(
-            child: Spinner(),
-          );
-        },
+        builder: (_) => const Center(child: Spinner()),
       ),
     );
 
     try {
-      final fullPlaylist = await getPlaylistInfoForWidget(
-        _resolvedPlaylistId!,
-      );
+      final fullPlaylist = await getPlaylistInfoForWidget(_resolvedPlaylistId);
+      if (!navContext.mounted) return;
+      Navigator.pop(navContext);
 
-      final appCtx = NavigationManager().context;
+      if (fullPlaylist == null || fullPlaylist['list'] == null) {
+        showToast(navContext, navContext.l10n!.error);
+        return;
+      }
 
-      if (appCtx.mounted) {
-        Navigator.pop(appCtx); // close spinner
-        if (fullPlaylist != null && fullPlaylist['list'] != null) {
-          final List<dynamic> tracks = fullPlaylist['list'];
-          if (tracks.isEmpty) {
-            showToast(appCtx, appCtx.l10n!.noSongsInPlaylist);
-            return;
-          }
-          unawaited(
-            Future.microtask(
-                () => showAddToPlaylistDialog(appCtx, songs: tracks)),
-          );
-        } else {
-          showToast(appCtx, appCtx.l10n!.error);
-        }
+      final tracks = fullPlaylist['list'] as List<dynamic>;
+      if (tracks.isEmpty) {
+        showToast(navContext, navContext.l10n!.noSongsInPlaylist);
+        return;
       }
+
+      showAddToPlaylistDialog(navContext, songs: tracks);
     } catch (e) {
-      final appCtx = NavigationManager().context;
-      if (appCtx.mounted) {
-        Navigator.pop(appCtx); // close spinner
-        showToast(appCtx, appCtx.l10n!.error);
+      if (navContext.mounted) {
+        Navigator.pop(navContext);
+        showToast(navContext, navContext.l10n!.error);
       }
     }
   }