feat: add a crash handler to write logs in Download folder

plateaukao · Sep 30, 2024 · e688d75 · e688d75
1 parent dbd3191
commit e688d75
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 3 deletions.
diff --git a/app/src/main/java/info/plateaukao/einkbro/EinkBroApplication.kt b/app/src/main/java/info/plateaukao/einkbro/EinkBroApplication.kt
@@ -14,6 +14,7 @@ import info.plateaukao.einkbro.pocket.PocketNetwork
 import info.plateaukao.einkbro.preference.ConfigManager
 import info.plateaukao.einkbro.service.TtsManager
 import info.plateaukao.einkbro.unit.LocaleManager
+import info.plateaukao.einkbro.util.CustomExceptionHandler
 import org.koin.android.ext.koin.androidContext
 import org.koin.core.context.GlobalContext.startKoin
 import org.koin.dsl.module
@@ -59,6 +60,10 @@ class EinkBroApplication : Application() {
         }
 
         instance = this
+
+        Thread.setDefaultUncaughtExceptionHandler(
+            CustomExceptionHandler(Thread.getDefaultUncaughtExceptionHandler())
+        )
     }
 
     override fun onTerminate() {

diff --git a/app/src/main/java/info/plateaukao/einkbro/util/CustomExceptionHandler.kt b/app/src/main/java/info/plateaukao/einkbro/util/CustomExceptionHandler.kt
@@ -0,0 +1,20 @@
+package info.plateaukao.einkbro.util
+
+import android.os.Environment
+import java.io.File
+
+class CustomExceptionHandler(private val defaultHandler: Thread.UncaughtExceptionHandler) :
+    Thread.UncaughtExceptionHandler {
+    override fun uncaughtException(thread: Thread, throwable: Throwable) {
+        val tempFile = File(
+            Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS).toString() + "/crash_log.txt"
+        )
+        if (tempFile.exists()) {
+            tempFile.delete()
+        }
+        tempFile.createNewFile()
+        tempFile.appendText(throwable.stackTraceToString())
+
+        defaultHandler.uncaughtException(thread, throwable)
+    }
+}
diff --git a/app/src/main/java/info/plateaukao/einkbro/view/dialog/compose/BookmarksDialogFragment.kt b/app/src/main/java/info/plateaukao/einkbro/view/dialog/compose/BookmarksDialogFragment.kt
@@ -326,7 +326,7 @@ fun BookmarkList(
                                     interactionSource = interactionSource,
                                     indication = null,
                                     onClick = { onBookmarkClick(bookmark) },
-                                    onLongClick = { onBookmarkLongClick?.invoke(bookmark) },
+                                    onLongClick = { onBookmarkLongClick(bookmark) },
                                 )
                             }
                         ),
@@ -350,7 +350,7 @@ fun BookmarkItem(
     dragModifier: Modifier = Modifier,
     iconClick: () -> Unit,
 ) {
-    val borderWidth = if (isPressed) 1.dp else -1.dp
+    val borderWidth = if (isPressed) 1.dp else (-1).dp
 
     Row(
         modifier = modifier

diff --git a/app/src/main/java/info/plateaukao/einkbro/view/dialog/compose/ToolbarConfigDialogFragment.kt b/app/src/main/java/info/plateaukao/einkbro/view/dialog/compose/ToolbarConfigDialogFragment.kt
@@ -123,7 +123,7 @@ private fun ToolbarList(
                 state,
                 key = info.hashCode(),
             ) { isDragging ->
-                val borderWidth = if (isDragging) 1.5.dp else -1.dp
+                val borderWidth = if (isDragging) 1.5.dp else (-1).dp
                 Column(
                     modifier = Modifier
                         .border(