Skip to content

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

Closed
FREDDYGAMBOABEDOR wants to merge 13 commits into
OCA:19.0from
FREDDYGAMBOABEDOR:19.0-mig-database_size
Closed

[19.0][MIG] database_size: Migration to 19.0#3664
FREDDYGAMBOABEDOR wants to merge 13 commits into
OCA:19.0from
FREDDYGAMBOABEDOR:19.0-mig-database_size

Conversation

@FREDDYGAMBOABEDOR

Copy link
Copy Markdown

Migration of database_size from 18.0 to 19.0.

Changes:

  • Bumped module version to 19.0.1.0.0
  • Converted _sql_constraints to the new models.Constraint syntax
  • Removed deprecated expand/string attributes from the <group> tag in the search view (groupby)
  • Updated ir_model_size_report.py translation calls to use named placeholders instead of f-strings, per pylint-odoo translation-fstring-interpolation/translation-positional-used rules

StefanRijnhart and others added 13 commits June 27, 2026 22:36
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/
@FREDDYGAMBOABEDOR

Copy link
Copy Markdown
Author

/ocabot migration database_size

@FREDDYGAMBOABEDOR

FREDDYGAMBOABEDOR commented Jun 28, 2026

Copy link
Copy Markdown
Author

Hi! This is my migration of database_size to 19.0. All checks are green and the git history has been preserved.

Could someone with the appropriate permissions please run the migration command for this addon and review the PR when you have time?

Thanks!

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @FREDDYGAMBOABEDOR. Your command failed:

Invalid options for command migration: database_size and take a look when you have a moment.

Ocabot commands

  • ocabot merge major|minor|patch|nobump
  • ocabot rebase
  • ocabot migration {MODULE_NAME}

More information

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Sorry @FREDDYGAMBOABEDOR you are not allowed to mark the addon to be migrated.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@StefanRijnhart

Copy link
Copy Markdown
Member

/ocabot migration database_size

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Jun 29, 2026
@StefanRijnhart

Copy link
Copy Markdown
Member

Thank you! Unfortunately, someone was there before you: #3632. Can you help review that one?

@FREDDYGAMBOABEDOR

Copy link
Copy Markdown
Author

No problem at all, thanks for letting me know! I'll be happy to review #3632. I'll also pick another unassigned module from #3400 to migrate. Thanks!

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.

7 participants