Version bump (1.0.1)

README.md

@@ -5,7 +5,9 @@
 
 A companion app that adds support for AirPod specific features to Android.
 
-Supported models:
+<img src="https://github.com/d4rken/capod/raw/master/.assets/banner.png" width="400">
+
+Currently supported models:
 
 * AirPods Gen1
 * AirPods Gen2
@@ -21,8 +23,8 @@ Supported models:
 ## Support the project
 
 * Buy the CAPod Pro In-App purchase on [Google Play](https://play.google.com/store/apps/details?id=eu.darken.capod)
-* [Buy me a coffee](https://www.buymeacoffee.com/tydarken)
 * Help translate CAPod [on Crowdin](https://crowdin.com/project/airpod-companion)
+* [Buy me a coffee](https://www.buymeacoffee.com/tydarken)
 
 ## Download
 
@@ -36,6 +38,11 @@ Supported models:
 
 ## Screenshots
 
+<img src="https://github.com/d4rken/capod/raw/master/.assets/screenshots/1.png" width="200">
+<img src="https://github.com/d4rken/capod/raw/master/.assets/screenshots/2.png" width="200">
+<img src="https://github.com/d4rken/capod/raw/master/.assets/screenshots/3.png" width="200">
+<img src="https://github.com/d4rken/capod/raw/master/.assets/screenshots/4.png" width="200">
+
 ## Thanks to
 
 * The [OpenPods](https://github.com/adolfintel/OpenPods) project,
@@ -49,8 +56,9 @@ Supported models:
 
 CAPod's code is available under a GPL v3 license, this excludes:
 
-* CAPod icons, logos, mascots and marketing materials.
+* CAPod icons, logos, mascots and marketing materials/assets.
 * CAPod animations and videos.
 * CAPod documentation.
 * Google Play screenshots.
 * Google Play texts & descriptions.
+* Translations.

app/src/gplay/java/eu/darken/capod/common/upgrade/core/client/BillingClientConnectionProvider.kt

@@ -14,10 +14,7 @@ import eu.darken.capod.common.flow.setupCommonEventHandlers
 import kotlinx.coroutines.CancellationException
 import kotlinx.coroutines.channels.awaitClose
 import kotlinx.coroutines.delay
-import kotlinx.coroutines.flow.Flow
-import kotlinx.coroutines.flow.MutableStateFlow
-import kotlinx.coroutines.flow.callbackFlow
-import kotlinx.coroutines.flow.retryWhen
+import kotlinx.coroutines.flow.*
 import javax.inject.Inject
 import javax.inject.Singleton
 import kotlin.coroutines.resume
@@ -112,8 +109,11 @@ class BillingClientConnectionProvider @Inject constructor(
             delay(3000 * attempt)
             true
         }
+        .catch {
+            log(TAG, ERROR) { "Unable to provide client connection:\n${it.asLog()}" }
+        }
 
     companion object {
-        val TAG: String = logTag("Upgrade", "Gplay", "Billing", "ClientProvider")
+        val TAG: String = logTag("Upgrade", "Gplay", "Billing", "Client", "ConnectionProvider")
     }
 }

app/src/main/java/eu/darken/capod/main/core/PermissionTool.kt

@@ -1,20 +1,43 @@
 package eu.darken.capod.main.core
 
 import android.content.Context
-import dagger.Reusable
 import dagger.hilt.android.qualifiers.ApplicationContext
+import eu.darken.capod.common.debug.logging.log
+import eu.darken.capod.common.debug.logging.logTag
 import eu.darken.capod.common.permissions.Permission
 import eu.darken.capod.common.permissions.isRequired
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.combine
+import kotlinx.coroutines.flow.onEach
+import java.util.*
 import javax.inject.Inject
+import javax.inject.Singleton
 
-@Reusable
+@Singleton
 class PermissionTool @Inject constructor(
     @ApplicationContext private val context: Context,
     private val generalSettings: GeneralSettings,
 ) {
+    private val permissionCheckTrigger = MutableStateFlow(UUID.randomUUID())
 
-    suspend fun missingPermissions(): Set<Permission> = Permission.values()
-        .filter { it != Permission.IGNORE_BATTERY_OPTIMIZATION || generalSettings.monitorMode.value == MonitorMode.ALWAYS }
-        .filter { it.isRequired(context) }
-        .toSet()
+    fun recheck() {
+        log(TAG) { "recheck()" }
+        permissionCheckTrigger.value = UUID.randomUUID()
+    }
+
+    val missingPermissions: Flow<Set<Permission>> = combine(
+        permissionCheckTrigger,
+        generalSettings.monitorMode.flow,
+    ) { _, monitorMode ->
+        Permission.values()
+            .filter { it != Permission.IGNORE_BATTERY_OPTIMIZATION || monitorMode == MonitorMode.ALWAYS }
+            .filter { it.isRequired(context) }
+            .toSet()
+    }
+        .onEach { log(TAG) { "Missing permission: $it" } }
+
+    companion object {
+        private val TAG = logTag("PermissionTool")
+    }
 }

app/src/main/java/eu/darken/capod/main/ui/overview/OverviewAdapter.kt

@@ -11,8 +11,8 @@ import eu.darken.capod.common.lists.differ.setupDiffer
 import eu.darken.capod.common.lists.modular.ModularAdapter
 import eu.darken.capod.common.lists.modular.mods.DataBinderMod
 import eu.darken.capod.common.lists.modular.mods.TypedVHCreatorMod
+import eu.darken.capod.main.ui.overview.cards.BluetoothDisabledVH
 import eu.darken.capod.main.ui.overview.cards.MissingMainDeviceVH
-import eu.darken.capod.main.ui.overview.cards.NoPairedDeviceCardVH
 import eu.darken.capod.main.ui.overview.cards.PermissionCardVH
 import eu.darken.capod.main.ui.overview.cards.pods.BasicSingleApplePodsCardVH
 import eu.darken.capod.main.ui.overview.cards.pods.DualApplePodsCardVH
@@ -33,8 +33,8 @@ class OverviewAdapter @Inject constructor() :
         modules.add(TypedVHCreatorMod({ data[it] is SingleApplePodsCardVH.Item }) { SingleApplePodsCardVH(it) })
         modules.add(TypedVHCreatorMod({ data[it] is BasicSingleApplePodsCardVH.Item }) { BasicSingleApplePodsCardVH(it) })
         modules.add(TypedVHCreatorMod({ data[it] is UnknownPodDeviceCardVH.Item }) { UnknownPodDeviceCardVH(it) })
-        modules.add(TypedVHCreatorMod({ data[it] is NoPairedDeviceCardVH.Item }) { NoPairedDeviceCardVH(it) })
         modules.add(TypedVHCreatorMod({ data[it] is MissingMainDeviceVH.Item }) { MissingMainDeviceVH(it) })
+        modules.add(TypedVHCreatorMod({ data[it] is BluetoothDisabledVH.Item }) { BluetoothDisabledVH(it) })
     }
 
     override fun getItemCount(): Int = data.size

app/src/main/java/eu/darken/capod/main/ui/overview/OverviewFragment.kt

@@ -70,8 +70,10 @@ class OverviewFragment : Fragment3(R.layout.main_fragment) {
             }
         }
 
-        vm.listItems.observe2(ui) {
-            adapter.update(it)
+        vm.listItems.observe2(ui) { adapter.update(it) }
+
+        vm.workerAutolaunch.observe2 {
+            // While UI is active, subscribe to the autolaunch routine
         }
 
         vm.requestPermissionEvent.observe2(ui) {

app/src/main/java/eu/darken/capod/main/ui/overview/OverviewFragmentVM.kt

@@ -8,6 +8,7 @@ import eu.darken.capod.common.bluetooth.BluetoothManager2
 import eu.darken.capod.common.coroutine.DispatcherProvider
 import eu.darken.capod.common.debug.autoreport.DebugSettings
 import eu.darken.capod.common.debug.logging.log
+import eu.darken.capod.common.flow.combine
 import eu.darken.capod.common.livedata.SingleLiveEvent
 import eu.darken.capod.common.navigation.navVia
 import eu.darken.capod.common.permissions.Permission
@@ -16,8 +17,8 @@ import eu.darken.capod.common.upgrade.UpgradeRepo
 import eu.darken.capod.main.core.GeneralSettings
 import eu.darken.capod.main.core.MonitorMode
 import eu.darken.capod.main.core.PermissionTool
+import eu.darken.capod.main.ui.overview.cards.BluetoothDisabledVH
 import eu.darken.capod.main.ui.overview.cards.MissingMainDeviceVH
-import eu.darken.capod.main.ui.overview.cards.NoPairedDeviceCardVH
 import eu.darken.capod.main.ui.overview.cards.PermissionCardVH
 import eu.darken.capod.main.ui.overview.cards.pods.*
 import eu.darken.capod.monitor.core.PodMonitor
@@ -26,7 +27,6 @@ import eu.darken.capod.pods.core.PodDevice
 import eu.darken.capod.pods.core.apple.BasicSingleApplePods
 import eu.darken.capod.pods.core.apple.DualApplePods
 import eu.darken.capod.pods.core.apple.SingleApplePods
-import kotlinx.coroutines.channels.awaitClose
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.flow.*
 import kotlinx.coroutines.isActive
@@ -57,9 +57,7 @@ class OverviewFragmentVM @Inject constructor(
         }
     }
 
-    private val permissionCheckTrigger = MutableStateFlow(UUID.randomUUID())
-    private val requiredPermissions: Flow<Collection<Permission>> = permissionCheckTrigger
-        .map { permissionTool.missingPermissions() }
+    val workerAutolaunch: LiveData<Unit> = permissionTool.missingPermissions
         .onEach {
             if (it.isNotEmpty()) {
                 log(TAG) { "Missing permissions: $it" }
@@ -76,26 +74,24 @@ class OverviewFragmentVM @Inject constructor(
                 monitorControl.startMonitor()
             }
         }
+        .map { Unit }
+        .asLiveData2()
 
     val requestPermissionEvent = SingleLiveEvent<Permission>()
 
-    private val pods: Flow<List<PodDevice>> = requiredPermissions
+    private val pods: Flow<List<PodDevice>> = permissionTool.missingPermissions
         .flatMapLatest { permissions ->
-            if (permissions.isEmpty()) {
-                generalSettings.showAll.flow
-                    .flatMapLatest { showAll ->
-                        if (showAll) {
-                            podMonitor.devices
-                        } else {
-                            podMonitor.mainDevice.map { mainDevice ->
-                                mainDevice?.let { listOf(it) } ?: emptyList()
-                            }
-                        }
+            if (permissions.isNotEmpty()) {
+                return@flatMapLatest flowOf(emptyList())
+            }
+
+            generalSettings.showAll.flow.flatMapLatest { showAll ->
+                if (showAll) {
+                    podMonitor.devices
+                } else {
+                    podMonitor.mainDevice.map { mainDevice ->
+                        mainDevice?.let { listOf(it) } ?: emptyList()
                     }
-            } else {
-                channelFlow {
-                    send(emptyList())
-                    awaitClose()
                 }
             }
         }
@@ -103,11 +99,12 @@ class OverviewFragmentVM @Inject constructor(
 
     val listItems: LiveData<List<OverviewAdapter.Item>> = combine(
         updateTicker,
-        requiredPermissions,
+        permissionTool.missingPermissions,
         pods,
         debugSettings.isDebugModeEnabled.flow,
         generalSettings.showAll.flow,
-    ) { tick, permissions, pods, isDebugMode, showAll ->
+        bluetoothManager.isBluetoothEnabled,
+    ) { _, permissions, pods, isDebugMode, showAll, isBluetoothEnabled ->
         val items = mutableListOf<OverviewAdapter.Item>()
 
         val mainPod = podMonitor.mainDevice.first()
@@ -144,20 +141,20 @@ class OverviewFragmentVM @Inject constructor(
             .run { items.addAll(this) }
 
         permissions
-            .map {
+            .map { perm ->
                 PermissionCardVH.Item(
-                    permission = it,
+                    permission = perm,
                     onRequest = { requestPermissionEvent.postValue(it) }
                 )
             }
             .run { items.addAll(this) }
 
-        if (!showAll && items.none { it is PodDeviceVH.Item } && permissions.isEmpty()) {
-            NoPairedDeviceCardVH.Item {
-                generalSettings.showAll.value = true
-            }.run { items.add(this) }
-        } else if (showAll && mainPod == null && permissions.isEmpty()) {
-            items.add(0, MissingMainDeviceVH.Item)
+        if (permissions.isEmpty()) {
+            if (!isBluetoothEnabled) {
+                items.add(0, BluetoothDisabledVH.Item)
+            } else if (mainPod == null) {
+                items.add(0, MissingMainDeviceVH.Item)
+            }
         }
 
         items
@@ -166,7 +163,7 @@ class OverviewFragmentVM @Inject constructor(
         .asLiveData2()
 
     fun onPermissionResult(granted: Boolean) {
-        if (granted) permissionCheckTrigger.value = UUID.randomUUID()
+        if (granted) permissionTool.recheck()
     }
 
     fun goToSettings() = launch {

app/src/main/java/eu/darken/capod/main/ui/overview/cards/BluetoothDisabledVH.kt

@@ -4,29 +4,27 @@ import android.view.ViewGroup
 import eu.darken.capod.R
 import eu.darken.capod.common.lists.binding
 import eu.darken.capod.common.lists.differ.DifferItem
-import eu.darken.capod.databinding.OverviewNopaireddeviceItemBinding
+import eu.darken.capod.databinding.OverviewBluetoothDisabledItemBinding
 import eu.darken.capod.main.ui.overview.OverviewAdapter
 
-class NoPairedDeviceCardVH(parent: ViewGroup) :
-    OverviewAdapter.BaseVH<NoPairedDeviceCardVH.Item, OverviewNopaireddeviceItemBinding>(
-        R.layout.overview_nopaireddevice_item,
+class BluetoothDisabledVH(parent: ViewGroup) :
+    OverviewAdapter.BaseVH<BluetoothDisabledVH.Item, OverviewBluetoothDisabledItemBinding>(
+        R.layout.overview_bluetooth_disabled_item,
         parent
     ) {
 
     override val viewBinding = lazy {
-        OverviewNopaireddeviceItemBinding.bind(itemView)
+        OverviewBluetoothDisabledItemBinding.bind(itemView)
     }
 
-    override val onBindData: OverviewNopaireddeviceItemBinding.(
+    override val onBindData: OverviewBluetoothDisabledItemBinding.(
         item: Item,
         payloads: List<Any>
     ) -> Unit = binding(payload = true) { item ->
-        showAllAction.setOnClickListener { item.onShowAll() }
+
     }
 
-    data class Item(
-        val onShowAll: () -> Unit
-    ) : OverviewAdapter.Item {
+    object Item : OverviewAdapter.Item {
         override val stableId: Long = Item::class.hashCode().toLong()
 
         override val payloadProvider: ((DifferItem, DifferItem) -> DifferItem?)

app/src/main/java/eu/darken/capod/monitor/core/PodMonitor.kt

@@ -32,17 +32,16 @@ class PodMonitor @Inject constructor(
     private val deviceCache = mutableMapOf<PodDevice.Id, PodDevice>()
     private val cacheLock = Mutex()
 
-    val devices: Flow<List<PodDevice>> = generalSettings.scannerMode.flow
-        .flatMapLatest { scannerMode ->
-            bluetoothManager.isBluetoothEnabled.flatMapLatest { isBluetoothEnabled ->
-                if (isBluetoothEnabled) {
-                    bleScanner.scan(scannerMode = scannerMode)
-                } else {
-                    log(TAG, WARN) { "Bluetooth is current disabled" }
-                    emptyFlow()
-                }
+    val devices: Flow<List<PodDevice>> = bluetoothManager.isBluetoothEnabled
+        .flatMapLatest { isBluetoothEnabled ->
+            if (isBluetoothEnabled) {
+                generalSettings.scannerMode.flow
+            } else {
+                log(TAG, WARN) { "Bluetooth is currently disabled" }
+                emptyFlow()
             }
         }
+        .flatMapLatest { bleScanner.scan(scannerMode = it) }
         .map { result ->
             // For each address we only want the newest result, upstream may batch data
             result.groupBy { it.address }

app/src/main/java/eu/darken/capod/monitor/core/worker/MonitorWorker.kt

@@ -76,7 +76,7 @@ class MonitorWorker @AssistedInject constructor(
     }
 
     private suspend fun doDoWork() {
-        val permissionsMissingOnStart = permissionTool.missingPermissions()
+        val permissionsMissingOnStart = permissionTool.missingPermissions.first()
         if (permissionsMissingOnStart.isNotEmpty()) {
             log(TAG, WARN) { "Aborting, missing permissions: $permissionsMissingOnStart" }
             return
@@ -99,7 +99,7 @@ class MonitorWorker @AssistedInject constructor(
 
         generalSettings.monitorMode.flow
             .flatMapLatest { monitorMode ->
-                val missingPermsFlow = permissionTool.missingPermissions()
+                val missingPermsFlow = permissionTool.missingPermissions.first()
                 if (missingPermsFlow.isNotEmpty()) {
                     log(TAG, WARN) { "Aborting, permissions are missing for $monitorMode: $missingPermsFlow" }
                     workerScope.coroutineContext.cancelChildren()

app/src/main/res/layout/overview_bluetooth_disabled_item.xml

@@ -14,42 +14,30 @@
         android:layout_height="wrap_content">
 
         <com.google.android.material.textview.MaterialTextView
-            android:id="@+id/permission_label"
+            android:id="@+id/bluetooth_label"
             style="@style/TextAppearance.MaterialComponents.Body1"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_marginHorizontal="16dp"
             android:layout_marginTop="16dp"
+            android:text="@string/overview_bluetooth_disabled_label"
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintTop_toTopOf="parent"
-            android:text="@string/overview_nopaireddevices_label" />
+            app:layout_constraintTop_toTopOf="parent" />
 
         <com.google.android.material.textview.MaterialTextView
-            android:id="@+id/permission_description"
+            android:id="@+id/bluetooth_description"
             style="@style/TextAppearance.MaterialComponents.Body2"
             android:layout_width="match_parent"
+            android:layout_height="wrap_content"
+            android:layout_marginHorizontal="16dp"
             android:layout_marginTop="4dp"
-            android:layout_marginHorizontal="16dp"
-            android:layout_height="wrap_content"
-            app:layout_constraintEnd_toEndOf="parent"
-            app:layout_constraintStart_toStartOf="parent"
-            app:layout_constraintTop_toBottomOf="@id/permission_label"
-            android:text="@string/overview_nopaireddevices_description" />
-
-        <com.google.android.material.button.MaterialButton
-            android:id="@+id/show_all_action"
-            style="@style/Widget.MaterialComponents.Button"
-            android:layout_width="wrap_content"
-            android:layout_marginHorizontal="16dp"
-            android:layout_height="wrap_content"
-            android:layout_marginBottom="8dp"
-            android:layout_marginTop="16dp"
-            android:text="@string/settings_showall_label"
+            android:layout_marginBottom="16dp"
+            android:text="@string/overview_bluetooth_disabled_description"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="parent"
-            app:layout_constraintTop_toBottomOf="@id/permission_description" />
-
+            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/bluetooth_label" />
 
     </androidx.constraintlayout.widget.ConstraintLayout>
 </com.google.android.material.card.MaterialCardView>

app/src/main/res/values/strings.xml

@@ -71,9 +71,7 @@
     <string name="pods_microphone_label">Microphone</string>
     <string name="pods_yours">Yours</string>
 
-    <string name="overview_nopaireddevices_label">No paired device connected.</string>
-    <string name="overview_nopaireddevices_description">Connect a paired device or enable the \'Show all\' option.</string>
-    <string name="overview_nomaindevice_label">Missing main device</string>
+    <string name="overview_nomaindevice_label">No primary device</string>
     <string name="overview_nomaindevice_description">All detected devices are unlikely to be yours. Power on and connect your device or adjust the settings.</string>
 
     <string name="settings_label">Settings</string>
@@ -146,4 +144,6 @@
     <string name="settings_popup_caseopen_label">Show popup</string>
     <string name="settings_popup_caseopen_description">Show a popup when the device case is opened (experimental).</string>
     <string name="notification_channel_reaction_popup_label">Popup device reactions</string>
+    <string name="overview_bluetooth_disabled_label">Bluetooth is disabled</string>
+    <string name="overview_bluetooth_disabled_description">Bluetooth is disabled, enable it ;).</string>
 </resources>

build.gradle

@@ -8,7 +8,7 @@ buildscript {
             'version'   : [
                     'major': 1,
                     'minor': 0,
-                    'patch': 0,
+                    'patch': 1,
                     'build': 0,
             ],
     ]