From 87fd81d7f9edd3d42bd77ad1ea1b4b1cd616dc7a Mon Sep 17 00:00:00 2001 From: "Laurent Stukkens (LTU)" Date: Tue, 20 Sep 2022 18:59:45 +0000 Subject: [PATCH] [REF] web: ensure factor override is taken into account in FloatToggleField Prior to this commit, the factor override, if any, was not taken into account when toggle was performed. task-2990758 Part-of: odoo/odoo#100626 --- .../src/views/fields/float_toggle/float_toggle_field.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) 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 2139f5ee594..0a7558f4a28 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 @@ -10,14 +10,15 @@ export class FloatToggleField extends Component { // TODO perf issue (because of update round trip) // we probably want to have a state and a useEffect or onWillUpateProps onChange() { - let currentIndex = this.props.range.indexOf(this.props.value * this.props.factor); + let currentIndex = this.props.range.indexOf(this.props.value * this.factor); currentIndex++; if (currentIndex > this.props.range.length - 1) { currentIndex = 0; } - this.props.update(this.props.range[currentIndex] / this.props.factor); + this.props.update(this.props.range[currentIndex] / this.factor); } + // This property has been created in order to allow overrides in other modules. get factor() { return this.props.factor; }