Do not use System.currentTimeMillis()

This commit is contained in:
Benoit Marty 2023-06-01 12:27:11 +02:00
parent cede53579c
commit 42bafbdc1a

View file

@ -29,6 +29,7 @@ import io.element.android.libraries.push.impl.log.pushLoggerTag
import io.element.android.libraries.push.impl.notifications.model.NotifiableEvent
import io.element.android.libraries.push.impl.notifications.model.NotifiableMessageEvent
import io.element.android.services.toolbox.api.strings.StringProvider
import io.element.android.services.toolbox.api.systemclock.SystemClock
import timber.log.Timber
import javax.inject.Inject
@ -46,6 +47,7 @@ class NotifiableEventResolver @Inject constructor(
// private val displayableEventFormatter: DisplayableEventFormatter,
private val matrixAuthenticationService: MatrixAuthenticationService,
private val buildMeta: BuildMeta,
private val clock: SystemClock,
) {
suspend fun resolveEvent(sessionId: SessionId, roomId: RoomId, eventId: EventId): NotifiableEvent? {
@ -96,7 +98,6 @@ class NotifiableEventResolver @Inject constructor(
isUpdated = false
)
}
}
/**
* TODO This is a temporary method for EAx.
@ -114,9 +115,10 @@ private fun NotificationData?.orDefault(roomId: RoomId, eventId: EventId): Notif
isEncrypted = false,
isDirect = false,
event = NotificationEvent(
timestamp = System.currentTimeMillis(),
timestamp = clock.epochMillis(),
content = "Message ${eventId.value.take(8)}… in room ${roomId.value.take(8)}",
contentUrl = null
)
)
}
}