From f005b31eaf5251af3b14c5bd98dd2166274ae87a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micka=C3=ABl=20Menu?= <mickael.menu@gmail.com>
Date: Fri, 19 Jan 2024 13:07:06 +0100
Subject: [PATCH] Minor fixes

---
 .../org/readium/r2/navigator/VisualNavigator.kt     |  6 +++---
 .../r2/navigator/media/MediaSessionNavigator.kt     |  4 ++--
 .../foreground/ForegroundDownloadManager.kt         | 13 +++++++++++--
 3 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/readium/navigator/src/main/java/org/readium/r2/navigator/VisualNavigator.kt b/readium/navigator/src/main/java/org/readium/r2/navigator/VisualNavigator.kt
index 94784ec412..18c2796251 100644
--- a/readium/navigator/src/main/java/org/readium/r2/navigator/VisualNavigator.kt
+++ b/readium/navigator/src/main/java/org/readium/r2/navigator/VisualNavigator.kt
@@ -24,7 +24,7 @@ import org.readium.r2.shared.publication.ReadingProgression as PublicationReadin
 public interface VisualNavigator : Navigator {
 
     @Deprecated(
-        "Moved to DirectionalNavigator",
+        "Renamed to OverflowableNavigator.Overflow",
         level = DeprecationLevel.ERROR
     )
     @OptIn(ExperimentalReadiumApi::class)
@@ -126,7 +126,7 @@ public interface VisualNavigator : Navigator {
      * Moves to the next content portion (eg. page) in the reading progression direction.
      */
     @Deprecated(
-        "Moved to DirectionalNavigator",
+        "Moved to OverflowableNavigator",
         level = DeprecationLevel.ERROR
     )
     public fun goForward(animated: Boolean = false, completion: () -> Unit = {}): Boolean
@@ -135,7 +135,7 @@ public interface VisualNavigator : Navigator {
      * Moves to the previous content portion (eg. page) in the reading progression direction.
      */
     @Deprecated(
-        "Moved to DirectionalNavigator",
+        "Moved to OverflowableNavigator",
         level = DeprecationLevel.ERROR
     )
     public fun goBackward(animated: Boolean = false, completion: () -> Unit = {}): Boolean
diff --git a/readium/navigator/src/main/java/org/readium/r2/navigator/media/MediaSessionNavigator.kt b/readium/navigator/src/main/java/org/readium/r2/navigator/media/MediaSessionNavigator.kt
index c150d34e12..11cd4d0b5d 100644
--- a/readium/navigator/src/main/java/org/readium/r2/navigator/media/MediaSessionNavigator.kt
+++ b/readium/navigator/src/main/java/org/readium/r2/navigator/media/MediaSessionNavigator.kt
@@ -208,7 +208,7 @@ public class MediaSessionNavigator(
         return go(locator, animated, completion)
     }
 
-    public fun goForward(animated: Boolean, completion: () -> Unit): Boolean {
+    public fun goForward(animated: Boolean = true, completion: () -> Unit = {}): Boolean {
         if (!isActive) return false
 
         seekRelative(skipForwardInterval)
@@ -216,7 +216,7 @@ public class MediaSessionNavigator(
         return true
     }
 
-    public fun goBackward(animated: Boolean, completion: () -> Unit): Boolean {
+    public fun goBackward(animated: Boolean = true, completion: () -> Unit = {}): Boolean {
         if (!isActive) return false
 
         seekRelative(-skipBackwardInterval)
diff --git a/readium/shared/src/main/java/org/readium/r2/shared/util/downloads/foreground/ForegroundDownloadManager.kt b/readium/shared/src/main/java/org/readium/r2/shared/util/downloads/foreground/ForegroundDownloadManager.kt
index dd34f94f7d..a6b065e9a8 100644
--- a/readium/shared/src/main/java/org/readium/r2/shared/util/downloads/foreground/ForegroundDownloadManager.kt
+++ b/readium/shared/src/main/java/org/readium/r2/shared/util/downloads/foreground/ForegroundDownloadManager.kt
@@ -57,8 +57,17 @@ public class ForegroundDownloadManager(
     }
 
     private suspend fun doRequest(request: DownloadManager.Request, id: DownloadManager.RequestId) {
-        val destination = withContext(Dispatchers.IO) {
-            File.createTempFile(UUID.randomUUID().toString(), null, downloadsDirectory)
+        val destination: File
+        try {
+            destination = withContext(Dispatchers.IO) {
+                File.createTempFile(UUID.randomUUID().toString(), null, downloadsDirectory)
+            }
+        } catch (exception: IOException) {
+            val error = DownloadManager.DownloadError.FileSystem(FileSystemError.IO(exception))
+            forEachListener(id) {
+                onDownloadFailed(id, error)
+            }
+            return
         }
 
         httpClient