From b3b6c06a0e2a84fbc0c1c58e5c8387ec1b2f72ac Mon Sep 17 00:00:00 2001
From: MohitMaliFtechiz <futuretechoworld.mohitmali@gmail.com>
Date: Wed, 27 Dec 2023 15:20:12 +0530
Subject: [PATCH] Open file with `AssetFileDescriptor` instead of `File` since
 we have restriction to directly access the files via filePath so instead of a
 file now we are acquiring the fileDescriptor approach since we can directly
 open ZIM files via file descriptor without any special permission. Therefore,
 we are now starting to utilize the fileDescriptor instead of a file.

---
 .../destination/reader/KiwixReaderFragment.kt | 22 ++++++++++---------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/org/kiwix/kiwixmobile/nav/destination/reader/KiwixReaderFragment.kt b/app/src/main/java/org/kiwix/kiwixmobile/nav/destination/reader/KiwixReaderFragment.kt
index dac02f2de..de11b6810 100644
--- a/app/src/main/java/org/kiwix/kiwixmobile/nav/destination/reader/KiwixReaderFragment.kt
+++ b/app/src/main/java/org/kiwix/kiwixmobile/nav/destination/reader/KiwixReaderFragment.kt
@@ -19,7 +19,6 @@
 package org.kiwix.kiwixmobile.nav.destination.reader
 
 import android.content.Intent
-import android.net.Uri
 import android.os.Bundle
 import android.os.Handler
 import android.os.Looper
@@ -33,6 +32,7 @@ import android.view.View.VISIBLE
 import android.view.ViewGroup
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.net.toFile
+import androidx.core.net.toUri
 import androidx.drawerlayout.widget.DrawerLayout
 import com.google.android.material.bottomnavigation.BottomNavigationView
 import org.kiwix.kiwixmobile.R
@@ -55,7 +55,6 @@ import org.kiwix.kiwixmobile.core.main.ToolbarScrollingKiwixWebView
 import org.kiwix.kiwixmobile.core.utils.SharedPreferenceUtil
 import org.kiwix.kiwixmobile.core.utils.TAG_CURRENT_FILE
 import org.kiwix.kiwixmobile.core.utils.TAG_KIWIX
-import org.kiwix.kiwixmobile.core.utils.files.FileUtils
 import org.kiwix.kiwixmobile.core.utils.files.FileUtils.getAssetFileDescriptorFromUri
 import java.io.File
 
@@ -108,15 +107,18 @@ class KiwixReaderFragment : CoreReaderFragment() {
   }
 
   private fun tryOpeningZimFile(zimFileUri: String) {
-    val filePath = FileUtils.getLocalFilePathByUri(
-      requireActivity().applicationContext, Uri.parse(zimFileUri)
-    )
-
-    if (filePath == null || !File(filePath).isFileExist()) {
-      activity.toast(R.string.error_file_not_found)
-      return
+    getAssetFileDescriptorFromUri(
+      requireActivity(),
+      zimFileUri.toUri()
+    )?.let { assetFileDescriptor ->
+      openZimFile(
+        null,
+        assetFileDescriptor = assetFileDescriptor,
+        assetFileDescriptorPath = zimFileUri
+      )
+    } ?: kotlin.run {
+      activity.toast(R.string.cannot_open_file)
     }
-    openZimFile(File(filePath))
   }
 
   override fun loadDrawerViews() {