Forgot to use application scope (#257)

JackEblan · Dec 15, 2024 · fa2bcbc · fa2bcbc
1 parent 753e959
commit fa2bcbc
Show file tree

Hide file tree

Showing 8 changed files with 18 additions and 18 deletions.
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -41,11 +41,4 @@
 
     </application>
 
-    <queries>
-        <intent>
-            <action android:name="android.intent.action.MAIN" />
-            <category android:name="android.intent.category.LAUNCHER" />
-        </intent>
-    </queries>
-
 </manifest>
diff --git a/.../src/main/kotlin/com/android/geto/broadcastreceiver/DefaultBroadcastReceiverController.kt b/.../src/main/kotlin/com/android/geto/broadcastreceiver/DefaultBroadcastReceiverController.kt
@@ -17,28 +17,26 @@
  */
 package com.android.geto.broadcastreceiver
 
-import com.android.geto.core.common.Dispatcher
-import com.android.geto.core.common.GetoDispatchers.IO
+import com.android.geto.core.common.di.ApplicationScope
 import com.android.geto.core.domain.broadcastreceiver.BroadcastReceiverController
 import com.android.geto.core.domain.foregroundservice.UsageStatsForegroundServiceManager
 import com.android.geto.core.domain.framework.NotificationManagerWrapper
 import com.android.geto.core.domain.model.AppSettingsResult
 import com.android.geto.core.domain.usecase.RevertAppSettingsUseCase
-import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.launch
 import javax.inject.Inject
 
 internal class DefaultBroadcastReceiverController @Inject constructor(
-    @Dispatcher(IO) private val ioDispatcher: CoroutineDispatcher,
+    @ApplicationScope private val appScope: CoroutineScope,
     private val revertAppSettingsUseCase: RevertAppSettingsUseCase,
     private val notificationManagerWrapper: NotificationManagerWrapper,
     private val usageStatsForegroundServiceManager: UsageStatsForegroundServiceManager,
 ) : BroadcastReceiverController {
     override fun revertSettings(packageName: String?, notificationId: Int?) {
         if (packageName == null || notificationId == null) return
 
-        CoroutineScope(ioDispatcher).launch {
+        appScope.launch {
             if (revertAppSettingsUseCase(packageName = packageName) == AppSettingsResult.Success) {
                 notificationManagerWrapper.cancel(notificationId)
             }

diff --git a/feature/shizuku/src/main/kotlin/com/android/geto/feature/shizuku/ShizukuEvent.kt b/feature/shizuku/src/main/kotlin/com/android/geto/feature/shizuku/ShizukuEvent.kt
@@ -18,7 +18,7 @@
 package com.android.geto.feature.shizuku
 
 sealed interface ShizukuEvent {
-    data object CheckShizukuShizuku : ShizukuEvent
+    data object CheckShizukuPermission : ShizukuEvent
 
     data object OnCreate : ShizukuEvent
 

diff --git a/feature/shizuku/src/main/kotlin/com/android/geto/feature/shizuku/ShizukuScreen.kt b/feature/shizuku/src/main/kotlin/com/android/geto/feature/shizuku/ShizukuScreen.kt
@@ -195,7 +195,7 @@ internal fun ShizukuScreen(
     Scaffold(
         topBar = {
             ShizukuTopAppBar(
-                title = "Shizuku",
+                title = stringResource(R.string.shizuku),
                 onNavigationIconClick = onNavigationIconClick,
             )
         },
@@ -216,7 +216,7 @@ internal fun ShizukuScreen(
                 modifier = modifier.fillMaxSize(),
                 active = shizukuStatus == ShizukuStatus.CanWriteSecureSettings,
                 onClick = {
-                    onEvent(ShizukuEvent.CheckShizukuShizuku)
+                    onEvent(ShizukuEvent.CheckShizukuPermission)
                 },
             )
         }

diff --git a/feature/shizuku/src/main/kotlin/com/android/geto/feature/shizuku/ShizukuViewModel.kt b/feature/shizuku/src/main/kotlin/com/android/geto/feature/shizuku/ShizukuViewModel.kt
@@ -37,7 +37,7 @@ class ShizukuViewModel @Inject constructor(private val shizukuWrapper: ShizukuWr
 
     fun onEvent(event: ShizukuEvent) {
         when (event) {
-            ShizukuEvent.CheckShizukuShizuku -> {
+            ShizukuEvent.CheckShizukuPermission -> {
                 shizukuWrapper.checkShizukuPermission()
             }
 

diff --git a/feature/shizuku/src/main/res/values/strings.xml b/feature/shizuku/src/main/res/values/strings.xml
@@ -26,4 +26,5 @@
     <string name="something_went_wrong_with_the_request">Something went wrong with the request</string>
     <string name="please_check_if_shizuku_is_properly_running">Please check if Shizuku is properly running</string>
     <string name="shizuku_dead_binder">Shizuku dead binder</string>
+    <string name="shizuku">Shizuku</string>
 </resources>
diff --git a/...in/kotlin/com/android/geto/foregroundservice/AndroidUsageStatsForegroundServiceManager.kt b/...in/kotlin/com/android/geto/foregroundservice/AndroidUsageStatsForegroundServiceManager.kt
@@ -80,6 +80,8 @@ class AndroidUsageStatsForegroundServiceManager @Inject constructor(@Application
 
         _isBound = false
 
-        _isActive.tryEmit(context.stopService(usageStatsServiceIntent).not())
+        context.stopService(usageStatsServiceIntent)
+
+        _isActive.tryEmit(false)
     }
 }
diff --git a/framework/package-manager/src/main/AndroidManifest.xml b/framework/package-manager/src/main/AndroidManifest.xml
@@ -15,6 +15,12 @@
   ~   limitations under the License.
   ~
   -->
-<manifest>
+<manifest xmlns:android="http://schemas.android.com/apk/res/android">
 
+    <queries>
+        <intent>
+            <action android:name="android.intent.action.MAIN" />
+            <category android:name="android.intent.category.LAUNCHER" />
+        </intent>
+    </queries>
 </manifest>