Skip to content

Commit 54b225f

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 85fc6bd commit 54b225f

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
@@ -210,10 +210,10 @@ class PhoneSettingsListener :
210210
serverId = serverManager.addServer(temporaryServer)
211211
serverManager.integrationRepository(serverId).registerDevice(
212212
DeviceRegistration(
213-
appVersionProvider(),
214-
deviceName,
215-
messagingTokenProvider(),
216-
false,
213+
appVersion = appVersionProvider(),
214+
deviceName = deviceName,
215+
pushToken = messagingTokenProvider(),
216+
pushWebsocket = false,
217217
),
218218
)
219219
launch {

0 commit comments

Comments
 (0)