From 97d07c8e0122ae4b281839465f0a9dd4d96b99dd Mon Sep 17 00:00:00 2001 From: Michele Date: Wed, 31 May 2023 15:07:59 +0000 Subject: [PATCH] [FIX] mrp: fix format of bom quantity in mrp_report_bom report If you are using a locale that uses "," as decimal_point and "." as thousand_seps the parseFloat around formatFloat is not parsing correctly the float because the parseFloat is not locale-based but uses always the "." as decimal point. example: in Italy the thousand separator is ".". parseFloat("3.000,00000") returns "3" as result but the correct result is "3000" . It is not necessary to use formatFloat but it is needed closes odoo/odoo#123711 X-original-commit: 48dd3d189a50b4b2a25ce0c3cf6dca0f744fd928 Signed-off-by: William Henrotin (whe) --- .../mrp_bom_overview_control_panel.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/addons/mrp/static/src/components/bom_overview_control_panel/mrp_bom_overview_control_panel.js b/addons/mrp/static/src/components/bom_overview_control_panel/mrp_bom_overview_control_panel.js index e722696fb1b..f9509a6ed4e 100644 --- a/addons/mrp/static/src/components/bom_overview_control_panel/mrp_bom_overview_control_panel.js +++ b/addons/mrp/static/src/components/bom_overview_control_panel/mrp_bom_overview_control_panel.js @@ -5,7 +5,6 @@ import { BomOverviewDisplayFilter } from "../bom_overview_display_filter/mrp_bom import { Dropdown } from "@web/core/dropdown/dropdown"; import { DropdownItem } from "@web/core/dropdown/dropdown_item"; import { Many2XAutocomplete } from "@web/views/fields/relational_utils"; -import { formatFloat } from "@web/views/fields/formatters"; const { Component } = owl; @@ -17,7 +16,7 @@ export class BomOverviewControlPanel extends Component { //---- Handlers ---- updateQuantity(ev) { - const newVal = isNaN(ev.target.value) ? 1 : parseFloat(formatFloat(parseFloat(ev.target.value), { digits: [false, this.precision] })); + const newVal = isNaN(ev.target.value) ? 1 : parseFloat(parseFloat(ev.target.value).toFixed(this.precision)); this.props.changeBomQuantity(newVal); }