From 26ca4af56808c731cf09985b5b82361a4ad805cf Mon Sep 17 00:00:00 2001
From: andrekir <andrekir@pm.me>
Date: Sat, 26 Oct 2024 05:44:59 -0300
Subject: [PATCH] refactor: move `Message` date formatting to ViewModel

---
 .../java/com/geeksville/mesh/model/Message.kt |  2 +-
 .../java/com/geeksville/mesh/model/UIState.kt | 20 +++++++++++++++----
 .../com/geeksville/mesh/ui/MessageListView.kt |  3 +--
 .../geeksville/mesh/ui/MessagesFragment.kt    | 13 ------------
 4 files changed, 18 insertions(+), 20 deletions(-)

diff --git a/app/src/main/java/com/geeksville/mesh/model/Message.kt b/app/src/main/java/com/geeksville/mesh/model/Message.kt
index 356265701..934cf611d 100644
--- a/app/src/main/java/com/geeksville/mesh/model/Message.kt
+++ b/app/src/main/java/com/geeksville/mesh/model/Message.kt
@@ -31,7 +31,7 @@ data class Message(
     val receivedTime: Long,
     val user: MeshProtos.User,
     val text: String,
-    val time: Long,
+    val time: String,
     val read: Boolean,
     val status: MessageStatus?,
     val routingError: Int,
diff --git a/app/src/main/java/com/geeksville/mesh/model/UIState.kt b/app/src/main/java/com/geeksville/mesh/model/UIState.kt
index 9b121c4b5..a41b1cf95 100644
--- a/app/src/main/java/com/geeksville/mesh/model/UIState.kt
+++ b/app/src/main/java/com/geeksville/mesh/model/UIState.kt
@@ -143,8 +143,8 @@ data class Contact(
 )
 
 // return time if within 24 hours, otherwise date
-internal fun getShortDateTime(time: Long): String? {
-    val date = if (time != 0L) Date(time) else return null
+private fun getShortDate(time: Long): String {
+    val date = Date(time)
     val isWithin24Hours = System.currentTimeMillis() - date.time <= TimeUnit.DAYS.toMillis(1)
 
     return if (isWithin24Hours) {
@@ -154,6 +154,18 @@ internal fun getShortDateTime(time: Long): String? {
     }
 }
 
+// return time if within 24 hours, otherwise date/time
+private fun getShortDateTime(time: Long): String {
+    val date = Date(time)
+    val isWithin24Hours = System.currentTimeMillis() - date.time <= TimeUnit.DAYS.toMillis(1)
+
+    return if (isWithin24Hours) {
+        DateFormat.getTimeInstance(DateFormat.SHORT).format(date)
+    } else {
+        DateFormat.getDateTimeInstance(DateFormat.SHORT, DateFormat.SHORT).format(date)
+    }
+}
+
 @Suppress("LongParameterList")
 @HiltViewModel
 class UIViewModel @Inject constructor(
@@ -316,7 +328,7 @@ class UIViewModel @Inject constructor(
                 contactKey = contactKey,
                 shortName = if (toBroadcast) "${data.channel}" else shortName,
                 longName = longName,
-                lastMessageTime = getShortDateTime(data.time),
+                lastMessageTime = getShortDate(data.time),
                 lastMessageText = if (fromLocal) data.text else "$shortName: ${data.text}",
                 unreadCount = packetRepository.getUnreadCount(contactKey),
                 messageCount = packetRepository.getMessageCount(contactKey),
@@ -337,7 +349,7 @@ class UIViewModel @Inject constructor(
                 receivedTime = it.received_time,
                 user = getUser(it.data.from),
                 text = it.data.text.orEmpty(),
-                time = it.data.time,
+                time = getShortDateTime(it.data.time),
                 read = it.read,
                 status = it.data.status,
                 routingError = it.routingError,
diff --git a/app/src/main/java/com/geeksville/mesh/ui/MessageListView.kt b/app/src/main/java/com/geeksville/mesh/ui/MessageListView.kt
index c4fbe2ed7..42c65cf60 100644
--- a/app/src/main/java/com/geeksville/mesh/ui/MessageListView.kt
+++ b/app/src/main/java/com/geeksville/mesh/ui/MessageListView.kt
@@ -20,7 +20,6 @@ import com.geeksville.mesh.ui.components.SimpleAlertDialog
 import kotlinx.coroutines.FlowPreview
 import kotlinx.coroutines.flow.collectLatest
 import kotlinx.coroutines.flow.debounce
-import java.util.Date
 
 @Composable
 internal fun MessageListView(
@@ -56,7 +55,7 @@ internal fun MessageListView(
             MessageItem(
                 shortName = msg.user.shortName.takeIf { msg.user.id != DataPacket.ID_LOCAL },
                 messageText = msg.text,
-                messageTime = getShortDateTime(Date(msg.time)),
+                messageTime = msg.time,
                 messageStatus = msg.status,
                 selected = selected,
                 onClick = { onClick(msg) },
diff --git a/app/src/main/java/com/geeksville/mesh/ui/MessagesFragment.kt b/app/src/main/java/com/geeksville/mesh/ui/MessagesFragment.kt
index 38d14b389..8e4843a06 100644
--- a/app/src/main/java/com/geeksville/mesh/ui/MessagesFragment.kt
+++ b/app/src/main/java/com/geeksville/mesh/ui/MessagesFragment.kt
@@ -36,19 +36,6 @@ import com.google.android.material.dialog.MaterialAlertDialogBuilder
 import dagger.hilt.android.AndroidEntryPoint
 import kotlinx.coroutines.flow.firstOrNull
 import kotlinx.coroutines.launch
-import java.text.DateFormat
-import java.util.Date
-
-// return time if within 24 hours, otherwise date/time
-internal fun getShortDateTime(date: Date): String {
-    val isWithin24Hours = System.currentTimeMillis() - date.time <= 24 * 60 * 60 * 1000L
-
-    return if (isWithin24Hours) {
-        DateFormat.getTimeInstance(DateFormat.SHORT).format(date)
-    } else {
-        DateFormat.getDateTimeInstance(DateFormat.SHORT, DateFormat.SHORT).format(date)
-    }
-}
 
 internal fun FragmentManager.navigateToMessages(contactKey: String, contactName: String) {
     val messagesFragment = MessagesFragment().apply {