Deter reporting the same log over and over

HiddenRamblings · Sep 20, 2023 · 2608914 · 2608914
1 parent 1cfa319
commit 2608914
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 9 deletions.
diff --git a/app/src/main/java/com/hiddenramblings/tagmo/eightbit/io/Debug.kt b/app/src/main/java/com/hiddenramblings/tagmo/eightbit/io/Debug.kt
@@ -40,6 +40,8 @@ object Debug {
         get() = TagMo.appContext
     private val mPrefs = Preferences(context)
 
+    private var lastBugReport = 0L
+
     private val manufacturer: String by lazy {
         try {
             @SuppressLint("PrivateApi")
@@ -229,7 +231,7 @@ object Debug {
     }
 
     private fun submitLogcat(context: Context, logText: String) {
-        if (BuildConfig.WEAR_OS) return
+        lastBugReport = System.currentTimeMillis()
         val subject = context.getString(R.string.git_issue_title, BuildConfig.COMMIT)
         val selectionTitle = if (logText.contains("AndroidRuntime"))
             context.getString(R.string.logcat_crash)
@@ -254,15 +256,19 @@ object Debug {
         }
     }
 
+    private fun setClipboard(context: Context, subject: String, logText: String) {
+        with (context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager) {
+            setPrimaryClip(ClipData.newPlainText(subject, logText))
+        }
+    }
+
     @JvmStatic
-    fun setClipboardException(context: Context, exception: String?) {
+    fun clipException(context: Context, exception: String?) {
         if (BuildConfig.WEAR_OS) return
         val log = getDeviceProfile(context)
         log.append(separator).append(separator).append(exception)
         val subject = context.getString(R.string.git_issue_title, BuildConfig.COMMIT)
-        with (context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager) {
-            setPrimaryClip(ClipData.newPlainText(subject, log.toString()))
-        }
+        setClipboard(context, subject, log.toString())
     }
 
     @JvmStatic
@@ -294,9 +300,25 @@ object Debug {
             log.setLength(0)
             withContext(Dispatchers.Main) {
                 val subject = context.getString(R.string.git_issue_title, BuildConfig.COMMIT)
-                with (context.getSystemService(Context.CLIPBOARD_SERVICE) as ClipboardManager) {
-                    setPrimaryClip(ClipData.newPlainText(subject, logText))
+                if (System.currentTimeMillis() < lastBugReport + 900000) {
+                    if (context is Activity) {
+                        IconifiedSnackbar(context).buildSnackbar(
+                                R.string.duplicate_reports,
+                                R.drawable.ic_support_required_menu,
+                                Snackbar.LENGTH_LONG
+                        ).also { status ->
+                            status.addCallback(object: Snackbar.Callback() {
+                                override fun onDismissed(transientBottomBar: Snackbar?, event: Int) {
+                                    setClipboard(context, subject, logText)
+                                }
+                            })
+                            status.show()
+                        }
+                    }
+                    return@withContext
                 }
+                setClipboard(context, subject, logText)
+                if (BuildConfig.WEAR_OS) return@withContext
                 if (context is Activity) {
                     IconifiedSnackbar(context).buildSnackbar(
                             subject,

diff --git a/app/src/main/java/com/hiddenramblings/tagmo/nfctech/NfcActivity.kt b/app/src/main/java/com/hiddenramblings/tagmo/nfctech/NfcActivity.kt
@@ -606,7 +606,7 @@ class NfcActivity : AppCompatActivity() {
                                     e.printStackTrace(PrintWriter(this))
                                 }
                                 try {
-                                    Debug.setClipboardException(this@NfcActivity, exception.toString())
+                                    Debug.clipException(this@NfcActivity, exception.toString())
                                 } catch (ignored: Exception) {
                                 }
                             }

diff --git a/app/src/main/java/com/hiddenramblings/tagmo/nfctech/ScanTag.kt b/app/src/main/java/com/hiddenramblings/tagmo/nfctech/ScanTag.kt
@@ -179,7 +179,7 @@ class ScanTag {
                                     e.printStackTrace(PrintWriter(this))
                                 }
                                 try {
-                                    Debug.setClipboardException(activity, exception.toString())
+                                    Debug.clipException(activity, exception.toString())
                                 } catch (ignored: Exception) { }
                             }
                         }

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -488,4 +488,6 @@
     <string name="conversion_message">GitHub and Google Play builds are now universal.\nPlease click continue to resolve installation conflicts.</string>
 
     <string name="disclaimer_general">No statues were harmed in making this product.</string>
+
+    <string name="duplicate_reports">Duplicate bug reports are not required</string>
 </resources>