odoo/enterprise#112849
Created by Accounting, Seif Gneedy (segn)
Statuses:
- legal/cla: Contributor License Agreement check
- ci/runbot: Odoo Test Suite
- ci/upgrade_enterprise: Test upgrades for enterprise master
- ci/style: Optional style check. Ignore it only if strictly necessary.
- ci/security: Required security check. Can only be ignored by security team.
- ci/l10n: (runtime 27s)
- label
- odoo-dev:master-poc-groupby-agg-segn
- head
- 5d8fd7350fc410825811d5ae808dcaf356d6708e
- merged
- 1 day ago by Accounting, Olivier Colson (oco)
| odoo/odoo | odoo/enterprise | odoo/upgrade | odoo/documentation | |
|---|---|---|---|---|
| master | #257280 | #112849 | #9835 | #17463 |
Master Support groupby on aggregations
[IMP] account_reports: foldability as selection field
-
Adapt AccountReportLineData, _get_groupby, and tests to the new 'foldability' selection.
-
Apply upgrade_code script for removing the unnecessary foldable field in reports data files if they
will be correctly computed. -
For inconsistencies reported by the script, manually set the foldable values on the reports and tests.
[IMP] account_reports: Support groupby on aggregations
The aggregation engine didn't support groupby before due to limitations
existed from the beginning of Reportalypse.
With the removal of the 'load more'<sup>1</sup> feature, other engines don't run a SQL LIMIT anymore.
Resulting on the aggregation engine can obtain the full result, and
'groupby' calculation be done.
That support will help with incoming improvements: Snapshot feature<sup>2</sup> and
Tax closing computation refactoring
<sup>1</sup> : ff2895144ecfc66d2b23bb872b39f5b446ae0da6
<sup>2</sup> : #113677