Skip to content

Commit bd375c8

Browse files
committed
fix: use named parameters in wear DeviceRegistration calls
The positional args broke when pushUrl/pushEncrypt were added to DeviceRegistration. Boolean was being passed where String? was expected. Named parameters make this resilient to parameter order.
1 parent ed6d0fe commit bd375c8

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

wear/src/main/kotlin/io/homeassistant/companion/android/onboarding/integration/MobileAppIntegrationPresenterImpl.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,10 @@ class MobileAppIntegrationPresenterImpl @Inject constructor(
3131

3232
private suspend fun createRegistration(deviceName: String): DeviceRegistration {
3333
return DeviceRegistration(
34-
appVersionProvider(),
35-
deviceName,
36-
messagingTokenProvider(),
37-
false,
34+
appVersion = appVersionProvider(),
35+
deviceName = deviceName,
36+
pushToken = messagingTokenProvider(),
37+
pushWebsocket = false,
3838
)
3939
}
4040

wear/src/main/kotlin/io/homeassistant/companion/android/phone/PhoneSettingsListener.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -205,10 +205,10 @@ class PhoneSettingsListener :
205205
serverId = serverManager.addServer(temporaryServer)
206206
serverManager.integrationRepository(serverId).registerDevice(
207207
DeviceRegistration(
208-
appVersionProvider(),
209-
deviceName,
210-
messagingTokenProvider(),
211-
false,
208+
appVersion = appVersionProvider(),
209+
deviceName = deviceName,
210+
pushToken = messagingTokenProvider(),
211+
pushWebsocket = false,
212212
),
213213
)
214214
launch {

0 commit comments

Comments
 (0)