Don't log tryOrNull errors unnecessarily

data/src/main/java/com/moez/QKSMS/manager/ExternalBlockingManagerImpl.kt

@@ -72,10 +72,10 @@ class ExternalBlockingManagerImpl @Inject constructor(
             // If either version of Should I Answer? is installed and SIA is enabled, build the
             // intent to request a rating
             if (prefs.sia.get()) {
-                intent = tryOrNull {
+                intent = tryOrNull(false) {
                     context.packageManager.getApplicationInfo("org.mistergroup.shouldianswerpersonal", 0).enabled
                     Intent("org.mistergroup.shouldianswerpersonal.PublicService").setPackage("org.mistergroup.shouldianswerpersonal")
-                } ?: tryOrNull {
+                } ?: tryOrNull(false) {
                     context.packageManager.getApplicationInfo("org.mistergroup.muzutozvednout", 0).enabled
                     Intent("org.mistergroup.muzutozvednout.PublicService").setPackage("org.mistergroup.muzutozvednout")
                 }

data/src/main/java/com/moez/QKSMS/repository/SyncRepositoryImpl.kt

@@ -163,7 +163,7 @@ class SyncRepositoryImpl @Inject constructor(
         }
 
         // If we don't have a valid id, return null
-        val id = tryOrNull { ContentUris.parseId(uri) } ?: return null
+        val id = tryOrNull(false) { ContentUris.parseId(uri) } ?: return null
 
         // Check if the message already exists, so we can reuse the id
         val existingId = Realm.getDefaultInstance().use { realm ->

data/src/main/java/com/moez/QKSMS/util/Utils.kt

@@ -20,11 +20,14 @@ package com.moez.QKSMS.util
 
 import timber.log.Timber
 
-fun <T> tryOrNull(body: () -> T?): T? {
+fun <T> tryOrNull(logOnError: Boolean = true, body: () -> T?): T? {
     return try {
         body()
     } catch (e: Exception) {
-        Timber.w(e)
+        if (logOnError) {
+            Timber.w(e)
+        }
+
         null
     }
 }