Odoo/addons/mail/static/src/models/chat_window/chat_window.js, branch 14.0

[FIX] mail: (mobile) Prevent server sync for open/close state

2021-02-12T10:08:57+00:00

This commit prevents the opening/closure of a channel on mobile when
the action is performed on desktop and reciprocally

closes odoo/odoo#64599

Tasks-id: 2267593
Related: odoo/enterprise#16296
Signed-off-by: Sébastien Theys (seb)

[FIX] mail: prevent messaging menu from hiding home menu in mobile

2020-12-15T12:33:54+00:00

task-2330277

closes odoo/odoo#63235

Signed-off-by: Alexandre Kühn (aku)

[FIX] mail: prevent unnecessary fold state RPC

2020-09-23T17:08:02+00:00

When the pending value is the same as the current server value, the pending
value should be ignored and no RPC should be done.

In particular this prevents a non-main tab from calling the RPC when receiving
a bus notification that would lead to closing a chat window, which in turn used
to call the close RPC again before this commit.
This led to inconsistencies especially on the main tab, if the chat window was
requested to be open again after the initial close, but before the "secondary"
close, it would end up closing instead of opening.

The current commit reduces the issue and prevents as many unnecessary RPC as
there are tabs open, which is a good improvement.

But it does not negate it completely. Indeed, when quickly closing then opening
the same chat window, it might still happen for network reasons that the RPC or
the corresponding bus notifications are received in the wrong order. This is
less frequent both functionally (to click so fast) and technically (to have the
network that slow) at the same time, so it is considering acceptable / out of
scope.

The opportunity is taken to clean up the code by removing unnecessary fields and
making use of compute and flux specific RPC instead of `_updateAfter`.

task-2273658
task-2329521

closes odoo/odoo#58367

X-original-commit: cbb1d855c7af5f78819db3cd476dbe0f985bc214
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)

[FIX] mail: ignore `undefined` in JS fields create/write

2020-09-23T09:52:30+00:00

Passing a field containing `undefined` should lead to the same result as not
passing the field.

Fields that relied on `undefined` as a value have been changed to use the newly
introduced `clear` command, that resets a field to its default value (often
that is equal to `undefined`).

task-2336491

closes odoo/odoo#58308

X-original-commit: b18824485483564d2ab80f41914a3548aa528ae0
Signed-off-by: Alexandre Kühn (aku)

[FIX] mail, sms, snailmail: prevent message_fetch for hidden threads

2020-09-04T14:59:01+00:00

Automatically load messages from a thread cache only if the thread cache is used
in at least one thread view.

Ensure thread viewer don't keep thread view records alive when they are not
displayed.

This change is particularly important for performance, to avoid spamming RPC at
init if many threads are opened in chat windows.

task-2310623

closes odoo/odoo#57061

X-original-commit: dd6981c7812134e0588f80a157eaba45512cd68e
Related: odoo/enterprise#12936
Signed-off-by: Alexandre Kühn (aku)

[FIX] mail, *: fix opens and redirects, adapt messaging menu close

2020-08-28T16:15:15+00:00

task-2244279
task-2262862
task-2274993
task-2277537
task-2282238

closes odoo/odoo#56745

X-original-commit: 109f202
Related: odoo/enterprise#12761
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)

[FIX] mail, *: rename thread viewer to thread view

2020-08-28T13:43:54+00:00

The "viewer" should be the "parent" not the view itself. This distinction will
become important in a future commit to allow viewers to not display the view in
some situations.

Rename done as a standalone commit to ease diff readability.

Part of task-2310623

closes odoo/odoo#56737

X-original-commit: 1b811e6a972d72de02965adda67c5b039bee023f
Related: odoo/enterprise#12756
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)

[FIX] mail: avoid unnecessary update of inverse in JS models

2020-08-21T14:16:33+00:00

Part of task-2308312

X-original-commit: 0e08e43decf18108d25d051ad76c093bcdf434db

[FIX] mail: tab cycling between chat window

2020-08-05T11:04:12+00:00

Before this commit, cycling between chat window in LTR language clycled in the
wrong direction.
This commit also handle the RTL language cycling.

task-2279706

closes odoo/odoo#55456

X-original-commit: 638720f9d72616f2e921a475a5fdfd2ad9a2ab8b
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)

[FIX] mail: save scroll position of chat window when folded

2020-07-07T15:57:46+00:00

- Remove no more used ThreadViewer prop : messageListInitialScrollTop
- Remove no more used ChatWindow prop : threadInitialScrollTop
- Rename ThreadViewer field threadCacheInitialPosition to
  threadCacheInitialScrollPosition
- Use the scroll position to set message list scroll on mounted
- Fix old test which was not simulating correctly home menu show
- Add tests to check that scroll position is correctly saved even if
  ChatWindow is folded
- Fix traceback when showing/hiding home menu with a folded chat window

task-2285691

closes odoo/odoo#54198

X-original-commit: 31123a94c156424b3962fe4aebc3cc60a7a89223
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)