odoo/o-spreadsheet#7430
Created by fw-bot
Merged
at b5c217ff8da30e45f9f02066ca68495c3b9f9d6d
Statuses:
- label
- odoo:saas-18.4-17.0-fix-invalid-sheet-name-adrm-475769-fw
- head
- 33f6804c289fd519d0e3c3a8ce42c4c66591d4f7
- merged
- 3 weeks ago by BI, Adrien Minne (adrm)
| odoo/o-spreadsheet | |
|---|---|
| 17.0 | #7337 |
| 18.0 | #7416 |
| saas-18.2 | #7419 |
| saas-18.3 | #7420 |
| saas-18.4 | #7430 |
| 19.0 | #7433 |
| master | #7445 |
[FIX] range: invalid sheet name with special character
Description
If we have a range with an invalid sheet name, the sheet name is not escaped with quotes when it contains special characters when converting the range back to a string.
Task: 5125762
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