Unverified Commit dc186c7a authored by Rafael Kellermann Streit's avatar Rafael Kellermann Streit Committed by GitHub

Merge pull request #1286 from RocketChat/fix/broadcast-replies-showing-outside-rooms

[FIX] Broadcast message replies showing outside of rooms
parents b9f51766 78b43eec
...@@ -153,7 +153,7 @@ class ChatRoomPresenter @Inject constructor( ...@@ -153,7 +153,7 @@ class ChatRoomPresenter @Inject constructor(
if (offset == 0L) { if (offset == 0L) {
val localMessages = messagesRepository.getByRoomId(chatRoomId) val localMessages = messagesRepository.getByRoomId(chatRoomId)
val oldMessages = mapper.map(localMessages, RoomViewModel(roles = chatRoles, val oldMessages = mapper.map(localMessages, RoomViewModel(roles = chatRoles,
isBroadcast = chatIsBroadcast)) isBroadcast = chatIsBroadcast, isRoom = true))
if (oldMessages.isNotEmpty()) { if (oldMessages.isNotEmpty()) {
view.showMessages(oldMessages) view.showMessages(oldMessages)
loadMissingMessages() loadMissingMessages()
...@@ -194,7 +194,7 @@ class ChatRoomPresenter @Inject constructor( ...@@ -194,7 +194,7 @@ class ChatRoomPresenter @Inject constructor(
} }
messagesRepository.saveAll(messages) messagesRepository.saveAll(messages)
view.showMessages(mapper.map(messages, RoomViewModel(roles = chatRoles, view.showMessages(mapper.map(messages, RoomViewModel(roles = chatRoles,
isBroadcast = chatIsBroadcast))) isBroadcast = chatIsBroadcast, isRoom = true)))
} }
fun sendMessage(chatRoomId: String, text: String, messageId: String?) { fun sendMessage(chatRoomId: String, text: String, messageId: String?) {
...@@ -373,7 +373,7 @@ class ChatRoomPresenter @Inject constructor( ...@@ -373,7 +373,7 @@ class ChatRoomPresenter @Inject constructor(
if (messages.result.isNotEmpty()) { if (messages.result.isNotEmpty()) {
val models = mapper.map(messages.result, RoomViewModel( val models = mapper.map(messages.result, RoomViewModel(
roles = chatRoles, isBroadcast = chatIsBroadcast)) roles = chatRoles, isBroadcast = chatIsBroadcast, isRoom = true))
messagesRepository.saveAll(messages.result) messagesRepository.saveAll(messages.result)
launchUI(strategy) { launchUI(strategy) {
......
...@@ -4,5 +4,6 @@ import chat.rocket.core.model.ChatRoomRole ...@@ -4,5 +4,6 @@ import chat.rocket.core.model.ChatRoomRole
data class RoomViewModel( data class RoomViewModel(
val roles: List<ChatRoomRole>, val roles: List<ChatRoomRole>,
val isBroadcast: Boolean = false val isBroadcast: Boolean = false,
val isRoom: Boolean = false
) )
\ No newline at end of file
...@@ -120,7 +120,7 @@ class ViewModelMapper @Inject constructor( ...@@ -120,7 +120,7 @@ class ViewModelMapper @Inject constructor(
private fun isBroadcastReplyAvailable(roomViewModel: RoomViewModel, message: Message): Boolean { private fun isBroadcastReplyAvailable(roomViewModel: RoomViewModel, message: Message): Boolean {
val senderUsername = message.sender?.username val senderUsername = message.sender?.username
return roomViewModel.isBroadcast && return roomViewModel.isRoom && roomViewModel.isBroadcast &&
!message.isSystemMessage() && !message.isSystemMessage() &&
senderUsername != currentUsername senderUsername != currentUsername
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment