odoo/o-spreadsheet#5932
Created by Mehdi Rachico (mera)
Merged
at e6e60cb10ada249bbced0cdf284de1c41ba7ce2c
Statuses:
- label
- odoo:saas-18.2-remove-ranges-in-dv-mera
- head
- 5e76d8405c5124d10e8435b93858c71586b39785
- merged
- 8 months ago by BI, Rémi Rahir (rar)
| odoo/o-spreadsheet | |
|---|---|
| saas-18.2 | #5932 |
| saas-18.3 | #6274 |
| saas-18.4 | |
| 19.0 | |
| master | #6275 |
[FIX] selection_input: fix range deletion
Prior to this commit, clicking the trash icon next to a range in DV/CF did not remove the range due to a missing event handler. This bug was introduced by this commit: 011e2237a56f0fa082d066a2aaea717b32d4c0e5
This commit ensures that the deletion action is correctly triggered, allowing a proper management of CF/DV rules
Steps to reproduce:
- Create a DV/CF rule
- Add a new range (valid or invalid)
- Click on the trash icon next to the range
- Nothing happens
Task: 4589387
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