diff options
| author | Alexandre Kühn <aku@odoo.com> | 2020-01-09 14:24:53 +0000 |
|---|---|---|
| committer | Alexandre Kühn <aku@odoo.com> | 2020-01-13 10:13:10 +0000 |
| commit | b58c649d721461d4d2e04c169c1a9cb0a86ee7a8 (patch) | |
| tree | 71887cd5a8aba819d8cfb966acf55524a2647a61 /addons/mail/static/tests/document_thread_window_tests.js | |
| parent | a1ea321f82cc1eda524e1aebeeb2635b32fa0aa6 (diff) | |
[FIX] web:tests: remove promise constructor antipattern
Revision on https://github.com/odoo/odoo/commit/141b34f152f36d970d4ff78abe53f563555a8e69
Having async/await as a new promise constructor is an antipattern.
The reason is that it may lead to unnoticed errors: if an inner
promise is rejected, it won't propagate the error to another promise
that will handle the error. As a result, this error becomes unnoticed
and the initial promise is pending indefinitely, which also may lead
to more bugs [1].
[1] https://stackoverflow.com/a/25569299
Closes #43072
Diffstat (limited to 'addons/mail/static/tests/document_thread_window_tests.js')
0 files changed, 0 insertions, 0 deletions