More fixes based on feedback

mullvad · Sep 16, 2024 · 7b0c6e5 · 7b0c6e5
1 parent ff77d4d
commit 7b0c6e5
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 10 deletions.
diff --git a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/constant/WireguardConstant.kt b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/constant/WireguardConstant.kt
@@ -7,4 +7,5 @@ val WIREGUARD_PRESET_PORTS = listOf(Port(51820), Port(53))
 val UDP2TCP_PRESET_PORTS = listOf(Port(80), Port(5001))
 val SHADOWSOCKS_PRESET_PORTS = emptyList<Port>()
 val SHADOWSOCKS_AVAILABLE_PORTS =
-    listOf(PortRange(IntRange(0, 65535))) // Currently we consider all ports to be available
+    // Currently we consider all ports to be available
+    listOf(PortRange(IntRange(Port.MIN_VALUE, Port.MAX_VALUE)))
diff --git a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/viewmodel/ShadowsocksSettingsViewModel.kt b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/viewmodel/ShadowsocksSettingsViewModel.kt
@@ -50,7 +50,7 @@ class ShadowsocksSettingsViewModel(
             val initialSettings = settingsRepository.settingsUpdates.filterNotNull().first()
             customPort.update {
                 val initialPort = initialSettings.getShadowSocksPort()
-                if (SHADOWSOCKS_PRESET_PORTS.contains(initialPort.getOrNull()).not()) {
+                if (initialPort.getOrNull() !in SHADOWSOCKS_PRESET_PORTS) {
                     initialPort.getOrNull()
                 } else {
                     null
@@ -65,10 +65,7 @@ class ShadowsocksSettingsViewModel(
                 .setCustomShadowsocksObfuscationPort(port)
                 .onLeft { Logger.e("Select shadowsocks port error $it") }
                 .onRight {
-                    if (
-                        port is Constraint.Only &&
-                            SHADOWSOCKS_PRESET_PORTS.contains(port.value).not()
-                    ) {
+                    if (port is Constraint.Only && port.value !in SHADOWSOCKS_PRESET_PORTS) {
                         customPort.update { port.getOrNull() }
                     }
                 }

diff --git a/android/app/src/main/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModel.kt b/android/app/src/main/kotlin/net/mullvad/mullvadvpn/viewmodel/VpnSettingsViewModel.kt
@@ -101,7 +101,7 @@ class VpnSettingsViewModel(
             val initialSettings = repository.settingsUpdates.filterNotNull().first()
             customPort.update {
                 val initialPort = initialSettings.getWireguardPort()
-                if (WIREGUARD_PRESET_PORTS.contains(initialPort.getOrNull()).not()) {
+                if (initialPort.getOrNull() !in WIREGUARD_PRESET_PORTS) {
                     initialPort.getOrNull()
                 } else {
                     null
@@ -232,7 +232,7 @@ class VpnSettingsViewModel(
     }
 
     fun onWireguardPortSelected(port: Constraint<Port>) {
-        if (port is Constraint.Only && WIREGUARD_PRESET_PORTS.contains(port.value).not()) {
+        if (port is Constraint.Only && port.value !in WIREGUARD_PRESET_PORTS) {
             customPort.update { port.value }
         }
         viewModelScope.launch {

diff --git a/android/lib/model/src/main/kotlin/net/mullvad/mullvadvpn/lib/model/Port.kt b/android/lib/model/src/main/kotlin/net/mullvad/mullvadvpn/lib/model/Port.kt
@@ -19,8 +19,8 @@ value class Port(val value: Int) : Parcelable {
             Port(number)
         }
 
-        private const val MIN_VALUE = 0
-        private const val MAX_VALUE = 65535
+        const val MIN_VALUE = 0
+        const val MAX_VALUE = 65535
     }
 }