odoo/o-spreadsheet#7621
Created by BI, Hendrickx Anthony (anhe)
Merged
at e8c577b85d0f4cacc0c4cc65beb135c966ab2787
Statuses:
- label
- odoo:master-chart-fix_zoom_slicer-anhe
- head
- 7c0492643520c1e1b80bab80f2fbbffc2108533b
- merged
- 5 days ago by BI, Pierre Rousseau (pro)
| odoo/o-spreadsheet | |
|---|---|
| master | #7621 |
[FIX] chart: fix zoom plugin
Task Description
This commit aims to fix the usage of the zoomWindowPlugin, which draws the zoomed window boundaries on the master chart in a chart.
Due to file order, when first calling the registerChartJsExtensions method, the zoomWindowPlugin wasn't added in the chartJsExtensionRegistry yet, and was therefore not added. Unfortunately, each following call to registerChartJsExtension were ignored by the early-return checking if the chartShowValuesPlugin
was already registered, which is the case.
This fix adds the zoomWindowPlugin in the registry at the same time as the others plugins, fixing this issue by ensuring every plugin is in the registry when we first call the registerChartJsExtension method.
Related Task
N./A.