[fork-CI] [19.0][MIG] database_size: Migration to 19.0#6
Draft
dnplkndll wants to merge 10 commits into
Draft
Conversation
dnplkndll
force-pushed
the
19.0-mig-database_size
branch
from
2cc9f09 to
c6d731d
Compare
Currently translated at 100.0% (68 of 68 strings) Translation: server-tools-18.0/server-tools-18.0-database_size Translate-URL: https://translation.odoo-community.org/projects/server-tools-18-0/server-tools-18-0-database_size/it/
Currently translated at 100.0% (68 of 68 strings) Translation: server-tools-18.0/server-tools-18.0-database_size Translate-URL: https://translation.odoo-community.org/projects/server-tools-18-0/server-tools-18-0-database_size/it/
Fixes
```
File "/home/odoo/.local/lib/python3.12/site-packages/sentry_sdk/client.py", line 595, in _prepare_event
new_event = before_send(event, hint or {})
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/odoo/src/user/oca-server-tools/sentry/hooks.py", line 66, in before_send
if event.setdefault("tags", {})["include_context"]:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^
KeyError: 'include_context'
2025-07-08 14:45:45,907 14296 INFO quatra-18-0-development-21441249 sentry_sdk.errors: before send dropped event
```
Error is only logged when running Sentry in debug mode, otherwise events are dropped silently.
Though TransactionCase rolls back database changes, it does not call `unlink` on `auditlog.rule` records, meaning that `unsubscribe` is never called to unpatch model methods. This causes subsequent tests in other modules to unexpectedly execute patched methods. This commit adds the `AuditLogRuleCommon` test class in an attempt to identify and prevent patched methods from contaminating the test environment.
Currently translated at 13.2% (9 of 68 strings) Translation: server-tools-18.0/server-tools-18.0-database_size Translate-URL: https://translation.odoo-community.org/projects/server-tools-18-0/server-tools-18-0-database_size/fr/
dnplkndll
force-pushed
the
19.0-mig-database_size
branch
from
c6d731d to
e562fc7
Compare
dnplkndll
force-pushed
the
19.0-mig-database_size
branch
from
e562fc7 to
fdbc49a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Port of
database_sizefrom 18.0 to 19.0 (migration guide).Non-mechanical adaptations worth flagging
read_groupoverride →_read_group: 19.0 deprecated the publicread_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_groupwith the 19.0 signature (groupby/aggregates/having/order); it still just defaults the group order for the size-report views._sql_constraints→models.Constraint(kept theuniq_model_measurement_datename)._compute_model_name: justified# pylint: disable=no-search-allon their.modelload (small registry table; new 19.0 pylint-odoo check). Theinformation_schemareferences are raw table-size SQL, unrelated to translatable-field jsonb assumptions.