Skip to content

[fork-CI] [19.0][MIG] database_size: Migration to 19.0#6

Draft
dnplkndll wants to merge 3 commits into
19.0from
19.0-mig-database_size
Draft

[fork-CI] [19.0][MIG] database_size: Migration to 19.0#6
dnplkndll wants to merge 3 commits into
19.0from
19.0-mig-database_size

Conversation

@dnplkndll
Copy link
Copy Markdown

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>
@dnplkndll dnplkndll force-pushed the 19.0-mig-database_size branch from 2cc9f09 to c6d731d Compare May 29, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants