fix images

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>

Author: mahibi

app/src/main/java/com/nextcloud/talk/chat/ui/model/ChatMessageUi.kt

@@ -8,14 +8,23 @@
 package com.nextcloud.talk.chat.ui.model
 
 import android.text.TextUtils
+import android.util.Log
 import androidx.compose.runtime.Immutable
 import androidx.compose.runtime.Stable
 import androidx.compose.ui.text.AnnotatedString
 import androidx.compose.ui.text.buildAnnotatedString
+import com.nextcloud.talk.R
+import com.nextcloud.talk.application.NextcloudTalkApplication.Companion.sharedApplication
 import com.nextcloud.talk.chat.data.model.ChatMessage
 import com.nextcloud.talk.data.database.model.SendStatus
+import com.nextcloud.talk.data.user.model.User
+import com.nextcloud.talk.utils.ApiUtils
 import com.nextcloud.talk.utils.DrawableUtils
+import java.security.MessageDigest
 import java.time.LocalDate
+import kotlin.collections.component1
+import kotlin.collections.component2
+import kotlin.collections.iterator
 
 // immutable class for chat message UI. only val, no vars!
 @Stable // TODO: or @Immutable ?
@@ -46,7 +55,7 @@ sealed interface MessageTypeContent {
         val todo: String
     ) : MessageTypeContent
 
-    data class Image(val imageUrl: String, val drawableResourceId: Int) : MessageTypeContent
+    data class Image(val imageUrl: String?, val drawableResourceId: Int) : MessageTypeContent
 
     data class Geolocation(val lat: Double, val lon: Double) : MessageTypeContent
 
@@ -70,6 +79,7 @@ enum class MessageStatusIcon {
 
 // Domain model (ChatMessage) to UI model (ChatMessageUi)
 fun ChatMessage.toUiModel(
+    user: User,
     chatMessage: ChatMessage,
     lastCommonReadMessageId: Int,
     parentMessage: ChatMessage?
@@ -93,10 +103,10 @@ fun ChatMessage.toUiModel(
         ),
         timestamp = timestamp,
         date = dateKey(),
-        content = getMessageTypeContent(chatMessage),
+        content = getMessageTypeContent(user, chatMessage),
         // setting parent message recursively might be a problem regarding recompositions? extract only what is needed
         // for UI?
-        parentMessage = parentMessage?.toUiModel(parentMessage, 0, null)
+        parentMessage = parentMessage?.toUiModel(user,parentMessage, 0, null)
     )
 
 fun resolveStatusIcon(
@@ -117,13 +127,16 @@ fun resolveStatusIcon(
     return status
 }
 
-fun getMessageTypeContent(message: ChatMessage): MessageTypeContent? =
+fun getMessageTypeContent(
+    user: User,
+    message: ChatMessage
+): MessageTypeContent? =
     if (!TextUtils.isEmpty(message.systemMessage)) {
         MessageTypeContent.SystemMessage
     } else if (message.isVoiceMessage) {
         getVoiceContent(message)
     } else if (message.hasFileAttachment()) {
-        getImageContent(message)
+        getImageContent(user, message)
     } else if (message.hasGeoLocation()) {
         getGeolocationContent(message)
     } else if (message.isLinkPreview()) {
@@ -141,17 +154,59 @@ fun getLinkPreviewContent(message: ChatMessage): MessageTypeContent.LinkPreview
         todo = "still todo..."
     )
 
-fun getImageContent(message: ChatMessage): MessageTypeContent.Image {
-    val imageUri = message.imageUrl
-    val mimetype = message.selectedIndividualHashMap!!["mimetype"]
+fun getImageContent(
+    user: User,
+    message: ChatMessage
+): MessageTypeContent.Image {
+    val imageUri = getImageUrl(user, message)
+    val mimetype = getMimetype(message)
     val drawableResourceId = DrawableUtils.getDrawableResourceIdForMimeType(mimetype)
 
     return MessageTypeContent.Image(
-        imageUri!!,
+        imageUri,
         drawableResourceId
     )
 }
 
+fun getImageUrl(
+    user: User,
+    message: ChatMessage
+): String? {
+    if (message.messageParameters != null && message.messageParameters!!.size > 0) {
+        for ((_, individualHashMap) in message.messageParameters!!) {
+            if (isHashMapEntryEqualTo(individualHashMap, "type", "file")) {
+                if (!message.isVoiceMessage) {
+                    if (user != null && user!!.baseUrl != null) {
+                        return ApiUtils.getUrlForFilePreviewWithFileId(
+                            user!!.baseUrl!!,
+                            individualHashMap["id"]!!,
+                            sharedApplication!!.resources.getDimensionPixelSize(R.dimen.maximum_file_preview_size)
+                        )
+                    }
+                }
+            }
+        }
+    }
+    return null
+}
+
+fun getMimetype(
+    message: ChatMessage
+): String? {
+    if (message.messageParameters != null && message.messageParameters!!.size > 0) {
+        for ((_, individualHashMap) in message.messageParameters!!) {
+            if (isHashMapEntryEqualTo(individualHashMap, "type", "file")) {
+                return individualHashMap["mimetype"]
+            }
+        }
+    }
+    return null
+}
+
+private fun isHashMapEntryEqualTo(map: HashMap<String?, String?>, key: String, searchTerm: String): Boolean =
+    map != null && MessageDigest.isEqual(map[key]!!.toByteArray(), searchTerm.toByteArray())
+
+
 fun getGeolocationContent(message: ChatMessage): MessageTypeContent.Geolocation? {
     if (message.messageParameters != null && message.messageParameters!!.isNotEmpty()) {
         for (key in message.messageParameters!!.keys) {

app/src/main/java/com/nextcloud/talk/chat/viewmodels/ChatViewModel.kt

@@ -496,6 +496,7 @@ class ChatViewModel @AssistedInject constructor(
                     val parent: ChatMessage? = messageMap[message.parentMessageId]
 
                     message.toUiModel(
+                        currentUser,
                         message,
                         lastRead,
                         parent