refactor: sim toolkit

app-unpriv/src/main/java/im/angry/openeuicc/ui/UnprivilegedEuiccManagementFragment.kt

@@ -24,10 +24,8 @@ class UnprivilegedEuiccManagementFragment : EuiccManagementFragment() {
         super.onCreateOptionsMenu(menu, inflater)
         inflater.inflate(R.menu.fragment_sim_toolkit, menu)
         menu.findItem(R.id.open_sim_toolkit).apply {
-            val slot = stk[slotId]
+            isVisible = stk.isAvailable(slotId)
-            if (slot == null) return@apply
+            setOnMenuItemClickListener { stk.launch(slotId) }
-            isVisible = true
-            setOnMenuItemClickListener { slot.launch() }
         }
     }
 }

app-unpriv/src/main/java/im/angry/openeuicc/util/SIMToolkit.kt

@@ -3,6 +3,7 @@ package im.angry.openeuicc.util
 import android.content.ComponentName
 import android.content.Context
 import android.content.Intent
+import android.content.pm.ActivityInfo
 import android.content.pm.PackageManager
 import android.net.Uri
 import android.provider.Settings
@@ -20,37 +21,36 @@ class SIMToolkit(private val context: Context) {
         put(1, getComponentNames(R.array.sim_toolkit_slot_2))
     }
 
-    operator fun get(slotId: Int): Slot? = when (slotId) {
+    private val packageNames: Iterable<String>
-        -1, EuiccChannelManager.USB_CHANNEL_ID -> null
+        get() = slots.values.flatten().map { it.packageName }.toSet()
-        else -> Slot(context, buildSet {
+
-            addAll(slots.getOrDefault(slotId, emptySet()))
+    private val activities: Iterable<ComponentName>
-            addAll(slots.getOrDefault(-1, emptySet()))
+        get() = packageNames.flatMap(context.packageManager::getActivities)
-        })
+            .filter { it.exported }
+            .map { ComponentName(it.packageName, it.name) }
+
+    private val launchIntent: Intent?
+        get() = packageNames.firstNotNullOfOrNull(context.packageManager::getLaunchIntent)
+
+    private fun getComponentsBySlotId(slotId: Int) = buildSet {
+        addAll(slots.getOrDefault(slotId, emptySet()))
+        addAll(slots.getOrDefault(-1, emptySet()))
     }
 
-    data class Slot(private val context: Context, private val components: Set<ComponentName>) {
+    private fun intentActivity(slotId: Int): Intent? {
-        private val packageNames: Iterable<String>
+        val component = getComponentsBySlotId(slotId).find(activities::contains)
-            get() = components.map { it.packageName }.toSet()
+            ?: return launchIntent
-
+        val disabled = try {
-        private val launchIntent: Intent?
+            isDisabledState(context.packageManager.getComponentEnabledSetting(component))
-            get() = packageNames.firstNotNullOfOrNull(context.packageManager::getLaunchIntent)
+        } catch (e: IllegalArgumentException) {
-
+            true
-        private val activities: Iterable<ComponentName>
-            get() = packageNames.flatMap(context.packageManager::getActivities)
-                .filter { it.exported }.map { ComponentName(it.packageName, it.name) }
-
-        private fun getIntent(): Intent? {
-            try {
-                val component = components.find(activities::contains) ?: return launchIntent
-                if (isDisabledState(context.packageManager.getComponentEnabledSetting(component)))
-                    return null
-                return Intent.makeMainActivity(component)
-            } catch (e: IllegalArgumentException) {
-                return null
-            }
         }
+        if (disabled) return null
+        return Intent.makeMainActivity(component)
+    }
 
-        private fun getDisabledPackageName() = packageNames.find {
+    private fun getDisabledPackageName(slotId: Int) =
+        getComponentsBySlotId(slotId).map { it.packageName }.toSet().find {
             try {
                 isDisabledState(context.packageManager.getApplicationEnabledSetting(it))
             } catch (e: IllegalArgumentException) {
@@ -58,23 +58,28 @@ class SIMToolkit(private val context: Context) {
             }
         }
 
-        fun launch(): Boolean {
+    fun isAvailable(slotId: Int) = when (slotId) {
-            var intent = getIntent()
+        -1 -> false
-            if (intent == null) {
+        EuiccChannelManager.USB_CHANNEL_ID -> false
-                val pkgName = getDisabledPackageName() ?: return false
+        else -> intentActivity(slotId) != null || getDisabledPackageName(slotId) != null
-                val message = context.getString(
+    }
-                    R.string.toast_prompt_to_enable_sim_toolkit,
+
-                    context.packageManager.getApplicationLabel(pkgName)
+    fun launch(slotId: Int): Boolean {
-                )
+        var intent = intentActivity(slotId)
-                intent = Intent(
+        if (intent == null) {
-                    Settings.ACTION_APPLICATION_DETAILS_SETTINGS,
+            val pkgName = getDisabledPackageName(slotId) ?: return false
-                    Uri.fromParts("package", pkgName, null)
+            val message = context.getString(
-                )
+                R.string.toast_prompt_to_enable_sim_toolkit,
-                Toast.makeText(context, message, Toast.LENGTH_LONG).show()
+                context.packageManager.getApplicationLabel(pkgName)
-            }
+            )
-            context.startActivity(intent)
+            intent = Intent(
-            return true
+                Settings.ACTION_APPLICATION_DETAILS_SETTINGS,
+                Uri.fromParts("package", pkgName, null)
+            )
+            Toast.makeText(context, message, Toast.LENGTH_LONG).show()
         }
+        context.startActivity(intent)
+        return true
     }
 }
 
@@ -97,5 +102,5 @@ private fun PackageManager.getActivities(packageName: String) = try {
     getPackageInfo(packageName, PackageManager.GET_ACTIVITIES)
         .activities?.toList() ?: emptyList()
 } catch (_: PackageManager.NameNotFoundException) {
-    emptyList()
+    emptyList<ActivityInfo>()
 }