Skip to content

Commit 804919c

Browse files
committed
Ignore new returned value from database query which know have type QueryResult<Long>.
1 parent 3a2403e commit 804919c

File tree

1 file changed

+19
-15
lines changed
  • libraries/session-storage/impl/src/main/kotlin/io/element/android/libraries/sessionstorage/impl

1 file changed

+19
-15
lines changed

libraries/session-storage/impl/src/main/kotlin/io/element/android/libraries/sessionstorage/impl/DatabaseSessionStore.kt

Lines changed: 19 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -48,25 +48,29 @@ class DatabaseSessionStore @Inject constructor(
4848
}
4949
}
5050

51-
override suspend fun storeData(sessionData: SessionData) = sessionDataMutex.withLock {
52-
database.sessionDataQueries.insertSessionData(sessionData.toDbModel())
51+
override suspend fun storeData(sessionData: SessionData) {
52+
sessionDataMutex.withLock {
53+
database.sessionDataQueries.insertSessionData(sessionData.toDbModel())
54+
}
5355
}
5456

55-
override suspend fun updateData(sessionData: SessionData) = sessionDataMutex.withLock {
56-
val result = database.sessionDataQueries.selectByUserId(sessionData.userId)
57-
.executeAsOneOrNull()
58-
?.toApiModel()
57+
override suspend fun updateData(sessionData: SessionData) {
58+
sessionDataMutex.withLock {
59+
val result = database.sessionDataQueries.selectByUserId(sessionData.userId)
60+
.executeAsOneOrNull()
61+
?.toApiModel()
5962

60-
if (result == null) {
61-
Timber.e("User ${sessionData.userId} not found in session database")
62-
return
63-
}
63+
if (result == null) {
64+
Timber.e("User ${sessionData.userId} not found in session database")
65+
return
66+
}
6467
// Copy new data from SDK, but keep login timestamp
65-
database.sessionDataQueries.updateSession(
66-
sessionData.copy(
67-
loginTimestamp = result.loginTimestamp,
68-
).toDbModel()
69-
)
68+
database.sessionDataQueries.updateSession(
69+
sessionData.copy(
70+
loginTimestamp = result.loginTimestamp,
71+
).toDbModel()
72+
)
73+
}
7074
}
7175

7276
override suspend fun getLatestSession(): SessionData? {

0 commit comments

Comments
 (0)