Camelot: Execute copying in background and add a result message

Change-Id: I9cc9922a2c501ee610bf23039cc3efc411906adb

Author: SebaUbuntu

app/src/main/java/org/lineageos/camelot/PdfViewerActivity.kt

@@ -18,6 +18,7 @@ import androidx.activity.enableEdgeToEdge
 import androidx.activity.result.contract.ActivityResultContracts
 import androidx.activity.viewModels
 import androidx.annotation.RequiresExtension
+import androidx.annotation.StringRes
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.util.Consumer
 import androidx.core.view.WindowInsetsCompat
@@ -26,6 +27,7 @@ import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
 import androidx.lifecycle.repeatOnLifecycle
 import com.google.android.material.appbar.MaterialToolbar
+import com.google.android.material.snackbar.Snackbar
 import kotlinx.coroutines.coroutineScope
 import kotlinx.coroutines.flow.collectLatest
 import kotlinx.coroutines.launch
@@ -74,7 +76,16 @@ class PdfViewerActivity : AppCompatActivity(R.layout.activity_main) {
     private val documentLauncher = registerForActivityResult(
         ActivityResultContracts.CreateDocument(MIME_TYPE_PDF)
     ) { uri ->
-        pdfViewModel.copyPdf(pdfUri, uri)
+        lifecycleScope.launch {
+            val result = pdfViewModel.copyPdf(pdfUri, uri)
+
+            showSnackbar(
+                when (result) {
+                    true -> R.string.download_successful
+                    false -> R.string.download_error
+                }
+            )
+        }
     }
 
     override fun onCreate(savedInstanceState: Bundle?) {
@@ -208,6 +219,17 @@ class PdfViewerActivity : AppCompatActivity(R.layout.activity_main) {
         }
     }
 
+    private fun showSnackbar(@StringRes messageStringResId: Int) {
+        Snackbar.make(
+            this,
+            pdfViewerFragment.requireView(),
+            getString(messageStringResId),
+            Snackbar.LENGTH_LONG
+        )
+            .setAction(android.R.string.ok) {}
+            .show()
+    }
+
     companion object {
         private const val MIME_TYPE_PDF = "application/pdf"
     }

app/src/main/java/org/lineageos/camelot/viewmodels/PdfViewModel.kt

@@ -8,8 +8,10 @@ package org.lineageos.camelot.viewmodels
 import android.app.Application
 import android.net.Uri
 import androidx.lifecycle.AndroidViewModel
+import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.asStateFlow
+import kotlinx.coroutines.withContext
 
 class PdfViewModel(application: Application) : AndroidViewModel(application) {
     private val _pdfName = MutableStateFlow<String?>(null)
@@ -33,16 +35,19 @@ class PdfViewModel(application: Application) : AndroidViewModel(application) {
         _toolbarHeight.value = toolbarHeight
     }
 
-    fun copyPdf(sourceUri: Uri?, destinationUri: Uri?) {
+    suspend fun copyPdf(sourceUri: Uri?, destinationUri: Uri?) = withContext(Dispatchers.IO) {
         val contentResolver = getApplication<Application>().contentResolver
 
-        val source = sourceUri ?: return
-        val destination = destinationUri ?: return
+        val source = sourceUri ?: return@withContext false
+        val destination = destinationUri ?: return@withContext false
 
-        contentResolver.openInputStream(source)?.use { input ->
-            contentResolver.openOutputStream(destination)?.use { output ->
-                input.copyTo(output)
+        runCatching {
+            contentResolver.openInputStream(source)?.use { input ->
+                contentResolver.openOutputStream(destination)?.use { output ->
+                    input.copyTo(output)
+                    true
+                }
             }
-        }
+        }.getOrNull() ?: false
     }
 }

app/src/main/res/values/strings.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-     SPDX-FileCopyrightText: 2024 The LineageOS Project
+     SPDX-FileCopyrightText: 2024-2025 The LineageOS Project
      SPDX-License-Identifier: Apache-2.0
 -->
 <resources>
@@ -11,4 +11,8 @@
     <string name="download">Download</string>
     <string name="pdf_document">PDF Document</string>
     <string name="print">Print</string>
+
+    <!-- Download results -->
+    <string name="download_successful">Download successful</string>
+    <string name="download_error">Error while downloading the file</string>
 </resources>