odoo/o-spreadsheet#6096
Created by fw-bot
Merged
at 74911e3e531fc825f299247577da173a6eafaaf4
Statuses:
- label
- odoo:saas-18.2-saas-17.4-fix-copy-paset-cf-adrm-429322-fw
- head
- a6ab3adf34fea19a268a049b5ac0ab430170d1a4
- merged
- 1 day ago by BI, Adrien Minne (adrm)
| odoo/o-spreadsheet | |
|---|---|
| saas-17.4 | #6080 |
| 18.0 | #6094 |
| saas-18.1 | #6095 |
| saas-18.2 | #6096 |
| master | #6098 |
[FW][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