odoo/o-spreadsheet#6074

Created by Fun Subscription - Sign - Cal, Florian Damhaut (flda)
Merged at d214ef0037752ca901c1d9a9c3984d630a94a51b

Statuses:

label
odoo:master-spreadsheet-registry-unicity-flda
head
83b32dcf34998f00652074116781497e3df1991d
merged
8 months ago by BI, Lucas Lefèvre (lul)
odoo/o-spreadsheet
master #6074

[IMP] registry: add key unicity on add

Throw an error if we try to add a key to
the registry that is already present.

.replace() allow to add/change an already
present key.

task-4712321

Description:

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

Task: 4712321

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