odoo/odoo#158943
Created by Discuss, Alexandre Kühn (aku)
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/template: Contact runbot team on discord for help.
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- label
- odoo-dev:17.0-fix-chat-deleted-message-counter-aku
- head
- 3063bfaf2a1c4158c6dfab5e8eb881298c7aaeaf
- merged
- 1 year ago by Discuss, Sébastien Theys (seb)
| odoo/odoo | |
|---|---|
| 17.0 | #158943 |
| saas-17.1 | #159104 |
| saas-17.2 | #159192 |
| 18.0 | |
| saas-18.1 | |
| saas-18.2 | |
| saas-18.3 | |
| saas-18.4 | |
| master | #159362 |
[FIX] mail: opening chat with deleted new message should mark as read
Before this commit, when a user of in chat posted a message and deleted it, all other users kept the chat as unread.
Steps to reproduce:
- Connect as Admin and March Demo
- Send a message to Admin as Demo in DM chat
- Demo deletes this message
- Marc opens the chat => the unread counter is 1 and cannot be removed
This happens because when a message is deleted, there's still a trace of it but the message is empty. However, empty messages could not be candidate of setting the last message being seing by a member, thus members were unable to mark the chat as read until someone else posted a newer message (and did not delete it).
This commit fixes the issue by taking empty messages into account for setting last message message of member, which allow to mark thread as read even when newer messages have been deleted.