REM-877 - Move logging to the separate module

app/build.gradle

@@ -167,6 +167,10 @@ configurations.testImplementation {
 }
 
 dependencies {
+    implementation project(':voice-engine-ktx')
+    implementation project(':logging-api')
+    implementation project(':logging')
+
     implementation('com.google.apis:google-api-services-calendar:v3-rev20230406-2.0.0') {
         exclude group: 'com.google.guava', module: 'listenablefuture'
     }
@@ -178,7 +182,6 @@ dependencies {
     implementation('com.google.apis:google-api-services-drive:v3-rev20230423-2.0.0') {
         exclude group: 'org.apache.httpcomponents'
     }
-    implementation project(':voice-engine-ktx')
 
     implementation 'io.insert-koin:koin-android:3.5.6'
     implementation 'io.insert-koin:koin-android-ext:3.0.2'
@@ -221,9 +224,9 @@ dependencies {
     implementation 'androidx.navigation:navigation-fragment-ktx:2.8.5'
     implementation 'androidx.navigation:navigation-ui-ktx:2.8.5'
 
-    implementation platform('com.google.firebase:firebase-bom:33.7.0')
+    implementation libs.firebase.bom
 
-    implementation 'com.google.firebase:firebase-crashlytics'
+    implementation libs.firebase.crashlytics
     implementation 'com.google.firebase:firebase-analytics-ktx'
     implementation 'com.google.firebase:firebase-auth-ktx'
     implementation 'com.google.firebase:firebase-config'
@@ -249,7 +252,7 @@ dependencies {
     implementation 'com.github.naz013:android-calendar-ext:1.0.4'
     implementation 'com.airbnb.android:lottie:6.6.1'
     implementation 'com.github.chrisbanes:PhotoView:2.3.0'
-    implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
+    implementation "org.jetbrains.kotlin:kotlin-stdlib:2.1.0"
     implementation 'org.threeten:threetenbp:1.7.0'
     implementation 'com.maxkeppeler.sheets:core:2.3.1'
     implementation 'com.maxkeppeler.sheets:lottie:2.3.1'

app/src/main/java/com/elementary/tasks/ReminderApp.kt

@@ -6,7 +6,6 @@ import androidx.multidex.MultiDex
 import androidx.multidex.MultiDexApplication
 import com.elementary.tasks.birthdays.birthdaysModule
 import com.elementary.tasks.calendar.calendarModule
-import com.elementary.tasks.core.analytics.Traces
 import com.elementary.tasks.core.appwidgets.widgetModule
 import com.elementary.tasks.core.data.adapter.adapterModule
 import com.elementary.tasks.core.data.repository.repositoryModule
@@ -31,6 +30,7 @@ import com.elementary.tasks.home.homeModule
 import com.elementary.tasks.notes.noteModule
 import com.elementary.tasks.reminder.reminderModule
 import com.elementary.tasks.voice.voiceModule
+import com.github.naz013.logging.initLogging
 import org.koin.android.ext.android.get
 import org.koin.android.ext.koin.androidContext
 import org.koin.androidx.workmanager.koin.workManagerFactory
@@ -39,7 +39,6 @@ import org.koin.core.context.startKoin
 import org.koin.core.logger.Level
 import org.koin.core.logger.Logger
 import org.koin.core.logger.MESSAGE
-import timber.log.Timber
 
 @Suppress("unused")
 class ReminderApp : MultiDexApplication(), KoinComponent {
@@ -51,8 +50,9 @@ class ReminderApp : MultiDexApplication(), KoinComponent {
 
   override fun onCreate() {
     super.onCreate()
-    if (BuildConfig.DEBUG) Timber.plant(Timber.DebugTree())
-    Traces.setUpKeys(this)
+    initLogging(
+      isDebug = BuildConfig.DEBUG
+    )
     AppCompatDelegate.setCompatVectorFromResourcesEnabled(true)
     val logger = object : Logger(level = Level.DEBUG) {
       override fun display(level: Level, msg: MESSAGE) {

app/src/main/java/com/elementary/tasks/birthdays/create/AddBirthdayViewModel.kt

@@ -7,7 +7,6 @@ import com.elementary.tasks.birthdays.work.SingleBackupWorker
 import com.elementary.tasks.core.analytics.AnalyticsEventSender
 import com.elementary.tasks.core.analytics.Feature
 import com.elementary.tasks.core.analytics.FeatureUsedEvent
-import com.elementary.tasks.core.analytics.Traces
 import com.elementary.tasks.core.appwidgets.UpdatesHelper
 import com.elementary.tasks.core.arch.BaseProgressViewModel
 import com.elementary.tasks.core.data.Commands
@@ -25,9 +24,9 @@ import com.elementary.tasks.core.utils.io.UriReader
 import com.elementary.tasks.core.utils.mutableLiveDataOf
 import com.elementary.tasks.core.utils.toLiveData
 import com.elementary.tasks.core.utils.work.WorkerLauncher
+import com.github.naz013.logging.Logger
 import kotlinx.coroutines.launch
 import org.threeten.bp.LocalDate
-import timber.log.Timber
 import java.util.UUID
 
 class AddBirthdayViewModel(
@@ -65,14 +64,14 @@ class AddBirthdayViewModel(
   fun load() {
     viewModelScope.launch(dispatcherProvider.default()) {
       val birthday = birthdaysDao.getById(id) ?: return@launch
-      Traces.logEvent("Birthday loaded from DB")
+      Logger.logEvent("Birthday loaded from DB")
       onBirthdayLoaded(birthday)
     }
   }
 
   fun onIntent() {
     intentDataHolder.get(Constants.INTENT_ITEM, Birthday::class.java)?.run {
-      Traces.logEvent("Birthday loaded from intent")
+      Logger.logEvent("Birthday loaded from intent")
       onBirthdayLoaded(this)
       isFromFile = true
       findSame(uuId)
@@ -83,7 +82,7 @@ class AddBirthdayViewModel(
     viewModelScope.launch(dispatcherProvider.default()) {
       runCatching {
         uriReader.readBirthdayObject(uri)?.also {
-          Traces.logEvent("Birthday loaded from file")
+          Logger.logEvent("Birthday loaded from file")
           onBirthdayLoaded(it)
           isFromFile = true
           findSame(it.uuId)
@@ -97,7 +96,7 @@ class AddBirthdayViewModel(
   }
 
   fun onDateChanged(localDate: LocalDate) {
-    Timber.d("onDateChanged: $localDate")
+    Logger.d("onDateChanged: $localDate")
     selectedDate = localDate
     _formattedDate.postValue(uiBirthdayDateFormatter.getDateFormatted(localDate))
   }
@@ -133,7 +132,7 @@ class AddBirthdayViewModel(
         updatedAt = dateTimeManager.getNowGmtDateTime(),
         ignoreYear = ignoreYear
       )
-      Traces.logEvent("Birthday saved")
+      Logger.logEvent("Birthday saved")
       analyticsEventSender.send(FeatureUsedEvent(Feature.CREATE_BIRTHDAY))
       saveBirthday(birthday)
     }
@@ -147,7 +146,7 @@ class AddBirthdayViewModel(
       updatesHelper.updateTasksWidget()
       updatesHelper.updateBirthdaysWidget()
       workerLauncher.startWork(BirthdayDeleteBackupWorker::class.java, Constants.INTENT_ID, id)
-      Traces.logEvent("Birthday deleted")
+      Logger.logEvent("Birthday deleted")
       postInProgress(false)
       postCommand(Commands.DELETED)
     }

app/src/main/java/com/elementary/tasks/core/appwidgets/singlenote/SingleNoteWidgetPrefsProvider.kt

@@ -1,10 +1,10 @@
 package com.elementary.tasks.core.appwidgets.singlenote
 
 import android.content.Context
-import com.elementary.tasks.core.analytics.Traces
 import com.elementary.tasks.core.appwidgets.WidgetPrefsProvider
 import com.elementary.tasks.core.utils.ThemeProvider
 import com.elementary.tasks.core.views.drawable.NoteDrawableParams
+import com.github.naz013.logging.Logger
 
 class SingleNoteWidgetPrefsProvider(
   context: Context,
@@ -57,7 +57,7 @@ class SingleNoteWidgetPrefsProvider(
 
   fun getTextColorPosition(): Int {
     return getInt(WIDGET_TEXT_COLOR_POSITION, def = ThemeProvider.Color.BLACK).also {
-      Traces.d("getTextColorPosition: $it")
+      Logger.d("getTextColorPosition: $it")
     }
   }
 
@@ -75,7 +75,7 @@ class SingleNoteWidgetPrefsProvider(
 
   fun getOverlayColorPosition(): Int {
     return getInt(WIDGET_OVERLAY_COLOR_POSITION, def = ThemeProvider.Color.WHITE).also {
-      Traces.d("getOverlayColorPosition: $it")
+      Logger.d("getOverlayColorPosition: $it")
     }
   }
 

app/src/main/java/com/elementary/tasks/core/cloud/DataFlow.kt

@@ -1,12 +1,12 @@
 package com.elementary.tasks.core.cloud
 
-import com.elementary.tasks.core.analytics.Traces
 import com.elementary.tasks.core.cloud.completables.Completable
 import com.elementary.tasks.core.cloud.converters.Convertible
 import com.elementary.tasks.core.cloud.converters.IndexTypes
 import com.elementary.tasks.core.cloud.repositories.Repository
 import com.elementary.tasks.core.cloud.storages.Storage
 import com.elementary.tasks.core.utils.datetime.DateTimeManager
+import com.github.naz013.logging.Logger
 import org.koin.core.component.KoinComponent
 import org.koin.core.component.inject
 
@@ -28,7 +28,7 @@ class DataFlow<T>(
     val stream = convertible.toOutputStream(item) ?: return
     val metadata = convertible.metadata(item)
     storage.backup(stream, metadata)
-    Traces.log("Backed up file with ext = ${metadata.fileExt} and id = ${metadata.id}")
+    Logger.i("Backed up file with ext = ${metadata.fileExt} and id = ${metadata.id}")
   }
 
   suspend fun restore(id: String, type: IndexTypes) {
@@ -47,7 +47,7 @@ class DataFlow<T>(
       true
     }
     if (needUpdate) {
-      Traces.log("Saved remote file with ext = ${metadata.fileExt} and id = $id")
+      Logger.i("Saved remote file with ext = ${metadata.fileExt} and id = $id")
       repository.insert(item)
       completable?.action(item)
     }
@@ -58,7 +58,7 @@ class DataFlow<T>(
     if (id.isEmpty() || fileName.isEmpty()) {
       return
     }
-    Traces.log("Delete file with type = $type and id = $id")
+    Logger.i("Delete file with type = $type and id = $id")
     runCatching {
       val t = repository.get(id)
       if (t != null) {