Skip to content

[IMP] */saas~19.5: remove Field.size from python - #505

Open
kmagusiak wants to merge 1 commit into
odoo:masterfrom
odoo-dev:master-fields-char-size-rm-krma
Open

[IMP] */saas~19.5: remove Field.size from python#505
kmagusiak wants to merge 1 commit into
odoo:masterfrom
odoo-dev:master-fields-char-size-rm-krma

Conversation

@kmagusiak

Copy link
Copy Markdown
Contributor

The Field.size attribute is deprecated for years. Use it only for UI as this silently trucates values.

odoo/odoo#284290

@robodoo

robodoo commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Pull request status dashboard

@KangOl

KangOl commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

upgradeci retry with always only base

Comment thread src/base/tests/test_util.py Outdated
("res_country", "name", False, "jsonb" if util.version_gte("16.0") else "varchar"), # translated field
("res_country", "code", False, "varchar"),
("res_country", "code", True, "varchar(2)"),
("res_country", "code", True, "varchar"),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. You broke the test for older versions.

@kmagusiak
kmagusiak force-pushed the master-fields-char-size-rm-krma branch from 234c7f6 to 904092c Compare August 31, 2026 20:51
The `Field.size` attribute is deprecated for years. Use it only for UI
as this silently trucates values.

odoo/odoo#284290
@kmagusiak
kmagusiak force-pushed the master-fields-char-size-rm-krma branch from 904092c to 4d822a7 Compare September 1, 2026 07:31
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.

3 participants