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