odoo/o-spreadsheet#6094

Created by fw-bot
Merged at 68931ee59de7639b1d52b63e06a073e2364717eb

Statuses:

label
odoo:18.0-saas-17.4-fix-copy-paset-cf-adrm-428969-fw
head
9f435e8460fbced5412b88dcd09e83250e82a42e
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

Forward-Port-Of: #6080