odoo/o-spreadsheet#4616

Created by fw-bot

label

odoo:saas-17.3-17.0-chart-not-updating-adrm-fOmt-fw

head

c5387c8d8785c9dce9a80bf4ccf69d0bc1ffafc2

merged

3 months ago by BI, Adrien Minne (adrm)

	odoo/o-spreadsheet
17.0	#4572
saas-17.1	#4602
saas-17.2	#4615
saas-17.3	#4616
master	#4617

[FW][REV] chart: avoid useless chart updates

Description

This reverts commit 6eb43533d.

It turns out that checking the deep equality of runtime while ignoring functions is not a good fix to avoid useless chart updates. The problem is that some runtime changes apply only to the callbacks (eg. changing the dataset format only change the ticks callback).

The only real alternative would be to add the variables that are used in the callbacks to the runtime, so the deepEquals would work. But this is very error prone: we'll 100% forget to add a variable at some point.

So we will accept the useless updates for now, until we see a real performance issue.

Task: : 4029016

review checklist

• [ ] feature is organized in plugin, or UI components
• [ ] support of duplicate sheet (deep copy)
• [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
• [ ] in model/UI: ranges are strings (to show the user)
• [ ] undo-able commands (uses this.history.update)
• [ ] multiuser-able commands (has inverse commands and transformations where needed)
• [ ] new/updated/removed commands are documented
• [ ] exportable in excel
• [ ] translations (_t("qmsdf %s", abc))
• [ ] unit tested
• [ ] clean commented code
• [ ] track breaking changes
• [ ] doc is rebuild (npm run doc)
• [ ] status is correct in Odoo

Forward-Port-Of: #4602
Forward-Port-Of: #4572