odoo/o-spreadsheet#8794
Created by BI, Adrien Minne (adrm)
Merged
at b16b3ce5d89dc759ae1346f5d8e24784ff325e4d
Statuses:
- label
- odoo:17.0-perf-menu-component-adrm
- head
- 196e995a56a4da95e7ad9860109d6c40b0ef688d
- merged
- 1 week ago by BI, Lucas Lefèvre (lul)
| odoo/o-spreadsheet | |
|---|---|
| 17.0 | #8794 |
| 18.0 | #8795 |
| saas-18.2 | #8796 |
| saas-18.3 | #8797 |
| saas-18.4 | #8798 |
| 19.0 | #8799 |
| saas-19.1 | #8802 |
| saas-19.2 | #8803 |
| saas-19.3 | #8805 |
| master | #8807 |
[PERF] menu: avoid quadratic complexity with proxies
In the menu component, we would compute whether to display the icons for each menu item. But to compute that, we need to loop on each menu items, leading to a quadratic complexity.
This could have been good enough as menus aren't too large, but the menu items are proxified by Owl, making the loop very slow.
Task: 6250318
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