From ae1b316aad4dc56eac334ffba49185cc784df15a Mon Sep 17 00:00:00 2001
From: Dr-Blank <64108942+Dr-Blank@users.noreply.github.com>
Date: Sun, 22 Sep 2024 22:04:36 -0400
Subject: [PATCH] refactor: simplify parameters in queueAudioBookDownload and
 improve logging message in deleteDownloadedItem

---
 lib/features/downloads/core/download_manager.dart | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/lib/features/downloads/core/download_manager.dart b/lib/features/downloads/core/download_manager.dart
index 77deff8..5b6229d 100644
--- a/lib/features/downloads/core/download_manager.dart
+++ b/lib/features/downloads/core/download_manager.dart
@@ -65,11 +65,8 @@ class AudiobookDownloadManager {
   late StreamSubscription<TaskUpdate> _updatesSubscription;
 
   Future<void> queueAudioBookDownload(
-    LibraryItemExpanded item, {
-    void Function(TaskStatusUpdate)? taskStatusCallback,
-    void Function(TaskProgressUpdate)? taskProgressCallback,
-    void Function(Task, NotificationType)? taskNotificationTapCallback,
-  }) async {
+    LibraryItemExpanded item,
+  ) async {
     _logger.info('queuing download for item: ${item.id}');
     // create a download task for each file in the item
     final directory = await getApplicationSupportDirectory();
@@ -119,8 +116,7 @@ class AudiobookDownloadManager {
   }
 
   bool isFileDownloaded(String filePath) {
-    final fileExists = File(filePath).existsSync();
-    return fileExists;
+    return File(filePath).existsSync();
   }
 
   Future<List<LibraryFile>> getDownloadedFilesMetadata(
@@ -159,7 +155,7 @@ class AudiobookDownloadManager {
   }
 
   Future<void> deleteDownloadedItem(LibraryItemExpanded item) async {
-    _logger.info('deleting downloaded item id: ${item.id}');
+    _logger.info('deleting downloaded item with id: ${item.id}');
     final directory = await getApplicationSupportDirectory();
     for (final file in item.libraryFiles) {
       final filePath = constructFilePath(directory, item, file);