[FW][FIX] Datavalidation: coreView plugin should not dispatch

fw-bot commented 5 days ago

See #5216 - CoreView plugins replay remote revisions and should not take the risk to dispatch during the replay. The code that was assigning arbitrary default values to the cells of a boolean datavalidation has been moved to a UI feature plugin, hence avoiding the problem raised in PR #5216.

Task: 4241141

Description:

description of this task, what is implemented and why it is implemented that way.

Task: TASK_ID

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: odoo/o-spreadsheet#5217 Forward-Port-Of: odoo/o-spreadsheet#5071

robodoo commented 5 days ago

fw-bot commented 5 days ago

@rrahir @pro-odoo this PR targets master and is the last of the forward-port chain containing:

odoo/o-spreadsheet#5221

To merge the full chain, use

@robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

rrahir commented 5 days ago

@robodoo r+

odoo / o-spreadsheet

[FW][FIX] Datavalidation: coreView plugin should not dispatch #5222

Description:

review checklist