[CLEANUP] Timber all the way 🌳

Author: hossain-khan

app/src/main/java/dev/hossain/keepalive/util/AppChecker.java

@@ -12,6 +12,8 @@
 import java.util.SortedMap;
 import java.util.TreeMap;
 
+import timber.log.Timber;
+
 public class AppChecker {
 
     public static boolean isGooglePhotosRunning(Context context) {
@@ -46,7 +48,7 @@ private static boolean isAppRunning(Context context, String packageName) {
                     return true;
                 }
             } else {
-                Log.d("AppChecker", "appList is null or empty. " + appList);
+                Timber.d("appList is null or empty. %s", appList);
             }
         }
         return false;
@@ -55,7 +57,7 @@ private static boolean isAppRunning(Context context, String packageName) {
     private static void printSortedMap(SortedMap<Long, UsageStats> sortedMap) {
         for (Long key : sortedMap.keySet()) {
             UsageStats usageStats = sortedMap.get(key);
-            Log.d("AppChecker", "UsageStats: " + usageStats.getPackageName() + " | " + usageStats.getLastTimeUsed());
+            Timber.d("UsageStats: " + usageStats.getPackageName() + " | " + usageStats.getLastTimeUsed());
         }
     }
 }

app/src/main/java/dev/hossain/keepalive/util/AppLauncher.kt

@@ -4,22 +4,20 @@ import android.content.ActivityNotFoundException
 import android.content.ComponentName
 import android.content.Context
 import android.content.Intent
-import android.util.Log
 import dev.hossain.keepalive.util.AppConfig.PHOTOS_APP_LAUNCH_ACTIVITY
 import dev.hossain.keepalive.util.AppConfig.PHOTOS_APP_PACKAGE_NAME
 import dev.hossain.keepalive.util.AppConfig.SYNC_APP_LAUNCH_ACTIVITY
 import dev.hossain.keepalive.util.AppConfig.SYNC_APP_PACKAGE_NAME
+import timber.log.Timber
 
 object AppLauncher {
-    private const val TAG = "AppLauncher"
-
     fun openGooglePhotos(context: Context) {
-        Log.i(TAG, "openGooglePhotos")
+        Timber.i("openGooglePhotos")
         startApplication(context, PHOTOS_APP_PACKAGE_NAME, PHOTOS_APP_LAUNCH_ACTIVITY)
     }
 
     fun openSyncthing(context: Context) {
-        Log.i(TAG, "openSyncthing")
+        Timber.i("openSyncthing")
         startApplication(context, SYNC_APP_PACKAGE_NAME, SYNC_APP_LAUNCH_ACTIVITY)
     }
 
@@ -41,7 +39,7 @@ object AppLauncher {
             context.startActivity(launchIntent)
         } catch (e: ActivityNotFoundException) {
             e.printStackTrace()
-            Log.e(TAG, "Unable to find activity: $launchIntent", e)
+            Timber.e(e, "Unable to find activity: $launchIntent")
         }
     }
 }

app/src/main/java/dev/hossain/keepalive/util/HttpPingSender.kt

@@ -1,28 +1,24 @@
 package dev.hossain.keepalive.util
 
 import android.content.Context
-import android.util.Log
 import okhttp3.OkHttpClient
 import okhttp3.Request
 import okio.IOException
+import timber.log.Timber
 
 /**
  * Sends HTTP ping to a given URL.
  */
 class HttpPingSender(private val context: Context) {
-    companion object {
-        private const val TAG = "HttpPingSender"
-    }
-
     private val client = OkHttpClient()
 
     fun sendPingToDevice() {
         val deviceModel = android.os.Build.MODEL
         AppConfig.phoneToUrlMap[deviceModel]?.let { pingUrl ->
-            Log.d(TAG, "sendPingToDevice: Pinging device $deviceModel with URL $pingUrl")
+            Timber.d("sendPingToDevice: Pinging device $deviceModel with URL $pingUrl")
             sendHttpPing(pingUrl)
         } ?: run {
-            Log.e(TAG, "sendPingToDevice: No URL found for device $deviceModel - Not supported.")
+            Timber.e("sendPingToDevice: No URL found for device $deviceModel - Not supported.")
         }
     }
 
@@ -44,9 +40,9 @@ class HttpPingSender(private val context: Context) {
                 .build()
         client.newCall(request).execute().use { response ->
             if (!response.isSuccessful) {
-                Log.e(TAG, "Unexpected code $response", IOException("Unexpected code $response"))
+                Timber.e(IOException("Unexpected code $response"), "Unexpected code $response")
             } else {
-                Log.d(TAG, "Heartbeat Ping Sent: Response: ${response.body?.string()}")
+                Timber.d("Heartbeat Ping Sent: Response: " + response.body?.string())
             }
         }
     }

app/src/main/java/dev/hossain/keepalive/util/NotificationHelper.kt

@@ -6,22 +6,21 @@ import android.app.NotificationManager
 import android.app.PendingIntent
 import android.content.Context
 import android.content.Intent
-import android.util.Log
 import androidx.core.app.NotificationCompat
 import dev.hossain.keepalive.MainActivity
 import dev.hossain.keepalive.R
+import timber.log.Timber
 
 /**
  * Helper class to create notification channel and build notification.
  */
 class NotificationHelper(private val context: Context) {
     companion object {
-        private const val TAG = "NotificationHelper"
         private const val CHANNEL_ID = "WatchdogServiceChannel"
     }
 
     fun createNotificationChannel() {
-        Log.d(TAG, "createNotificationChannel() called")
+        Timber.d("createNotificationChannel() called")
 
         val channel =
             NotificationChannel(
@@ -35,7 +34,7 @@ class NotificationHelper(private val context: Context) {
     }
 
     fun buildNotification(): Notification {
-        Log.d(TAG, "buildNotification() called")
+        Timber.d("buildNotification() called")
 
         val notificationIntent = Intent(context, MainActivity::class.java)
         val pendingIntent =