odoo/o-spreadsheet#6080

Created by BI, Adrien Minne (adrm)

label

odoo:saas-17.4-fix-copy-paset-cf-adrm

head

c6ced146c5fb9c14bca1cb5a111c9ec42b5b1791

merged

1 day ago by BI, Vincent Schippefilt (vsc)

	odoo/o-spreadsheet
saas-17.4	#6080
18.0	#6094
saas-18.1	#6095
saas-18.2	#6096
master	#6098

[FIX] conditional format: huge revisions on copy/paste cf

Description

When copy/pasting a conditional format, the revision had one ADD_CONDITIONAL_FORMAT command per copied cell. That lead to huge revisions if, for example, a whole column with a CF was copied.

Now the revision only has a single ADD_CONDITIONAL_FORMAT command per modified CF.

The exact same work was done for data validation rules.

Task: 4718522

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