app-common/src/main/java/im/angry/openeuicc/ui/NotificationsActivity.kt

@@ -164,14 +164,14 @@ class NotificationsActivity: BaseEuiccAccessActivity(), OpenEuiccContextMarker {
         }
 
 
-        private fun operationToLocalizedText(operation: ProfileManagementOperation?) =
+        private fun operationToLocalizedText(operation: ProfileManagementOperation) =
-            root.context.getString(when (operation) {
+            root.context.getText(
-                ProfileManagementOperation.Install -> R.string.profile_notification_operation_download
+                when (operation) {
-                ProfileManagementOperation.Delete -> R.string.profile_notification_operation_delete
+                    ProfileManagementOperation.Install -> R.string.profile_notification_operation_download
-                ProfileManagementOperation.Enable -> R.string.profile_notification_operation_enable
+                    ProfileManagementOperation.Delete -> R.string.profile_notification_operation_delete
-                ProfileManagementOperation.Disable -> R.string.profile_notification_operation_disable
+                    ProfileManagementOperation.Enable -> R.string.profile_notification_operation_enable
-                else -> throw IllegalStateException("Unknown operation: $operation")
+                    ProfileManagementOperation.Disable -> R.string.profile_notification_operation_disable
-            })
+                })
 
         fun updateNotification(value: LocalProfileNotificationWrapper) {
             notification = value

libs/lpac-jni/src/main/java/net/typeblog/lpac_jni/EuiccConfiguredAddresses.kt

@@ -9,21 +9,21 @@ private val invalidDPAddresses = setOf(
     "testrootsmds.example.com",
 )
 
-data class EuiccConfiguredAddresses(
+class EuiccConfiguredAddresses(defaultDPAddress: String, rootDSAddress: String) {
-    val defaultDPAddress: String,
+    val defaultDPAddress = defaultDPAddress.takeUnless(::isInvalidDPAddress)
-    val rootDSAddress: String
+    val rootDSAddress = rootDSAddress.takeUnless(::isInvalidDSAddress)
-) {
+
     val discoverable: Boolean
-        get() = isValidDPAddress(defaultDPAddress) ||
+        get() = !defaultDPAddress.isNullOrBlank() || !rootDSAddress.isNullOrBlank()
-                isValidDSAddress(rootDSAddress)
 }
 
-private fun isValidDPAddress(address: String): Boolean {
+private fun isInvalidDPAddress(address: String?): Boolean {
-    return address.isNotBlank() && Patterns.DOMAIN_NAME.matcher(address).matches()
+    if (address.isNullOrBlank()) return true
+    return !Patterns.DOMAIN_NAME.matcher(address).matches()
 }
 
-private fun isValidDSAddress(address: String): Boolean {
+private fun isInvalidDSAddress(address: String?): Boolean {
-    if (address.isBlank()) return false
+    if (address.isNullOrBlank()) return true
-    if (address in invalidDPAddresses) return false
+    if (address in invalidDPAddresses) return true
-    return Patterns.DOMAIN_NAME.matcher(address).matches()
+    return !Patterns.DOMAIN_NAME.matcher(address).matches()
 }

libs/lpac-jni/src/main/java/net/typeblog/lpac_jni/LocalProfileNotification.kt

@@ -2,7 +2,7 @@ package net.typeblog.lpac_jni
 
 data class LocalProfileNotification(
     val seqNumber: Long,
-    val profileManagementOperation: ProfileManagementOperation?,
+    val profileManagementOperation: ProfileManagementOperation,
     val notificationAddress: String,
     val iccid: String,
 )