Add method for users to save error logs to a file
This commit is contained in:
parent
a68417a0b0
commit
db2dd4b6c6
@ -62,6 +62,12 @@ object Notifications {
|
|||||||
const val ID_BACKUP_COMPLETE = -502
|
const val ID_BACKUP_COMPLETE = -502
|
||||||
const val ID_RESTORE_COMPLETE = -504
|
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(
|
private val deprecatedChannels = listOf(
|
||||||
"downloader_channel",
|
"downloader_channel",
|
||||||
"backup_restore_complete_channel"
|
"backup_restore_complete_channel"
|
||||||
@ -143,7 +149,12 @@ object Notifications {
|
|||||||
group = GROUP_BACKUP_RESTORE
|
group = GROUP_BACKUP_RESTORE
|
||||||
setShowBadge(false)
|
setShowBadge(false)
|
||||||
setSound(null, null)
|
setSound(null, null)
|
||||||
}
|
},
|
||||||
|
NotificationChannel(
|
||||||
|
CHANNEL_CRASH_LOGS,
|
||||||
|
context.getString(R.string.channel_crash_logs),
|
||||||
|
NotificationManager.IMPORTANCE_HIGH
|
||||||
|
)
|
||||||
).forEach(context.notificationManager::createNotificationChannel)
|
).forEach(context.notificationManager::createNotificationChannel)
|
||||||
|
|
||||||
// Delete old notification channels
|
// Delete old notification channels
|
||||||
|
@ -17,6 +17,7 @@ import eu.kanade.tachiyomi.data.library.LibraryUpdateService
|
|||||||
import eu.kanade.tachiyomi.data.library.LibraryUpdateService.Target
|
import eu.kanade.tachiyomi.data.library.LibraryUpdateService.Target
|
||||||
import eu.kanade.tachiyomi.network.NetworkHelper
|
import eu.kanade.tachiyomi.network.NetworkHelper
|
||||||
import eu.kanade.tachiyomi.ui.base.controller.DialogController
|
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.defaultValue
|
||||||
import eu.kanade.tachiyomi.util.preference.onClick
|
import eu.kanade.tachiyomi.util.preference.onClick
|
||||||
import eu.kanade.tachiyomi.util.preference.preference
|
import eu.kanade.tachiyomi.util.preference.preference
|
||||||
@ -49,6 +50,16 @@ class SettingsAdvancedController : SettingsController() {
|
|||||||
defaultValue = true
|
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) {
|
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
|
||||||
preference {
|
preference {
|
||||||
key = "pref_disable_battery_optimization"
|
key = "pref_disable_battery_optimization"
|
||||||
|
54
app/src/main/java/eu/kanade/tachiyomi/util/CrashLogUtil.kt
Normal file
54
app/src/main/java/eu/kanade/tachiyomi/util/CrashLogUtil.kt
Normal file
@ -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())
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
@ -271,6 +271,11 @@
|
|||||||
<string name="pref_read_with_volume_keys">Volume keys</string>
|
<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_volume_keys_inverted">Invert volume keys</string>
|
||||||
<string name="pref_read_with_tapping">Tapping</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_read_with_long_tap">Long tap dialog</string>
|
||||||
<string name="pref_reader_theme">Background color</string>
|
<string name="pref_reader_theme">Background color</string>
|
||||||
<string name="white_background">White</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_covers">Refresh library manga covers</string>
|
||||||
<string name="pref_refresh_library_tracking">Refresh tracking</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_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">Disable battery optimization</string>
|
||||||
<string name="pref_disable_battery_optimization_summary">Helps with background library updates and backups</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>
|
<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="group_backup_restore">Backup and restore</string>
|
||||||
<string name="channel_new_chapters">Chapter updates</string>
|
<string name="channel_new_chapters">Chapter updates</string>
|
||||||
<string name="channel_ext_updates">Extension updates</string>
|
<string name="channel_ext_updates">Extension updates</string>
|
||||||
<string name="pref_read_with_tapping_inverted">Invert tapping</string>
|
<string name="channel_crash_logs">Crash logs</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>
|
|
||||||
|
|
||||||
<!-- S Pen actions -->
|
<!-- S Pen actions -->
|
||||||
<string name="spen_previous_page">Previous page</string>
|
<string name="spen_previous_page">Previous page</string>
|
||||||
|
Loading…
Reference in New Issue
Block a user