From 8cde3e84bb70a2bd097921c08c0059bf65bee602 Mon Sep 17 00:00:00 2001
From: Jorge Pinna Puissant <jpp@odoo.com>
Date: Tue, 14 Feb 2023 15:02:41 +0000
Subject: [PATCH] [REF] web, * : simplify concrete fields API - remove update
 prop

This commit, is part of a series of commits that aim to simplifie
the concrete fields API.

In this commit we will remove update prop from concrete fields. Now each
field will directly use this.props.record.update to make changes, and
handle the save in fields that need to (e.g. priority). As a consequence
of this, the record props need to be mandatory.

task-id 3179751

Part-of: odoo/odoo#112792
---
 .../analytic_distribution.js                  |  2 +-
 .../views/fields/assign_user_command_hook.js  | 20 +++++++++-------
 addons/mrp/static/src/widgets/timer.js        |  2 +-
 .../components/chatter/chatter_composer.js    |  9 ++++----
 .../components/chatter/chatter_composer.xml   | 10 ++++----
 .../static/src/views/fields/ace/ace_field.js  |  2 +-
 .../badge_selection/badge_selection_field.js  |  8 ++++---
 .../src/views/fields/boolean/boolean_field.js |  2 +-
 .../boolean_favorite_field.js                 |  4 ++++
 .../boolean_favorite_field.xml                |  2 +-
 .../fields/boolean_icon/boolean_icon_field.js |  4 ++++
 .../boolean_icon/boolean_icon_field.xml       |  2 +-
 .../boolean_toggle/boolean_toggle_field.js    | 12 ++++++++++
 .../list_boolean_toggle_field.js              |  2 +-
 .../src/views/fields/char/char_field.js       |  7 +++---
 .../src/views/fields/color/color_field.xml    |  2 +-
 .../fields/color_picker/color_picker_field.js |  2 +-
 .../src/views/fields/date/date_field.js       |  2 +-
 .../views/fields/daterange/daterange_field.js |  2 +-
 .../views/fields/datetime/datetime_field.js   |  2 +-
 .../src/views/fields/domain/domain_field.js   |  4 ++--
 addons/web/static/src/views/fields/field.js   | 14 -----------
 .../src/views/fields/float/float_field.js     | 10 +++++---
 .../fields/float_factor/float_factor_field.js | 23 ++++++++-----------
 .../float_factor/float_factor_field.xml       |  8 -------
 .../fields/float_toggle/float_toggle_field.js |  4 +++-
 .../font_selection/font_selection_field.js    |  2 +-
 .../src/views/fields/image/image_field.js     |  4 ++--
 .../src/views/fields/input_field_hook.js      | 17 ++++++--------
 .../views/fields/many2one/many2one_field.js   | 16 +++++++++----
 .../views/fields/priority/priority_field.js   | 18 ++++++++++++---
 .../fields/properties/properties_field.js     | 10 ++++----
 .../src/views/fields/radio/radio_field.js     |  4 ++--
 .../views/fields/reference/reference_field.js | 15 ++++++------
 .../remaining_days/remaining_days_field.js    |  4 ++--
 .../views/fields/selection/selection_field.js |  8 ++++---
 .../views/fields/signature/signature_field.js |  2 +-
 .../src/views/fields/standard_field_props.js  |  3 +--
 .../state_selection/state_selection_field.js  | 14 ++++++++++-
 .../state_selection/state_selection_field.xml |  2 +-
 .../views/fields/statusbar/statusbar_field.js | 16 +++++++++++--
 .../src/views/fields/text/text_field.js       |  8 ++++---
 .../web/static/src/views/relational_model.js  | 11 +++++----
 .../fields/upgrade_boolean_field.js           |  2 +-
 .../web/static/tests/views/list_view_tests.js |  2 +-
 addons/web/static/tests/views/record_tests.js |  2 +-
 .../static/src/js/backend/html_field.js       | 10 ++++----
 47 files changed, 194 insertions(+), 137 deletions(-)
 delete mode 100644 addons/web/static/src/views/fields/float_factor/float_factor_field.xml

diff --git a/addons/analytic/static/src/components/analytic_distribution/analytic_distribution.js b/addons/analytic/static/src/components/analytic_distribution/analytic_distribution.js
index 357f472c651..b15cfa83630 100644
--- a/addons/analytic/static/src/components/analytic_distribution/analytic_distribution.js
+++ b/addons/analytic/static/src/components/analytic_distribution/analytic_distribution.js
@@ -415,7 +415,7 @@ export class AnalyticDistribution extends Component {
     async save() {
         const currentDistribution = this.listForJson;
         const dataToSave = currentDistribution;
-        await this.props.update(dataToSave);
+        await this.props.record.update({ [this.props.name]: dataToSave });
     }
 
     onSaveNew() {
diff --git a/addons/mail/static/src/views/fields/assign_user_command_hook.js b/addons/mail/static/src/views/fields/assign_user_command_hook.js
index 39440f3e485..07592037215 100644
--- a/addons/mail/static/src/views/fields/assign_user_command_hook.js
+++ b/addons/mail/static/src/views/fields/assign_user_command_hook.js
@@ -33,22 +33,26 @@ export function useAssignUserCommand() {
 
     const add = async (record) => {
         if (component.props.type === "many2one") {
-            component.props.update(record);
+            component.props.record.update({ [component.props.name]: record });
         } else if (component.props.type === "many2many") {
-            component.props.update({
-                operation: "REPLACE_WITH",
-                resIds: [...getCurrentIds(), record[0]],
+            component.props.record.update({
+                [component.props.name]: {
+                    operation: "REPLACE_WITH",
+                    resIds: [...getCurrentIds(), record[0]],
+                },
             });
         }
     };
 
     const remove = async (record) => {
         if (component.props.type === "many2one") {
-            component.props.update([]);
+            component.props.record.update({ [component.props.name]: [] });
         } else if (component.props.type === "many2many") {
-            component.props.update({
-                operation: "REPLACE_WITH",
-                resIds: getCurrentIds().filter((id) => id !== record[0]),
+            component.props.record.update({
+                [component.props.name]: {
+                    operation: "REPLACE_WITH",
+                    resIds: getCurrentIds().filter((id) => id !== record[0]),
+                },
             });
         }
     };
diff --git a/addons/mrp/static/src/widgets/timer.js b/addons/mrp/static/src/widgets/timer.js
index f3d8dedc538..365e2efd3a0 100644
--- a/addons/mrp/static/src/widgets/timer.js
+++ b/addons/mrp/static/src/widgets/timer.js
@@ -8,7 +8,7 @@ import { standardFieldProps } from "@web/views/fields/standard_field_props";
 
 const { Component, useState, onWillUpdateProps, onWillStart, onWillDestroy } = owl;
 
-function formatMinutes(value) {
+export function formatMinutes(value) {
     if (value === false) {
         return "";
     }
diff --git a/addons/project/static/src/project_sharing/components/chatter/chatter_composer.js b/addons/project/static/src/project_sharing/components/chatter/chatter_composer.js
index f4219ea8acb..3905c3eba29 100644
--- a/addons/project/static/src/project_sharing/components/chatter/chatter_composer.js
+++ b/addons/project/static/src/project_sharing/components/chatter/chatter_composer.js
@@ -4,8 +4,7 @@ import { useService } from '@web/core/utils/hooks';
 import { TextField } from '@web/views/fields/text/text_field';
 import { PortalAttachDocument } from '../portal_attach_document/portal_attach_document';
 import { ChatterAttachmentsViewer } from './chatter_attachments_viewer';
-
-const { Component, useState, onWillUpdateProps } = owl;
+import { Component, useState, onWillUpdateProps, useRef } from "@odoo/owl";
 
 export class ChatterComposer extends Component {
     setup() {
@@ -16,6 +15,7 @@ export class ChatterComposer extends Component {
             message: '',
             loading: false,
         });
+        this.inputRef = useRef("textarea");
 
         onWillUpdateProps(this.onWillUpdateProps);
     }
@@ -23,6 +23,7 @@ export class ChatterComposer extends Component {
     onWillUpdateProps(nextProps) {
         this.clearErrors();
         this.state.message = '';
+        this.inputRef.el.value = "";
         this.state.attachments = nextProps.attachments.map(file => file.state === 'done');
     }
 
@@ -30,9 +31,9 @@ export class ChatterComposer extends Component {
         return `${window.location.href.split('#')[0]}#discussion`;
     }
 
-    update(change) {
+    update() {
         this.clearErrors();
-        this.state.message = change;
+        this.state.message = this.inputRef.el.value;
     }
 
     prepareMessageData() {
diff --git a/addons/project/static/src/project_sharing/components/chatter/chatter_composer.xml b/addons/project/static/src/project_sharing/components/chatter/chatter_composer.xml
index 6fda276a2a4..47cf7239a38 100644
--- a/addons/project/static/src/project_sharing/components/chatter/chatter_composer.xml
+++ b/addons/project/static/src/project_sharing/components/chatter/chatter_composer.xml
@@ -20,11 +20,13 @@
                     <div class="flex-grow-1">
                         <div class="o_portal_chatter_composer_input">
                             <div class="o_portal_chatter_composer_body mb32">
-                                <TextField
-                                    rowCount="4"
-                                    placeholder="'Write a message...'"
+                                <textarea
+                                    class="o_input"
+                                    placeholder="Write a message..."
+                                    rows="4"
                                     value="state.message"
-                                    update.bind="update"
+                                    t-on-input="update"
+                                    t-ref="textarea"
                                 />
                                 <ChatterAttachmentsViewer
                                     attachments="state.attachments"
diff --git a/addons/web/static/src/views/fields/ace/ace_field.js b/addons/web/static/src/views/fields/ace/ace_field.js
index 7fa48f2f36b..894d797b48e 100644
--- a/addons/web/static/src/views/fields/ace/ace_field.js
+++ b/addons/web/static/src/views/fields/ace/ace_field.js
@@ -113,7 +113,7 @@ export class AceField extends Component {
         if (!this.props.readonly) {
             const value = this.aceSession.getValue();
             if (this.props.value !== value) {
-                return this.props.update(value);
+                return this.props.record.update({ [this.props.name]: value });
             }
         }
     }
diff --git a/addons/web/static/src/views/fields/badge_selection/badge_selection_field.js b/addons/web/static/src/views/fields/badge_selection/badge_selection_field.js
index 7c52a932658..aa73459914d 100644
--- a/addons/web/static/src/views/fields/badge_selection/badge_selection_field.js
+++ b/addons/web/static/src/views/fields/badge_selection/badge_selection_field.js
@@ -54,13 +54,15 @@ export class BadgeSelectionField extends Component {
         switch (this.props.type) {
             case "many2one":
                 if (value === false) {
-                    this.props.update(false);
+                    this.props.record.update({ [this.props.name]: false });
                 } else {
-                    this.props.update(this.options.find((option) => option[0] === value));
+                    this.props.record.update({
+                        [this.props.name]: this.options.find((option) => option[0] === value),
+                    });
                 }
                 break;
             case "selection":
-                this.props.update(value);
+                this.props.record.update({ [this.props.name]: value });
                 break;
         }
     }
diff --git a/addons/web/static/src/views/fields/boolean/boolean_field.js b/addons/web/static/src/views/fields/boolean/boolean_field.js
index c2eb677138e..fd50b015f83 100644
--- a/addons/web/static/src/views/fields/boolean/boolean_field.js
+++ b/addons/web/static/src/views/fields/boolean/boolean_field.js
@@ -22,7 +22,7 @@ export class BooleanField extends Component {
      * @param {boolean} newValue
      */
     onChange(newValue) {
-        this.props.update(newValue);
+        this.props.record.update({ [this.props.name]: newValue });
     }
 }
 
diff --git a/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.js b/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.js
index f969361e061..e5f0af08a75 100644
--- a/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.js
+++ b/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.js
@@ -16,6 +16,10 @@ export class BooleanFavoriteField extends Component {
     static defaultProps = {
         noLabel: false,
     };
+
+    update() {
+        this.props.record.update({ [this.props.name]: !this.props.value });
+    }
 }
 
 export const booleanFavoriteField = {
diff --git a/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.xml b/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.xml
index a491fe70ebc..16b399a041c 100644
--- a/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.xml
+++ b/addons/web/static/src/views/fields/boolean_favorite/boolean_favorite_field.xml
@@ -2,7 +2,7 @@
 <templates xml:space="preserve">
 
     <t t-name="web.BooleanFavoriteField" owl="1">
-        <div class="o_favorite" t-on-click.prevent.stop="() => props.update(!props.value)">
+        <div class="o_favorite" t-on-click.prevent.stop="update">
             <a href="#">
                 <i
                     class="fa"
diff --git a/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.js b/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.js
index 8385e6f2e25..4d3e95c7286 100644
--- a/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.js
+++ b/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.js
@@ -18,6 +18,10 @@ export class BooleanIconField extends Component {
     get label() {
         return this.props.record.activeFields[this.props.name].string;
     }
+
+    update() {
+        this.props.record.update({ [this.props.name]: !this.props.value });
+    }
 }
 
 export const booleanIconField = {
diff --git a/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.xml b/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.xml
index ba7c444c238..c16feb215a5 100644
--- a/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.xml
+++ b/addons/web/static/src/views/fields/boolean_icon/boolean_icon_field.xml
@@ -2,7 +2,7 @@
 <templates xml:space="preserve">
 
     <t t-name="web.BooleanIconField" owl="1">
-        <button t-attf-class="btn btn-sm btn-{{props.value ? 'primary' : 'outline-secondary'}} mx-2 py-1 fa {{props.icon}}" t-att-data-tooltip="label" t-on-click.prevent.stop="() => props.update(!props.value)" />
+        <button t-attf-class="btn btn-sm btn-{{props.value ? 'primary' : 'outline-secondary'}} mx-2 py-1 fa {{props.icon}}" t-att-data-tooltip="label" t-on-click.prevent.stop="update" />
     </t>
 
 </templates>
diff --git a/addons/web/static/src/views/fields/boolean_toggle/boolean_toggle_field.js b/addons/web/static/src/views/fields/boolean_toggle/boolean_toggle_field.js
index d50a4057b96..8e3bda722e4 100644
--- a/addons/web/static/src/views/fields/boolean_toggle/boolean_toggle_field.js
+++ b/addons/web/static/src/views/fields/boolean_toggle/boolean_toggle_field.js
@@ -10,6 +10,18 @@ export class BooleanToggleField extends BooleanField {
     get isReadonly() {
         return this.props.record.isReadonly(this.props.name);
     }
+
+    async onChange(newValue) {
+        await this.props.record.update({ [this.props.name]: newValue });
+        const rootRecord =
+            this.props.record.model.root instanceof this.props.record.constructor &&
+            this.props.record.model.root;
+        const isInEdition = rootRecord ? rootRecord.isInEdition : this.props.record.isInEdition;
+        // We save only if we're on view mode readonly and no readonly field modifier
+        if (!isInEdition) {
+            return this.props.record.save();
+        }
+    }
 }
 
 export const booleanToggleField = {
diff --git a/addons/web/static/src/views/fields/boolean_toggle/list_boolean_toggle_field.js b/addons/web/static/src/views/fields/boolean_toggle/list_boolean_toggle_field.js
index fab772db4f5..1043a688593 100644
--- a/addons/web/static/src/views/fields/boolean_toggle/list_boolean_toggle_field.js
+++ b/addons/web/static/src/views/fields/boolean_toggle/list_boolean_toggle_field.js
@@ -8,7 +8,7 @@ export class ListBooleanToggleField extends BooleanToggleField {
 
     onClick() {
         if (!this.props.readonly) {
-            this.props.update(!this.props.value);
+            this.props.record.update({ [this.props.name]: !this.props.value });
         }
     }
 }
diff --git a/addons/web/static/src/views/fields/char/char_field.js b/addons/web/static/src/views/fields/char/char_field.js
index 3daa54a1966..212f70899fa 100644
--- a/addons/web/static/src/views/fields/char/char_field.js
+++ b/addons/web/static/src/views/fields/char/char_field.js
@@ -66,9 +66,10 @@ export class CharField extends Component {
             let dynamicPlaceholder = "{{object." + chain.join(".");
             dynamicPlaceholder +=
                 defaultValue && defaultValue !== "" ? ` or '''${defaultValue}'''}}` : "}}";
-            this.props.update(
-                this.input.el.value.replace(triggerKeyReplaceRegex, "") + dynamicPlaceholder
-            );
+            this.props.record.update({
+                [this.props.name]:
+                    this.input.el.value.replace(triggerKeyReplaceRegex, "") + dynamicPlaceholder,
+            });
         }
     }
     onDynamicPlaceholderClose() {
diff --git a/addons/web/static/src/views/fields/color/color_field.xml b/addons/web/static/src/views/fields/color/color_field.xml
index 46a47d119dc..fd71fdddd7a 100644
--- a/addons/web/static/src/views/fields/color/color_field.xml
+++ b/addons/web/static/src/views/fields/color/color_field.xml
@@ -3,7 +3,7 @@
 
     <t t-name="web.ColorField" owl="1">
         <div class="o_field_color d-flex" t-att-class="{ 'o_field_cursor_disabled': isReadonly }" t-attf-style="background: #{state.color or 'url(/web/static/img/transparent.png)'}">
-            <input t-on-click.stop="" class="w-100 h-100 opacity-0" type="color" t-att-value="state.color" t-att-disabled="isReadonly" t-on-input="(ev) => this.state.color = ev.target.value" t-on-change="(ev) => this.props.update(ev.target.value)" />
+            <input t-on-click.stop="" class="w-100 h-100 opacity-0" type="color" t-att-value="state.color" t-att-disabled="isReadonly" t-on-input="(ev) => this.state.color = ev.target.value" t-on-change="(ev) => this.props.record.update({ [this.props.name]: ev.target.value })" />
         </div>
     </t>
 
diff --git a/addons/web/static/src/views/fields/color_picker/color_picker_field.js b/addons/web/static/src/views/fields/color_picker/color_picker_field.js
index d2b523f77b7..6f7a61db791 100644
--- a/addons/web/static/src/views/fields/color_picker/color_picker_field.js
+++ b/addons/web/static/src/views/fields/color_picker/color_picker_field.js
@@ -26,7 +26,7 @@ export class ColorPickerField extends Component {
     }
 
     switchColor(colorIndex) {
-        this.props.update(colorIndex);
+        this.props.record.update({ [this.props.name]: colorIndex });
     }
 }
 
diff --git a/addons/web/static/src/views/fields/date/date_field.js b/addons/web/static/src/views/fields/date/date_field.js
index fc1ed6d6f07..3de21385ab9 100644
--- a/addons/web/static/src/views/fields/date/date_field.js
+++ b/addons/web/static/src/views/fields/date/date_field.js
@@ -46,7 +46,7 @@ export class DateField extends Component {
 
     onDateTimeChanged(date) {
         if (!areDateEquals(this.date || "", date)) {
-            this.props.update(date);
+            this.props.record.update({ [this.props.name]: date });
         }
     }
     onDatePickerInput(ev) {
diff --git a/addons/web/static/src/views/fields/daterange/daterange_field.js b/addons/web/static/src/views/fields/daterange/daterange_field.js
index 09494c25a1e..c681c5ad9d7 100644
--- a/addons/web/static/src/views/fields/daterange/daterange_field.js
+++ b/addons/web/static/src/views/fields/daterange/daterange_field.js
@@ -114,7 +114,7 @@ export class DateRangeField extends Component {
             this.props.record.setInvalidField(this.props.name);
             return;
         }
-        this.props.update(value);
+        this.props.record.update({ [this.props.name]: value });
     }
 
     onWindowScroll(ev) {
diff --git a/addons/web/static/src/views/fields/datetime/datetime_field.js b/addons/web/static/src/views/fields/datetime/datetime_field.js
index bbf1cce4e0a..b333ec6b6d1 100644
--- a/addons/web/static/src/views/fields/datetime/datetime_field.js
+++ b/addons/web/static/src/views/fields/datetime/datetime_field.js
@@ -36,7 +36,7 @@ export class DateTimeField extends Component {
 
     onDateTimeChanged(date) {
         if (!areDateEquals(this.props.value || "", date)) {
-            this.props.update(date);
+            this.props.record.update({ [this.props.name]: date });
         }
     }
     onDatePickerInput(ev) {
diff --git a/addons/web/static/src/views/fields/domain/domain_field.js b/addons/web/static/src/views/fields/domain/domain_field.js
index bd53da421d8..e49f076dcf1 100644
--- a/addons/web/static/src/views/fields/domain/domain_field.js
+++ b/addons/web/static/src/views/fields/domain/domain_field.js
@@ -125,7 +125,7 @@ export class DomainField extends Component {
 
     update(domain, isDebugEdited) {
         this.isDebugEdited = isDebugEdited;
-        return this.props.update(domain);
+        this.props.record.update({ [this.props.name]: domain });
     }
 
     onEditDialogBtnClick() {
@@ -134,7 +134,7 @@ export class DomainField extends Component {
             initialValue: this.props.value || "[]",
             readonly: this.props.readonly,
             isDebugMode: !!this.env.debug,
-            onSelected: this.props.update,
+            onSelected: (value) => this.props.record.update({ [this.props.name]: value }),
         });
     }
 }
diff --git a/addons/web/static/src/views/fields/field.js b/addons/web/static/src/views/fields/field.js
index 235d89866b7..a558882620e 100644
--- a/addons/web/static/src/views/fields/field.js
+++ b/addons/web/static/src/views/fields/field.js
@@ -160,20 +160,6 @@ export class Field extends Component {
 
         return {
             ...fieldInfo.props,
-            update: async (value) => {
-                await record.update({ [this.props.name]: value });
-                if (record.selected && record.model.multiEdit) {
-                    return;
-                }
-                const rootRecord =
-                    record.model.root instanceof record.constructor && record.model.root;
-                const isInEdition = rootRecord ? rootRecord.isInEdition : record.isInEdition;
-                // We save only if we're on view mode readonly and no readonly field modifier
-                if (!isInEdition && !readonlyFromModifiers) {
-                    // TODO: maybe move this in the model
-                    return record.save();
-                }
-            },
             value: this.props.record.data[this.props.name],
             decorations: decorationMap,
             readonly: !record.isInEdition || readonlyFromModifiers || false,
diff --git a/addons/web/static/src/views/fields/float/float_field.js b/addons/web/static/src/views/fields/float/float_field.js
index 5e6066a9296..8afe5d20fae 100644
--- a/addons/web/static/src/views/fields/float/float_field.js
+++ b/addons/web/static/src/views/fields/float/float_field.js
@@ -37,10 +37,14 @@ export class FloatField extends Component {
     }
 
     get formattedValue() {
-        if (this.props.inputType === "number" && !this.props.readonly && this.props.value) {
-            return this.props.value;
+        if (this.props.inputType === "number" && !this.props.readonly && this.value) {
+            return this.value;
         }
-        return formatFloat(this.props.value, { digits: this.props.digits });
+        return formatFloat(this.value, { digits: this.props.digits });
+    }
+
+    get value() {
+        return this.props.value;
     }
 }
 
diff --git a/addons/web/static/src/views/fields/float_factor/float_factor_field.js b/addons/web/static/src/views/fields/float_factor/float_factor_field.js
index 946e6f350bd..f95504e31f2 100644
--- a/addons/web/static/src/views/fields/float_factor/float_factor_field.js
+++ b/addons/web/static/src/views/fields/float_factor/float_factor_field.js
@@ -2,11 +2,8 @@
 
 import { registry } from "@web/core/registry";
 import { floatField, FloatField } from "../float/float_field";
-import { Component } from "@odoo/owl";
 
-export class FloatFactorField extends Component {
-    static template = "web.FloatFactorField";
-    static components = { FloatField };
+export class FloatFactorField extends FloatField {
     static props = {
         ...FloatField.props,
         factor: { type: Number, optional: true },
@@ -16,18 +13,16 @@ export class FloatFactorField extends Component {
         factor: 1,
     };
 
-    get factor() {
-        return this.props.factor;
+    parse(value) {
+        let factorValue = value / this.props.factor;
+        if (this.props.inputType !== "number") {
+            factorValue = factorValue.toString();
+        }
+        return super.parse(factorValue);
     }
 
-    get floatFieldProps() {
-        const result = {
-            ...this.props,
-            value: this.props.value * this.factor,
-            update: (value) => this.props.update(value / this.factor),
-        };
-        delete result.factor;
-        return result;
+    get value() {
+        return this.props.value * this.props.factor;
     }
 }
 
diff --git a/addons/web/static/src/views/fields/float_factor/float_factor_field.xml b/addons/web/static/src/views/fields/float_factor/float_factor_field.xml
deleted file mode 100644
index 57b4c9d97af..00000000000
--- a/addons/web/static/src/views/fields/float_factor/float_factor_field.xml
+++ /dev/null
@@ -1,8 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<templates id="template" xml:space="preserve">
-
-    <t t-name="web.FloatFactorField" owl="1">
-        <FloatField t-props="floatFieldProps" />
-    </t>
-
-</templates>
diff --git a/addons/web/static/src/views/fields/float_toggle/float_toggle_field.js b/addons/web/static/src/views/fields/float_toggle/float_toggle_field.js
index 60edbec9a1f..d1f7c0a570e 100644
--- a/addons/web/static/src/views/fields/float_toggle/float_toggle_field.js
+++ b/addons/web/static/src/views/fields/float_toggle/float_toggle_field.js
@@ -29,7 +29,9 @@ export class FloatToggleField extends Component {
         if (currentIndex > this.props.range.length - 1) {
             currentIndex = 0;
         }
-        this.props.update(this.props.range[currentIndex] / this.factor);
+        this.props.record.update({
+            [this.props.name]: this.props.range[currentIndex] / this.factor,
+        });
     }
 
     // This property has been created in order to allow overrides in other modules.
diff --git a/addons/web/static/src/views/fields/font_selection/font_selection_field.js b/addons/web/static/src/views/fields/font_selection/font_selection_field.js
index d8f297c76cd..2d7ff16be63 100644
--- a/addons/web/static/src/views/fields/font_selection/font_selection_field.js
+++ b/addons/web/static/src/views/fields/font_selection/font_selection_field.js
@@ -35,7 +35,7 @@ export class FontSelectionField extends Component {
      */
     onChange(ev) {
         const value = JSON.parse(ev.target.value);
-        this.props.update(value);
+        this.props.record.update({ [this.props.name]: value });
     }
 }
 
diff --git a/addons/web/static/src/views/fields/image/image_field.js b/addons/web/static/src/views/fields/image/image_field.js
index c42e9a29b84..7e65fed0858 100644
--- a/addons/web/static/src/views/fields/image/image_field.js
+++ b/addons/web/static/src/views/fields/image/image_field.js
@@ -112,13 +112,13 @@ export class ImageField extends Component {
     }
     onFileRemove() {
         this.state.isValid = true;
-        this.props.update(false);
+        this.props.record.update({ [this.props.name]: false });
     }
     onFileUploaded(info) {
         this.state.isValid = true;
         // Invalidate the `rawCacheKey`.
         this.rawCacheKey = null;
-        this.props.update(info.data);
+        this.props.record.update({ [this.props.name]: info.data });
     }
     onLoadFailed() {
         this.state.isValid = false;
diff --git a/addons/web/static/src/views/fields/input_field_hook.js b/addons/web/static/src/views/fields/input_field_hook.js
index a32a9ec0294..4c4b36a0679 100644
--- a/addons/web/static/src/views/fields/input_field_hook.js
+++ b/addons/web/static/src/views/fields/input_field_hook.js
@@ -64,16 +64,16 @@ export function useInputField(params) {
                 try {
                     val = params.parse(val);
                 } catch {
-                    if (component.props.record) {
-                        component.props.record.setInvalidField(component.props.name);
-                    }
+                    component.props.record.setInvalidField(component.props.name);
                     isInvalid = true;
                 }
             }
 
             if (!isInvalid) {
                 pendingUpdate = true;
-                Promise.resolve(component.props.update(val)).then(() => {
+                Promise.resolve(
+                    component.props.record.update({ [component.props.name]: val })
+                ).then(() => {
                     pendingUpdate = false;
                 });
                 lastSetValue = ev.target.value;
@@ -114,10 +114,7 @@ export function useInputField(params) {
      * If it is not such a case, we update the field with the new value.
      */
     useEffect(() => {
-        const isInvalid = component.props.record
-            ? component.props.record.isInvalid(component.props.name)
-            : false;
-        if (inputRef.el && !isDirty && !isInvalid) {
+        if (inputRef.el && !isDirty && !component.props.record.isInvalid(component.props.name)) {
             inputRef.el.value = params.getValue();
             lastSetValue = inputRef.el.value;
         }
@@ -148,7 +145,7 @@ export function useInputField(params) {
                     isInvalid = true;
                     if (urgent) {
                         return;
-                    } else if (component.props.record) {
+                    } else {
                         component.props.record.setInvalidField(component.props.name);
                     }
                 }
@@ -159,7 +156,7 @@ export function useInputField(params) {
             }
 
             if ((val || false) !== (component.props.value || false)) {
-                await component.props.update(val);
+                await component.props.record.update({ [component.props.name]: val });
                 lastSetValue = inputRef.el.value;
                 if (component.props.setDirty) {
                     component.props.setDirty(isDirty);
diff --git a/addons/web/static/src/views/fields/many2one/many2one_field.js b/addons/web/static/src/views/fields/many2one/many2one_field.js
index 88e72b19afe..5a399ff7eb1 100644
--- a/addons/web/static/src/views/fields/many2one/many2one_field.js
+++ b/addons/web/static/src/views/fields/many2one/many2one_field.js
@@ -57,6 +57,7 @@ export class Many2OneField extends Component {
         relation: { type: String, optional: true },
         string: { type: String, optional: true },
         canScanBarcode: { type: Boolean, optional: true },
+        update: { type: Function, optional: true },
         openTarget: {
             type: String,
             validate: (v) => ["current", "new"].includes(v),
@@ -101,7 +102,7 @@ export class Many2OneField extends Component {
             isToMany: false,
             onRecordSaved: async (record) => {
                 await this.props.record.load();
-                await this.props.update(m2oTupleFromData(record.data));
+                await this.updateRecord(m2oTupleFromData(record.data));
                 if (this.props.record.model.root.id !== this.props.record.id) {
                     this.props.record.switchMode("readonly");
                 }
@@ -115,7 +116,7 @@ export class Many2OneField extends Component {
                 value = m2oTupleFromData(value[0]);
             }
             this.state.isFloating = false;
-            return this.props.update(value);
+            return this.updateRecord(value);
         };
 
         if (this.props.canQuickCreate) {
@@ -123,7 +124,7 @@ export class Many2OneField extends Component {
                 if (params.triggeredOnBlur) {
                     return this.openConfirmationDialog(name);
                 }
-                return this.props.update([false, name]);
+                return this.updateRecord([false, name]);
             };
         }
 
@@ -137,6 +138,14 @@ export class Many2OneField extends Component {
         });
     }
 
+    updateRecord(value) {
+        const changes = { [this.props.name]: value };
+        if (this.props.update) {
+            return this.props.update(changes);
+        }
+        return this.props.record.update(changes);
+    }
+
     get relation() {
         return this.props.relation || this.props.record.fields[this.props.name].relation;
     }
@@ -299,7 +308,6 @@ export const many2OneField = {
             nameCreateField: attrs.options.create_name_field,
             canScanBarcode: !!attrs.options.can_scan_barcode,
             openTarget: attrs.open_target,
-
             relation: field.relation,
             string: attrs.string || field.string,
         };
diff --git a/addons/web/static/src/views/fields/priority/priority_field.js b/addons/web/static/src/views/fields/priority/priority_field.js
index 31925f27153..fa9ce33afa4 100644
--- a/addons/web/static/src/views/fields/priority/priority_field.js
+++ b/addons/web/static/src/views/fields/priority/priority_field.js
@@ -31,7 +31,7 @@ export class PriorityField extends Component {
                                     this.options.map((value) => ({
                                         name: value[1],
                                         action: () => {
-                                            this.props.update(value[0]);
+                                            this.updateRecord(value[0]);
                                         },
                                     })),
                             },
@@ -66,9 +66,21 @@ export class PriorityField extends Component {
     onStarClicked(value) {
         if (this.props.value === value) {
             this.state.index = -1;
-            this.props.update(this.options[0][0]);
+            this.updateRecord(this.options[0][0]);
         } else {
-            this.props.update(value);
+            this.updateRecord(value);
+        }
+    }
+
+    async updateRecord(value) {
+        await this.props.record.update({ [this.props.name]: value });
+        const rootRecord =
+            this.props.record.model.root instanceof this.props.record.constructor &&
+            this.props.record.model.root;
+        const isInEdition = rootRecord ? rootRecord.isInEdition : this.props.record.isInEdition;
+        // We save only if we're on view mode readonly and no readonly field modifier
+        if (!isInEdition) {
+            return this.props.record.save();
         }
     }
 }
diff --git a/addons/web/static/src/views/fields/properties/properties_field.js b/addons/web/static/src/views/fields/properties/properties_field.js
index 2fffc5ee913..e7d2d82bdc4 100644
--- a/addons/web/static/src/views/fields/properties/properties_field.js
+++ b/addons/web/static/src/views/fields/properties/properties_field.js
@@ -193,7 +193,7 @@ export class PropertiesField extends Component {
         propertiesValues[targetIndex] = propertiesValues[propertyIndex];
         propertiesValues[propertyIndex] = prop;
         propertiesValues[propertyIndex].definition_changed = true;
-        this.props.update(propertiesValues).then(() => {
+        this.props.record.update({ [this.props.name]: propertiesValues }).then(() => {
             // move the popover once the DOM is updated
             this.shouldUpdatePopoverPosition = true;
         });
@@ -209,7 +209,7 @@ export class PropertiesField extends Component {
     onPropertyValueChange(propertyName, propertyValue) {
         const propertiesValues = this.propertiesList;
         propertiesValues.find((property) => property.name === propertyName).value = propertyValue;
-        this.props.update(propertiesValues);
+        this.props.record.update({ [this.props.name]: propertiesValues });
     }
 
     /**
@@ -247,7 +247,7 @@ export class PropertiesField extends Component {
         this._regeneratePropertyName(propertyDefinition);
 
         propertiesValues[propertyIndex] = propertyDefinition;
-        this.props.update(propertiesValues);
+        this.props.record.update({ [this.props.name]: propertiesValues });
     }
 
     /**
@@ -275,7 +275,7 @@ export class PropertiesField extends Component {
                 propertiesDefinitions.find(
                     (property) => property.name === propertyName
                 ).definition_deleted = true;
-                this.props.update(propertiesDefinitions);
+                this.props.record.update({ [this.props.name]: propertiesDefinitions });
             },
             cancel: () => {},
         };
@@ -307,7 +307,7 @@ export class PropertiesField extends Component {
             definition_changed: true,
         });
         this.openLastPropertyDefinition = true;
-        this.props.update(propertiesDefinitions);
+        this.props.record.update({ [this.props.name]: propertiesDefinitions });
     }
 
     /**
diff --git a/addons/web/static/src/views/fields/radio/radio_field.js b/addons/web/static/src/views/fields/radio/radio_field.js
index 9f0630d73c2..e4055d3567d 100644
--- a/addons/web/static/src/views/fields/radio/radio_field.js
+++ b/addons/web/static/src/views/fields/radio/radio_field.js
@@ -58,10 +58,10 @@ export class RadioField extends Component {
     onChange(value) {
         switch (this.props.type) {
             case "selection":
-                this.props.update(value[0]);
+                this.props.record.update({ [this.props.name]: value[0] });
                 break;
             case "many2one":
-                this.props.update(value);
+                this.props.record.update({ [this.props.name]: value });
                 break;
         }
     }
diff --git a/addons/web/static/src/views/fields/reference/reference_field.js b/addons/web/static/src/views/fields/reference/reference_field.js
index 336e6a730d4..edffa378310 100644
--- a/addons/web/static/src/views/fields/reference/reference_field.js
+++ b/addons/web/static/src/views/fields/reference/reference_field.js
@@ -34,7 +34,7 @@ export class ReferenceField extends Component {
                 this.state.resModel &&
                 this.getRelation(nextProps) !== this.state.resModel
             ) {
-                nextProps.update(false);
+                nextProps.record.update({ [this.props.name]: false });
             }
         });
     }
@@ -102,20 +102,21 @@ export class ReferenceField extends Component {
 
     updateModel(value) {
         this.state.resModel = value;
-        this.props.update(false);
+        this.props.record.update({ [this.props.name]: false });
     }
 
-    updateM2O(value) {
+    updateM2O(data) {
+        const value = data[this.props.name];
         if (!this.state.resModel) {
             this.state.resModel = this.relation;
         }
-        this.props.update(
-            value && {
+        this.props.record.update({
+            [this.props.name]: value && {
                 resModel: this.state.resModel,
                 resId: value[0],
                 displayName: value[1],
-            }
-        );
+            },
+        });
     }
 }
 
diff --git a/addons/web/static/src/views/fields/remaining_days/remaining_days_field.js b/addons/web/static/src/views/fields/remaining_days/remaining_days_field.js
index c3f83258216..6449ae401da 100644
--- a/addons/web/static/src/views/fields/remaining_days/remaining_days_field.js
+++ b/addons/web/static/src/views/fields/remaining_days/remaining_days_field.js
@@ -39,10 +39,10 @@ export class RemainingDaysField extends Component {
 
     onDateTimeChanged(datetime) {
         if (datetime) {
-            this.props.update(datetime);
+            this.props.record.update({ [this.props.name]: datetime });
         } else if (typeof datetime === "string") {
             // when the date is cleared
-            this.props.update(false);
+            this.props.record.update({ [this.props.name]: false });
         }
     }
 }
diff --git a/addons/web/static/src/views/fields/selection/selection_field.js b/addons/web/static/src/views/fields/selection/selection_field.js
index 911bc7813cf..02785ca7f3f 100644
--- a/addons/web/static/src/views/fields/selection/selection_field.js
+++ b/addons/web/static/src/views/fields/selection/selection_field.js
@@ -57,13 +57,15 @@ export class SelectionField extends Component {
         switch (this.props.type) {
             case "many2one":
                 if (value === false) {
-                    this.props.update(false);
+                    this.props.record.update({ [this.props.name]: false });
                 } else {
-                    this.props.update(this.options.find((option) => option[0] === value));
+                    this.props.record.update({
+                        [this.props.name]: this.options.find((option) => option[0] === value),
+                    });
                 }
                 break;
             case "selection":
-                this.props.update(value);
+                this.props.record.update({ [this.props.name]: value });
                 break;
         }
     }
diff --git a/addons/web/static/src/views/fields/signature/signature_field.js b/addons/web/static/src/views/fields/signature/signature_field.js
index 7740a74658b..06a37b80bac 100644
--- a/addons/web/static/src/views/fields/signature/signature_field.js
+++ b/addons/web/static/src/views/fields/signature/signature_field.js
@@ -130,7 +130,7 @@ export class SignatureField extends Component {
      * @private
      */
     uploadSignature({ signatureImage }) {
-        return this.props.update(signatureImage[1] || false);
+        return this.props.record.update({ [this.props.name]: signatureImage[1] || false });
     }
 }
 
diff --git a/addons/web/static/src/views/fields/standard_field_props.js b/addons/web/static/src/views/fields/standard_field_props.js
index 26bec6d06ba..d227b56a8e8 100644
--- a/addons/web/static/src/views/fields/standard_field_props.js
+++ b/addons/web/static/src/views/fields/standard_field_props.js
@@ -4,9 +4,8 @@ export const standardFieldProps = {
     id: { type: String, optional: true },
     name: { type: String, optional: true },
     readonly: { type: Boolean, optional: true },
-    record: { type: Object, optional: true },
+    record: { type: Object },
     type: { type: String, optional: true },
-    update: { type: Function, optional: true },
     value: true,
     decorations: { type: Object, optional: true },
     setDirty: { type: Function, optional: true },
diff --git a/addons/web/static/src/views/fields/state_selection/state_selection_field.js b/addons/web/static/src/views/fields/state_selection/state_selection_field.js
index 0779990c73c..937a5f4022f 100644
--- a/addons/web/static/src/views/fields/state_selection/state_selection_field.js
+++ b/addons/web/static/src/views/fields/state_selection/state_selection_field.js
@@ -37,7 +37,7 @@ export class StateSelectionField extends Component {
         for (const [index, [value, label]] of this.options.entries()) {
             useCommand(
                 sprintf(this.env._t("Set kanban state as %s"), label),
-                () => this.props.update(value),
+                () => this.updateRecord(value),
                 {
                     category: "smart_action",
                     hotkey: "alt+" + hotkeys[index],
@@ -76,6 +76,18 @@ export class StateSelectionField extends Component {
     statusColor(value) {
         return this.colors[value] ? this.colorPrefix + this.colors[value] : "";
     }
+
+    async updateRecord(value) {
+        await this.props.record.update({ [this.props.name]: value });
+        const rootRecord =
+            this.props.record.model.root instanceof this.props.record.constructor &&
+            this.props.record.model.root;
+        const isInEdition = rootRecord ? rootRecord.isInEdition : this.props.record.isInEdition;
+        // We save only if we're on view mode readonly and no readonly field modifier
+        if (!isInEdition) {
+            return this.props.record.save();
+        }
+    }
 }
 
 export const stateSelectionField = {
diff --git a/addons/web/static/src/views/fields/state_selection/state_selection_field.xml b/addons/web/static/src/views/fields/state_selection/state_selection_field.xml
index 636f41f8773..2c9fc133871 100644
--- a/addons/web/static/src/views/fields/state_selection/state_selection_field.xml
+++ b/addons/web/static/src/views/fields/state_selection/state_selection_field.xml
@@ -17,7 +17,7 @@
                     </div>
                 </t>
                 <t t-foreach="availableOptions" t-as="option" t-key="option[0]">
-                    <DropdownItem onSelected="() => props.update(option[0])">
+                    <DropdownItem onSelected="() => this.updateRecord(option[0])">
                         <div class="d-flex align-items-center">
                             <span t-attf-class="o_status {{ statusColor(option[0]) }} "/>
                             <span class="ms-2" t-esc="option[1]"/>
diff --git a/addons/web/static/src/views/fields/statusbar/statusbar_field.js b/addons/web/static/src/views/fields/statusbar/statusbar_field.js
index 4131c280a65..ae477bb9942 100644
--- a/addons/web/static/src/views/fields/statusbar/statusbar_field.js
+++ b/addons/web/static/src/views/fields/statusbar/statusbar_field.js
@@ -192,12 +192,24 @@ export class StatusBarField extends Component {
     }
 
     selectItem(item) {
+        const rootRecord =
+            this.props.record.model.root instanceof this.props.record.constructor &&
+            this.props.record.model.root;
+        const isInEdition = rootRecord ? rootRecord.isInEdition : this.props.record.isInEdition;
         switch (this.props.type) {
             case "many2one":
-                this.props.update([item.id, item.name]);
+                this.props.record.update({ [this.props.name]: [item.id, item.name] });
+                // We save only if we're on view mode readonly and no readonly field modifier
+                if (!isInEdition) {
+                    return this.props.record.save();
+                }
                 break;
             case "selection":
-                this.props.update(item.id);
+                this.props.record.update({ [this.props.name]: item.id });
+                // We save only if we're on view mode readonly and no readonly field modifier
+                if (!isInEdition) {
+                    return this.props.record.save();
+                }
                 break;
         }
     }
diff --git a/addons/web/static/src/views/fields/text/text_field.js b/addons/web/static/src/views/fields/text/text_field.js
index f43817ce93b..ef6b17e23f8 100644
--- a/addons/web/static/src/views/fields/text/text_field.js
+++ b/addons/web/static/src/views/fields/text/text_field.js
@@ -73,9 +73,11 @@ export class TextField extends Component {
             let dynamicPlaceholder = "{{object." + chain.join(".");
             dynamicPlaceholder +=
                 defaultValue && defaultValue !== "" ? ` or '''${defaultValue}'''}}` : "}}";
-            this.props.update(
-                this.textareaRef.el.value.replace(triggerKeyReplaceRegex, "") + dynamicPlaceholder
-            );
+            this.props.record.update({
+                [this.props.name]:
+                    this.textareaRef.el.value.replace(triggerKeyReplaceRegex, "") +
+                    dynamicPlaceholder,
+            });
         }
     }
     onDynamicPlaceholderClose() {
diff --git a/addons/web/static/src/views/relational_model.js b/addons/web/static/src/views/relational_model.js
index 3b594e4a611..ff3fcce2072 100644
--- a/addons/web/static/src/views/relational_model.js
+++ b/addons/web/static/src/views/relational_model.js
@@ -1669,10 +1669,13 @@ class DynamicList extends DataPoint {
                 fieldNodes: this.model.fieldNodes,
             };
             this.model.trigger("list-confirmation-dialog-will-open");
-            await this.model.dialogService.add(ListConfirmationDialog, dialogProps, {
-                onClose: () => {
-                    this.model.trigger("list-confirmation-dialog-closed");
-                },
+            await new Promise((resolve) => {
+                this.model.dialogService.add(ListConfirmationDialog, dialogProps, {
+                    onClose: () => {
+                        this.model.trigger("list-confirmation-dialog-closed");
+                        resolve();
+                    },
+                });
             });
         } else {
             await record._save();
diff --git a/addons/web/static/src/webclient/settings_form_view/fields/upgrade_boolean_field.js b/addons/web/static/src/webclient/settings_form_view/fields/upgrade_boolean_field.js
index ac3ade9fb70..f8b1bd8f265 100644
--- a/addons/web/static/src/webclient/settings_form_view/fields/upgrade_boolean_field.js
+++ b/addons/web/static/src/webclient/settings_form_view/fields/upgrade_boolean_field.js
@@ -24,7 +24,7 @@ export class UpgradeBooleanField extends BooleanField {
                 {},
                 {
                     onClose: () => {
-                        this.props.update(false);
+                        this.props.record.update({ [this.props.name]: false });
                     },
                 }
             );
diff --git a/addons/web/static/tests/views/list_view_tests.js b/addons/web/static/tests/views/list_view_tests.js
index 44e34548b72..516b7ca1b32 100644
--- a/addons/web/static/tests/views/list_view_tests.js
+++ b/addons/web/static/tests/views/list_view_tests.js
@@ -11811,7 +11811,7 @@ QUnit.module("Views", (hooks) => {
         );
 
         // Close the confirmation dialog
-        await click(target.querySelector(".modal .btn-primary"));
+        await click(target.querySelectorAll(".modal")[1], ".btn-primary");
 
         assert.strictEqual(
             target.querySelector(".o_data_cell").innerText,
diff --git a/addons/web/static/tests/views/record_tests.js b/addons/web/static/tests/views/record_tests.js
index 2c54d4fcd91..4008695ff74 100644
--- a/addons/web/static/tests/views/record_tests.js
+++ b/addons/web/static/tests/views/record_tests.js
@@ -277,7 +277,7 @@ QUnit.module("Record Component", (hooks) => {
         Parent.components = { Record, Many2OneField };
         Parent.template = xml`
             <Record resModel="'partner'" fieldNames="['foo']" fields="fields" values="values" t-slot-scope="data" onRecordChanged.bind="onRecordChanged">
-                <Many2OneField name="'foo'" record="data.record" relation="'bar'" value="data.record.data.foo" update="(value) => data.record.update({ foo: value })"/>
+                <Many2OneField name="'foo'" record="data.record" relation="'bar'" value="data.record.data.foo"/>
             </Record>
         `;
 
diff --git a/addons/web_editor/static/src/js/backend/html_field.js b/addons/web_editor/static/src/js/backend/html_field.js
index fa053c9d4ce..9a87a60a4cd 100644
--- a/addons/web_editor/static/src/js/backend/html_field.js
+++ b/addons/web_editor/static/src/js/backend/html_field.js
@@ -284,7 +284,7 @@ export class HtmlField extends Component {
         if (value !== null && !(!lastValue && stripHistoryIds(value) === "<p><br></p>") && value !== lastValue) {
             this.props.setDirty(false);
             this.currentEditingValue = value;
-            await this.props.update(value);
+            await this.props.record.update({ [this.props.name]: value });
         }
     }
     async startWysiwyg(wysiwyg) {
@@ -320,12 +320,12 @@ export class HtmlField extends Component {
         if (this.state.showCodeView) {
             this.wysiwyg.odooEditor.toolbarHide();
             const value = this.wysiwyg.getValue();
-            this.props.update(value);
+            this.props.record.update({ [this.props.name]: value });
         } else {
             this.wysiwyg.odooEditor.observerActive('toggleCodeView');
             const $codeview = $(this.codeViewRef.el);
             const value = $codeview.val();
-            this.props.update(value);
+            this.props.record.update({ [this.props.name]: value });
 
         }
     }
@@ -562,7 +562,7 @@ export class HtmlField extends Component {
             checked: !checked,
         });
         if (value) {
-            this.props.update(value);
+            this.props.record.update({ [this.props.name]: value });
         }
     }
     async _onReadonlyClickStar(ev) {
@@ -590,7 +590,7 @@ export class HtmlField extends Component {
             rating,
         });
         if (value) {
-            this.props.update(value);
+            this.props.record.update({ [this.props.name]: value });
         }
     }
 }