From 930539ca483a9fbedd40a241ee133e28a9076a94 Mon Sep 17 00:00:00 2001
From: Kingkor Roy Tirtho <krtirtho@gmail.com>
Date: Mon, 15 Apr 2024 19:47:15 +0600
Subject: [PATCH] chore: fix analyzer issues

---
 lib/components/desktop_login/login_form.dart       | 3 +--
 lib/components/shared/waypoint.dart                | 8 +++-----
 lib/hooks/configurators/use_get_storage_perms.dart | 6 +++---
 lib/hooks/utils/use_palette_color.dart             | 7 +++----
 lib/pages/root/root_app.dart                       | 3 +--
 5 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/lib/components/desktop_login/login_form.dart b/lib/components/desktop_login/login_form.dart
index 2949fbae..6091829c 100644
--- a/lib/components/desktop_login/login_form.dart
+++ b/lib/components/desktop_login/login_form.dart
@@ -16,7 +16,6 @@ class TokenLoginForm extends HookConsumerWidget {
   Widget build(BuildContext context, ref) {
     final authenticationNotifier = ref.watch(authenticationProvider.notifier);
     final directCodeController = useTextEditingController();
-    final mounted = useIsMounted();
 
     final isLoading = useState(false);
 
@@ -57,7 +56,7 @@ class TokenLoginForm extends HookConsumerWidget {
                         await AuthenticationCredentials.fromCookie(
                             cookieHeader),
                       );
-                      if (mounted()) {
+                      if (context.mounted) {
                         onDone?.call();
                       }
                     } finally {
diff --git a/lib/components/shared/waypoint.dart b/lib/components/shared/waypoint.dart
index 08e9088a..cf00e29b 100644
--- a/lib/components/shared/waypoint.dart
+++ b/lib/components/shared/waypoint.dart
@@ -20,8 +20,6 @@ class Waypoint extends HookWidget {
 
   @override
   Widget build(BuildContext context) {
-    final isMounted = useIsMounted();
-
     useEffect(() {
       if (isGrid) {
         return null;
@@ -32,19 +30,19 @@ class Waypoint extends HookWidget {
 
         // scrollController fetches the next paginated data when the current
         // position of the user on the screen has surpassed
-        if (controller.position.pixels >= nextPageTrigger && isMounted()) {
+        if (controller.position.pixels >= nextPageTrigger && context.mounted) {
           await onTouchEdge?.call();
         }
       }
 
       WidgetsBinding.instance.addPostFrameCallback((_) {
-        if (controller.hasClients && isMounted()) {
+        if (controller.hasClients && context.mounted) {
           listener();
           controller.addListener(listener);
         }
       });
       return () => controller.removeListener(listener);
-    }, [controller, onTouchEdge, isMounted]);
+    }, [controller, onTouchEdge]);
 
     if (isGrid) {
       return VisibilityDetector(
diff --git a/lib/hooks/configurators/use_get_storage_perms.dart b/lib/hooks/configurators/use_get_storage_perms.dart
index 86b495c4..db51af14 100644
--- a/lib/hooks/configurators/use_get_storage_perms.dart
+++ b/lib/hooks/configurators/use_get_storage_perms.dart
@@ -7,7 +7,7 @@ import 'package:spotube/components/library/user_local_tracks.dart';
 import 'package:spotube/hooks/utils/use_async_effect.dart';
 
 void useGetStoragePermissions(WidgetRef ref) {
-  final isMounted = useIsMounted();
+  final context = useContext();
 
   useAsyncEffect(
     () async {
@@ -25,11 +25,11 @@ void useGetStoragePermissions(WidgetRef ref) {
 
       if (hasNoStoragePerm) {
         await Permission.storage.request();
-        if (isMounted()) ref.invalidate(localTracksProvider);
+        if (context.mounted) ref.invalidate(localTracksProvider);
       }
       if (hasNoAudioPerm) {
         await Permission.audio.request();
-        if (isMounted()) ref.invalidate(localTracksProvider);
+        if (context.mounted) ref.invalidate(localTracksProvider);
       }
     },
     null,
diff --git a/lib/hooks/utils/use_palette_color.dart b/lib/hooks/utils/use_palette_color.dart
index 9269edd7..e6d8b398 100644
--- a/lib/hooks/utils/use_palette_color.dart
+++ b/lib/hooks/utils/use_palette_color.dart
@@ -14,7 +14,6 @@ PaletteColor usePaletteColor(String imageUrl, WidgetRef ref) {
   final context = useContext();
   final theme = Theme.of(context);
   final paletteColor = ref.watch(_paletteColorState);
-  final mounted = useIsMounted();
 
   useEffect(() {
     WidgetsBinding.instance.addPostFrameCallback((timeStamp) async {
@@ -25,7 +24,7 @@ PaletteColor usePaletteColor(String imageUrl, WidgetRef ref) {
           width: 50,
         ),
       );
-      if (!mounted()) return;
+      if (!context.mounted) return;
       final color = theme.brightness == Brightness.light
           ? palette.lightMutedColor ?? palette.lightVibrantColor
           : palette.darkMutedColor ?? palette.darkVibrantColor;
@@ -41,7 +40,7 @@ PaletteColor usePaletteColor(String imageUrl, WidgetRef ref) {
 
 PaletteGenerator usePaletteGenerator(String imageUrl) {
   final palette = useState(PaletteGenerator.fromColors([]));
-  final mounted = useIsMounted();
+  final context = useContext();
 
   useEffect(() {
     WidgetsBinding.instance.addPostFrameCallback((timeStamp) async {
@@ -52,7 +51,7 @@ PaletteGenerator usePaletteGenerator(String imageUrl) {
           width: 50,
         ),
       );
-      if (!mounted()) return;
+      if (!context.mounted) return;
 
       palette.value = newPalette;
     });
diff --git a/lib/pages/root/root_app.dart b/lib/pages/root/root_app.dart
index 56ea43a6..5ac0689a 100644
--- a/lib/pages/root/root_app.dart
+++ b/lib/pages/root/root_app.dart
@@ -38,7 +38,6 @@ class RootApp extends HookConsumerWidget {
 
   @override
   Widget build(BuildContext context, ref) {
-    final isMounted = useIsMounted();
     final showingDialogCompleter = useRef(Completer()..complete());
     final downloader = ref.watch(downloadManagerProvider);
     final scaffoldMessenger = ScaffoldMessenger.of(context);
@@ -129,7 +128,7 @@ class RootApp extends HookConsumerWidget {
 
     useEffect(() {
       downloader.onFileExists = (track) async {
-        if (!isMounted()) return false;
+        if (!context.mounted) return false;
 
         if (!showingDialogCompleter.value.isCompleted) {
           await showingDialogCompleter.value.future;