From db2dd4b6c647dc01650b98472664a4e043f8f572 Mon Sep 17 00:00:00 2001
From: arkon <eugcheung94@gmail.com>
Date: Thu, 21 Jan 2021 18:21:03 -0500
Subject: [PATCH] Add method for users to save error logs to a file

---
 .../data/notification/Notifications.kt        | 13 ++++-
 .../ui/setting/SettingsAdvancedController.kt  | 11 ++++
 .../eu/kanade/tachiyomi/util/CrashLogUtil.kt  | 54 +++++++++++++++++++
 app/src/main/res/values/strings.xml           | 14 +++--
 4 files changed, 86 insertions(+), 6 deletions(-)
 create mode 100644 app/src/main/java/eu/kanade/tachiyomi/util/CrashLogUtil.kt

diff --git a/app/src/main/java/eu/kanade/tachiyomi/data/notification/Notifications.kt b/app/src/main/java/eu/kanade/tachiyomi/data/notification/Notifications.kt
index b55b53fedd..97a4b2ef3d 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/data/notification/Notifications.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/data/notification/Notifications.kt
@@ -62,6 +62,12 @@ object Notifications {
     const val ID_BACKUP_COMPLETE = -502
     const val ID_RESTORE_COMPLETE = -504
 
+    /**
+     * Notification channel used for crash log file sharing.
+     */
+    const val CHANNEL_CRASH_LOGS = "crash_logs_channel"
+    const val ID_CRASH_LOGS = -601
+
     private val deprecatedChannels = listOf(
         "downloader_channel",
         "backup_restore_complete_channel"
@@ -143,7 +149,12 @@ object Notifications {
                 group = GROUP_BACKUP_RESTORE
                 setShowBadge(false)
                 setSound(null, null)
-            }
+            },
+            NotificationChannel(
+                CHANNEL_CRASH_LOGS,
+                context.getString(R.string.channel_crash_logs),
+                NotificationManager.IMPORTANCE_HIGH
+            )
         ).forEach(context.notificationManager::createNotificationChannel)
 
         // Delete old notification channels
diff --git a/app/src/main/java/eu/kanade/tachiyomi/ui/setting/SettingsAdvancedController.kt b/app/src/main/java/eu/kanade/tachiyomi/ui/setting/SettingsAdvancedController.kt
index 00d7ad7442..69cb08033e 100644
--- a/app/src/main/java/eu/kanade/tachiyomi/ui/setting/SettingsAdvancedController.kt
+++ b/app/src/main/java/eu/kanade/tachiyomi/ui/setting/SettingsAdvancedController.kt
@@ -17,6 +17,7 @@ import eu.kanade.tachiyomi.data.library.LibraryUpdateService
 import eu.kanade.tachiyomi.data.library.LibraryUpdateService.Target
 import eu.kanade.tachiyomi.network.NetworkHelper
 import eu.kanade.tachiyomi.ui.base.controller.DialogController
+import eu.kanade.tachiyomi.util.CrashLogUtil
 import eu.kanade.tachiyomi.util.preference.defaultValue
 import eu.kanade.tachiyomi.util.preference.onClick
 import eu.kanade.tachiyomi.util.preference.preference
@@ -49,6 +50,16 @@ class SettingsAdvancedController : SettingsController() {
             defaultValue = true
         }
 
+        preference {
+            key = "dump_crash_logs"
+            titleRes = R.string.pref_dump_crash_logs
+            summaryRes = R.string.pref_dump_crash_logs_summary
+
+            onClick {
+                CrashLogUtil(context).dumpLogs()
+            }
+        }
+
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
             preference {
                 key = "pref_disable_battery_optimization"
diff --git a/app/src/main/java/eu/kanade/tachiyomi/util/CrashLogUtil.kt b/app/src/main/java/eu/kanade/tachiyomi/util/CrashLogUtil.kt
new file mode 100644
index 0000000000..751bd8cd01
--- /dev/null
+++ b/app/src/main/java/eu/kanade/tachiyomi/util/CrashLogUtil.kt
@@ -0,0 +1,54 @@
+package eu.kanade.tachiyomi.util
+
+import android.content.Context
+import android.net.Uri
+import eu.kanade.tachiyomi.R
+import eu.kanade.tachiyomi.data.notification.NotificationReceiver
+import eu.kanade.tachiyomi.data.notification.Notifications
+import eu.kanade.tachiyomi.util.storage.getUriCompat
+import eu.kanade.tachiyomi.util.system.notificationBuilder
+import eu.kanade.tachiyomi.util.system.notificationManager
+import eu.kanade.tachiyomi.util.system.toast
+import java.io.File
+import java.io.IOException
+
+class CrashLogUtil(private val context: Context) {
+
+    private val notificationBuilder = context.notificationBuilder(Notifications.CHANNEL_CRASH_LOGS) {
+        setSmallIcon(R.drawable.ic_tachi)
+    }
+
+    fun dumpLogs() {
+        try {
+            val file = File(context.externalCacheDir, "tachiyomi_crash_logs.txt")
+            if (file.exists()) {
+                file.delete()
+            }
+            file.createNewFile()
+            Runtime.getRuntime().exec("logcat *:E -d -f ${file.absolutePath}")
+
+            showNotification(file.getUriCompat(context))
+        } catch (e: IOException) {
+            context.toast("Failed to get logs")
+        }
+    }
+
+    private fun showNotification(uri: Uri) {
+        context.notificationManager.cancel(Notifications.ID_CRASH_LOGS)
+
+        with(notificationBuilder) {
+            setContentTitle(context.getString(R.string.crash_log_saved))
+
+            // Clear old actions if they exist
+            clearActions()
+
+            addAction(
+                R.drawable.ic_folder_24dp,
+                context.getString(R.string.action_open_log),
+                NotificationReceiver.openErrorLogPendingActivity(context, uri)
+            )
+
+            context.notificationManager.notify(Notifications.ID_CRASH_LOGS, build())
+        }
+    }
+}
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 85bf2a2219..5048277ee5 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -271,6 +271,11 @@
     <string name="pref_read_with_volume_keys">Volume keys</string>
     <string name="pref_read_with_volume_keys_inverted">Invert volume keys</string>
     <string name="pref_read_with_tapping">Tapping</string>
+    <string name="pref_read_with_tapping_inverted">Invert tapping</string>
+    <string name="tapping_inverted_none">None</string>
+    <string name="tapping_inverted_horizontal">Horizontal</string>
+    <string name="tapping_inverted_vertical">Vertical</string>
+    <string name="tapping_inverted_both">Both</string>
     <string name="pref_read_with_long_tap">Long tap dialog</string>
     <string name="pref_reader_theme">Background color</string>
     <string name="white_background">White</string>
@@ -410,6 +415,9 @@
     <string name="pref_refresh_library_covers">Refresh library manga covers</string>
     <string name="pref_refresh_library_tracking">Refresh tracking</string>
     <string name="pref_refresh_library_tracking_summary">Updates status, score and last chapter read from the tracking services</string>
+    <string name="pref_dump_crash_logs">Dump crash logs</string>
+    <string name="pref_dump_crash_logs_summary">Saves error logs to a file for sharing with the developers</string>
+    <string name="crash_log_saved">Crash logs saved</string>
     <string name="pref_disable_battery_optimization">Disable battery optimization</string>
     <string name="pref_disable_battery_optimization_summary">Helps with background library updates and backups</string>
     <string name="battery_optimization_disabled">Battery optimization is already disabled</string>
@@ -729,11 +737,7 @@
     <string name="group_backup_restore">Backup and restore</string>
     <string name="channel_new_chapters">Chapter updates</string>
     <string name="channel_ext_updates">Extension updates</string>
-    <string name="pref_read_with_tapping_inverted">Invert tapping</string>
-    <string name="tapping_inverted_none">None</string>
-    <string name="tapping_inverted_horizontal">Horizontal</string>
-    <string name="tapping_inverted_vertical">Vertical</string>
-    <string name="tapping_inverted_both">Both</string>
+    <string name="channel_crash_logs">Crash logs</string>
 
     <!-- S Pen actions -->
     <string name="spen_previous_page">Previous page</string>