Introduced a Open/Download dialog for Epub/Pdf files.

* This was requested in #3453 and this PR is related to the Epub opening issue so we are introducing this here.

app/src/main/java/org/kiwix/kiwixmobile/nav/destination/reader/KiwixReaderFragment.kt

@@ -248,7 +248,7 @@ class KiwixReaderFragment : CoreReaderFragment() {
   override fun createWebView(attrs: AttributeSet?): ToolbarScrollingKiwixWebView {
     return ToolbarScrollingKiwixWebView(
       requireContext(), this, attrs!!, activityMainRoot as ViewGroup, videoView!!,
-      CoreWebViewClient(this, zimReaderContainer!!, sharedPreferenceUtil!!),
+      CoreWebViewClient(this, zimReaderContainer!!),
       toolbarContainer!!, bottomToolbar!!, sharedPreferenceUtil = sharedPreferenceUtil!!,
       parentNavigationBar = requireActivity().findViewById(R.id.bottom_nav_view)
     )

core/src/main/java/org/kiwix/kiwixmobile/core/main/CoreReaderFragment.kt

@@ -156,6 +156,7 @@ import org.kiwix.kiwixmobile.core.utils.TAG_FILE_SEARCHED_NEW_TAB
 import org.kiwix.kiwixmobile.core.utils.TAG_KIWIX
 import org.kiwix.kiwixmobile.core.utils.UpdateUtils.reformatProviderUrl
 import org.kiwix.kiwixmobile.core.utils.dialog.DialogShower
+import org.kiwix.kiwixmobile.core.utils.dialog.DownloadOrOpenEpubAndPdfHandler
 import org.kiwix.kiwixmobile.core.utils.dialog.KiwixDialog
 import org.kiwix.kiwixmobile.core.utils.files.FileUtils.deleteCachedFiles
 import org.kiwix.kiwixmobile.core.utils.files.FileUtils.readFile
@@ -317,6 +318,10 @@ abstract class CoreReaderFragment :
   @JvmField
   @Inject
   var externalLinkOpener: ExternalLinkOpener? = null
+
+  @JvmField
+  @Inject
+  var downloadOrOpenEpubAndPdfHandler: DownloadOrOpenEpubAndPdfHandler? = null
   private var hideBackToTopTimer: CountDownTimer? = null
   private var documentSections: MutableList<DocumentSection>? = null
   private var isBackToTopEnabled = false
@@ -1163,7 +1168,7 @@ abstract class CoreReaderFragment :
     return if (activityMainRoot != null) {
       ToolbarScrollingKiwixWebView(
         requireActivity(), this, attrs!!, (activityMainRoot as ViewGroup?)!!, videoView!!,
-        CoreWebViewClient(this, zimReaderContainer!!, sharedPreferenceUtil!!),
+        CoreWebViewClient(this, zimReaderContainer!!),
         toolbarContainer!!, bottomToolbar!!,
         sharedPreferenceUtil!!
       )
@@ -1470,6 +1475,10 @@ abstract class CoreReaderFragment :
     externalLinkOpener?.openExternalUrl(intent)
   }
 
+  override fun showDownloadOrOpenEpubAndPdfDialog(url: String, documentType: String?) {
+    downloadOrOpenEpubAndPdfHandler?.showDownloadOrOpenEpubAndPdfDialog(url, documentType)
+  }
+
   protected fun openZimFile(
     file: File?,
     isCustomApp: Boolean = false,

core/src/main/java/org/kiwix/kiwixmobile/core/main/CoreWebViewClient.kt

@@ -17,7 +17,6 @@
  */
 package org.kiwix.kiwixmobile.core.main
 
-import android.content.Context
 import android.content.Intent
 import android.net.Uri
 import android.util.Log
@@ -27,18 +26,14 @@ import android.webkit.WebResourceRequest
 import android.webkit.WebResourceResponse
 import android.webkit.WebView
 import android.webkit.WebViewClient
-import androidx.core.content.FileProvider
 import org.kiwix.kiwixmobile.core.CoreApp.Companion.instance
 import org.kiwix.kiwixmobile.core.reader.ZimFileReader
 import org.kiwix.kiwixmobile.core.reader.ZimReaderContainer
-import org.kiwix.kiwixmobile.core.utils.SharedPreferenceUtil
 import org.kiwix.kiwixmobile.core.utils.TAG_KIWIX
-import org.kiwix.kiwixmobile.core.utils.files.FileUtils.downloadFileFromUrl
 
 open class CoreWebViewClient(
   protected val callback: WebViewCallback,
-  protected val zimReaderContainer: ZimReaderContainer,
-  private val sharedPreferenceUtil: SharedPreferenceUtil
+  protected val zimReaderContainer: ZimReaderContainer
 ) : WebViewClient() {
   private var urlWithAnchor: String? = null
 
@@ -85,26 +80,7 @@ open class CoreWebViewClient(
   private fun handleEpubAndPdf(url: String): Boolean {
     val extension = MimeTypeMap.getFileExtensionFromUrl(url)
     if (DOCUMENT_TYPES.containsKey(extension)) {
-      downloadFileFromUrl(
-        url,
-        null,
-        zimReaderContainer,
-        sharedPreferenceUtil
-      )?.let {
-        if (it.exists()) {
-          val context: Context = instance
-          val uri = FileProvider.getUriForFile(
-            context,
-            context.packageName + ".fileprovider", it
-          )
-          val intent = Intent(Intent.ACTION_VIEW).apply {
-            setDataAndType(uri, DOCUMENT_TYPES[extension])
-            flags = Intent.FLAG_ACTIVITY_NO_HISTORY
-            addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION)
-          }
-          callback.openExternalUrl(intent)
-        }
-      }
+      callback.showDownloadOrOpenEpubAndPdfDialog(url, DOCUMENT_TYPES[extension])
       return true
     }
     return false

core/src/main/java/org/kiwix/kiwixmobile/core/main/WebViewCallback.kt

@@ -25,6 +25,7 @@ interface WebViewCallback {
   fun webViewUrlFinishedLoading()
   fun webViewFailedLoading(failingUrl: String)
   fun openExternalUrl(intent: Intent)
+  fun showDownloadOrOpenEpubAndPdfDialog(url: String, documentType: String?)
   fun webViewProgressChanged(progress: Int, webView: WebView)
   fun webViewTitleUpdated(title: String)
   fun webViewPageChanged(page: Int, maxPages: Int)

core/src/main/java/org/kiwix/kiwixmobile/core/utils/dialog/DownloadOrOpenEpubAndPdfHandler.kt

@@ -0,0 +1,87 @@
+/*
+ * Kiwix Android
+ * Copyright (c) 2024 Kiwix <android.kiwix.org>
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ */
+
+package org.kiwix.kiwixmobile.core.utils.dialog
+
+import android.app.Activity
+import android.content.Intent
+import android.util.Log
+import androidx.core.content.FileProvider
+import org.kiwix.kiwixmobile.core.R
+import org.kiwix.kiwixmobile.core.extensions.toast
+import org.kiwix.kiwixmobile.core.reader.ZimReaderContainer
+import org.kiwix.kiwixmobile.core.utils.SharedPreferenceUtil
+import org.kiwix.kiwixmobile.core.utils.files.FileUtils.downloadFileFromUrl
+import java.io.File
+import javax.inject.Inject
+
+class DownloadOrOpenEpubAndPdfHandler @Inject constructor(
+  private val activity: Activity,
+  private val sharedPreferenceUtil: SharedPreferenceUtil,
+  private val alertDialogShower: AlertDialogShower,
+  private val zimReaderContainer: ZimReaderContainer
+) {
+
+  fun showDownloadOrOpenEpubAndPdfDialog(url: String, documentType: String?) {
+    alertDialogShower.show(
+      KiwixDialog.DownloadOrOpenEpubAndPdf,
+      { openOrDownloadFile(url, documentType, true) },
+      { openOrDownloadFile(url, documentType, false) },
+      { }
+    )
+  }
+
+  private fun openOrDownloadFile(url: String, documentType: String?, openFile: Boolean) {
+    downloadFileFromUrl(
+      url,
+      null,
+      zimReaderContainer,
+      sharedPreferenceUtil
+    )?.let { savedFile ->
+      if (openFile) {
+        openFile(savedFile, documentType)
+      } else {
+        activity.toast(activity.getString(R.string.save_media_saved, savedFile.name)).also {
+          Log.e("DownloadOrOpenEpubAndPdf", "File downloaded at = ${savedFile.path}")
+        }
+      }
+    } ?: run {
+      activity.toast(R.string.save_media_error)
+    }
+  }
+
+  private fun openFile(savedFile: File, documentType: String?) {
+    if (savedFile.exists()) {
+      val uri = FileProvider.getUriForFile(
+        activity,
+        "${activity.packageName}.fileprovider",
+        savedFile
+      )
+      val intent = Intent(Intent.ACTION_VIEW).apply {
+        setDataAndType(uri, documentType)
+        flags = Intent.FLAG_ACTIVITY_NO_HISTORY
+        addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION)
+      }
+      if (intent.resolveActivity(activity.packageManager) != null) {
+        activity.startActivity(intent)
+      } else {
+        activity.toast(R.string.no_reader_application_installed)
+      }
+    }
+  }
+}

core/src/main/java/org/kiwix/kiwixmobile/core/utils/dialog/KiwixDialog.kt

@@ -108,6 +108,14 @@ sealed class KiwixDialog(
     cancelable = false
   )
 
+  object DownloadOrOpenEpubAndPdf : KiwixDialog(
+    R.string.download_or_open_pdf_and_epub_content_dialog_title,
+    R.string.download_or_open_pdf_and_epub_content_dialog_message,
+    R.string.open,
+    R.string.download,
+    neutralMessage = R.string.no_thanks
+  )
+
   data class ShowHotspotDetails(override val args: List<Any>) :
     KiwixDialog(
       R.string.hotspot_turned_on,

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

@@ -355,4 +355,6 @@
   <string name="zim_file_content_description">ZIM file which has the reading content</string>
   <string name="select_language_content_description">Selecting this language will prioritize displaying downloadable books in that language at the top.</string>
   <string name="toolbar_back_button_content_description">Go to previous screen</string>
+  <string name="download_or_open_pdf_and_epub_content_dialog_title">Download or Open this (Epub/Pdf) file?</string>
+  <string name="download_or_open_pdf_and_epub_content_dialog_message">Choosing Open will open this file in your external Epub/PDF reader application.</string>
 </resources>