[FIX] web: form: prevent from calling mounted twice

With this commit, the FormRenderer no longer directly extends
the WidgetAdapterMixin. It doesn't need to as its parent class
(the BasicRenderer) already uses it. Worse, they both called
WidgetAdapterMixin.on_attach_callback, and thus mounted
was called twice on Components in form views.

closes odoo/odoo#55401

X-original-commit: 4c627abd310a131a4828e9ee6be451aeee4708db
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>

1 files changed, 1 insertions, 11 deletions

diff --git a/addons/web/static/src/js/views/form/form_renderer.js b/addons/web/static/src/js/views/form/form_renderer.js
index 66d4c5d8bd2..2b91c5820ee 100644
--- a/addons/web/static/src/js/views/form/form_renderer.js
+++ b/addons/web/static/src/js/views/form/form_renderer.js
@@ -5,7 +5,6 @@ var BasicRenderer = require('web.BasicRenderer');
 var config = require('web.config');
 var core = require('web.core');
 var dom = require('web.dom');
-const { WidgetAdapterMixin } = require('web.OwlCompatibility');
 var viewUtils = require('web.viewUtils');
 
 var _t = core._t;
@@ -14,7 +13,7 @@ var qweb = core.qweb;
 // symbol used as key to set the <field> node id on its widget
 const symbol = Symbol('form');
 
-var FormRenderer = BasicRenderer.extend(WidgetAdapterMixin, {
+var FormRenderer = BasicRenderer.extend({
     className: "o_form_view",
     events: _.extend({}, BasicRenderer.prototype.events, {
         'click .o_notification_box .oe_field_translate': '_onTranslate',
@@ -54,17 +53,9 @@ var FormRenderer = BasicRenderer.extend(WidgetAdapterMixin, {
         return this._super.apply(this, arguments);
     },
     /**
-     * @override
-     */
-    destroy() {
-        this._super(...arguments);
-        WidgetAdapterMixin.destroy.call(this);
-    },
-    /**
      * Called each time the form view is attached into the DOM
      */
     on_attach_callback: function () {
-        WidgetAdapterMixin.on_attach_callback.call(this);
         this._isInDom = true;
         _.invoke(this.widgets, 'on_attach_callback');
         this._super.apply(this, arguments);
@@ -73,7 +64,6 @@ var FormRenderer = BasicRenderer.extend(WidgetAdapterMixin, {
      * Called each time the renderer is detached from the DOM.
      */
     on_detach_callback: function () {
-        WidgetAdapterMixin.on_detach_callback.call(this);
         this._isInDom = false;
         this._super.apply(this, arguments);
     },