Prevent duplicate GetOrCreateChannel calls when paginating messages

CHANGELOG.md

@@ -36,6 +36,7 @@
 
 ## stream-chat-android-state
 ### 🐞 Fixed
+- Prevent duplicate `GetOrCreateChannel` calls when paginating messages. [#6016](https://github.com/GetStream/stream-chat-android/pull/6016)
 
 ### ⬆️ Improved
 

stream-chat-android-state/src/main/java/io/getstream/chat/android/state/plugin/logic/channel/internal/ChannelLogic.kt

@@ -237,7 +237,12 @@ internal class ChannelLogic(
         val offlineChannel = runChannelQueryOffline(request)
 
         val onlineResult = runChannelQueryOnline(request)
-            .onSuccess { fillTheGap(request.messagesLimit(), loadedMessages, it.messages) }
+            .onSuccess {
+                // Fill the missing messages gap only if loading around ID
+                if (request.isFilteringAroundIdMessages()) {
+                    fillTheGap(request.messagesLimit(), loadedMessages, it.messages)
+                }
+            }
 
         return when {
             onlineResult is Result.Success -> onlineResult
@@ -480,15 +485,6 @@ internal class ChannelLogic(
         channelStateLogic.removeMessagesBefore(date, systemMessage)
     }
 
-    /**
-     * Hides the messages created before the given date.
-     *
-     * @param date The date used for generating result.
-     */
-    internal fun hideMessagesBefore(date: Date) {
-        channelStateLogic.hideMessagesBefore(date)
-    }
-
     private fun upsertReminder(messageId: String, reminder: MessageReminder) {
         val message = reminder.message ?: mutableState.getMessageById(messageId) ?: return
         upsertEventMessage(message.copy(reminder = reminder.toMessageReminderInfo()))

stream-chat-android-ui-common/src/main/kotlin/io/getstream/chat/android/ui/common/feature/messages/list/MessageListController.kt

@@ -1210,15 +1210,14 @@ public class MessageListController(
         logger.d { "[loadOlderMessages] messageLimit: $messageLimit" }
         if (clientState.isOffline) return
 
-        _mode.value.run {
-            when (this) {
-                is MessageMode.Normal -> {
-                    if (channelState.value?.endOfOlderMessages?.value == true) return
-                    chatClient.loadOlderMessages(cid, messageLimit).enqueue()
-                }
-
-                is MessageMode.MessageThread -> threadLoadMore(this)
+        when (val mode = _mode.value) {
+            is MessageMode.Normal -> {
+                val endOfOlderMessages = channelState.value?.endOfOlderMessages?.value == true
+                val loadingOlderMessages = channelState.value?.loadingOlderMessages?.value == true
+                if (endOfOlderMessages || loadingOlderMessages) return
+                chatClient.loadOlderMessages(cid, messageLimit).enqueue()
             }
+            is MessageMode.MessageThread -> threadLoadMore(mode)
         }
     }