Remove share persisted log if file logger is disabled

Telefonica · Dec 12, 2024 · a9cb4db · a9cb4db
1 parent 49457ba
commit a9cb4db
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/library/src/enabled/java/com/telefonica/androidlogger/domain/AppLogger.kt b/library/src/enabled/java/com/telefonica/androidlogger/domain/AppLogger.kt
@@ -41,6 +41,9 @@ internal fun getLogs(): LiveData<List<LogEntry>> =
 internal fun getCategories(): List<LogCategory> =
     appLoggerBLInstance?.categories ?: emptyList()
 
+internal fun arePersistedLogsEnabled() : Boolean =
+    appLoggerBLInstance?.arePersistedLogsEnabled() ?: false
+
 internal fun getPersistedLogs(callback: TaskCallback<Uri>) {
     appLoggerBLInstance?.getPersistedLogs(callback)
 }

diff --git a/library/src/enabled/java/com/telefonica/androidlogger/domain/AppLoggerBL.kt b/library/src/enabled/java/com/telefonica/androidlogger/domain/AppLoggerBL.kt
@@ -54,6 +54,9 @@ internal open class AppLoggerBL(
     open fun getLogs(): LiveData<List<LogEntry>> =
             logsData
 
+    open fun arePersistedLogsEnabled(): Boolean =
+        fileLogger != null
+
     open fun getPersistedLogs(callback: TaskCallback<Uri>) {
         fileLogger?.getReport(callback)
     }

diff --git a/library/src/enabled/java/com/telefonica/androidlogger/ui/AppLoggerActivity.kt b/library/src/enabled/java/com/telefonica/androidlogger/ui/AppLoggerActivity.kt
@@ -32,6 +32,7 @@ import com.google.android.material.chip.Chip
 import com.google.android.material.chip.ChipGroup
 import com.telefonica.androidlogger.R
 import com.telefonica.androidlogger.domain.LogCategory
+import com.telefonica.androidlogger.domain.arePersistedLogsEnabled
 import com.telefonica.androidlogger.io.executor.TaskCallback
 import com.telefonica.androidlogger.ui.adapter.LogListAdapter
 import com.telefonica.androidlogger.ui.viewmodel.AppLoggerViewModel
@@ -79,7 +80,9 @@ class AppLoggerActivity : AppCompatActivity() {
 
     override fun onCreateOptionsMenu(menu: Menu): Boolean {
         menuInflater.inflate(R.menu.menu_app_logger, menu)
-
+        if (!arePersistedLogsEnabled()) {
+            menu.removeItem(R.id.action_share_all)
+        }
         searchView = (menu.findItem(R.id.action_search).actionView as SearchView).apply {
             val searchManager = getSystemService(Context.SEARCH_SERVICE) as SearchManager
             setSearchableInfo(searchManager.getSearchableInfo(componentName))