Skip to content

[19.0][MIG] database_size: Migration to 19.0#3632

Open
dnplkndll wants to merge 3 commits into
OCA:19.0from
ledoent:19.0-mig-database_size
Open

[19.0][MIG] database_size: Migration to 19.0#3632
dnplkndll wants to merge 3 commits into
OCA:19.0from
ledoent:19.0-mig-database_size

Conversation

@dnplkndll
Copy link
Copy Markdown
Contributor

Port of database_size from 18.0 to 19.0 (migration guide).

Non-mechanical adaptations worth flagging

  • read_group override → _read_group: 19.0 deprecated the public read_group (would fail checklog) and the web client no longer routes group views through it, so the old override was both noisy and dead. Re-implemented on the non-deprecated _read_group with the 19.0 signature (groupby/aggregates/having/order); it still just defaults the group order for the size-report views.
  • _sql_constraintsmodels.Constraint (kept the uniq_model_measurement_date name).
  • _compute_model_name: justified # pylint: disable=no-search-all on the ir.model load (small registry table; new 19.0 pylint-odoo check). The information_schema references are raw table-size SQL, unrelated to translatable-field jsonb assumptions.

StefanRijnhart and others added 3 commits May 29, 2026 13:43
Signed-off-by: Don Kendall <dkendall@ledoweb.com>
- read_group override → _read_group (19.0 deprecated public read_group + web no longer routes through it). New signature (groupby/aggregates/having/order); still defaults the group order for the report views.
- search view: dropped `expand`/`string` from the Group By <group> (19.0 RelaxNG rejects them on search-view groups).
- _sql_constraints → models.Constraint (kept uniq_model_measurement_date name).
- report: f-strings inside self.env._() → %(name)s placeholders (19.0 pylint translation-fstring-interpolation).
- _compute_model_name: justified `# pylint: disable=no-search-all` (small ir.model registry load).

Signed-off-by: Don Kendall <dkendall@ledoweb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants