[FIX] web_editor: fix module auto install with web
The web_editor dependency with mail was problematic: - web_editor would not auto_intall without the mail being installed - A circular dependency problem with discuss In order to avoid creating a new module shared by mail and web editor, the use of a hack has been decided. By checking if `notification_type` is in the session, we ensure that the module mail is installed. Task-2698655 closes odoo/odoo#80640 X-original-commit: 4e611d7527f85fc074115bd018adb7d6e5ea2c53 Signed-off-by: David Monjoie (dmo) <dmo@odoo.com> Co-authored-by: Antony Lesuisse <al@odoo.com>
This commit is contained in:
co-authored by
Antony Lesuisse
parent
9f480ba234
commit
f98db923e2
@@ -9,7 +9,7 @@ Odoo Web Editor widget.
|
||||
==========================
|
||||
|
||||
""",
|
||||
'depends': ['web', 'mail'],
|
||||
'depends': ['web'],
|
||||
'data': [
|
||||
'security/ir.model.access.csv',
|
||||
'data/editor_assets.xml',
|
||||
|
||||
@@ -82,7 +82,11 @@ const Wysiwyg = Widget.extend({
|
||||
const commands = this._getCommands();
|
||||
|
||||
let editorCollaborationOptions;
|
||||
if (options.collaborationChannel) {
|
||||
if (
|
||||
options.collaborationChannel &&
|
||||
// Hack: check if mail module is installed.
|
||||
this.getSession()['notification_type']
|
||||
) {
|
||||
editorCollaborationOptions = this.setupCollaboration(options.collaborationChannel);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user