Skip to content

[19.0][MIG] web_widget_x2many_2d_matrix: Migration to 19.0#3310

Open
bizzappdev wants to merge 153 commits into
OCA:19.0from
BizzAppDev-Systems:19.0-mig-web_widget_x2many_2d_matrix-BAD
Open

[19.0][MIG] web_widget_x2many_2d_matrix: Migration to 19.0#3310
bizzappdev wants to merge 153 commits into
OCA:19.0from
BizzAppDev-Systems:19.0-mig-web_widget_x2many_2d_matrix-BAD

Conversation

@bizzappdev

Copy link
Copy Markdown
Contributor

No description provided.

hbrunn and others added 30 commits October 8, 2025 16:22
of one of the axes is a many2one field
[ADD] show grand total
[IMP] classify floats as floats
[IMP] write formatted value to back to input
[RFR] and being on it, make reacting to changes in overrides simple
this is necessary for correct operation after creating new records
* README update to newest OCA template
* Example in README
* Massive performance boost for big matrices, specially on Firefox
* Assign id on row in order to find it back in all cases
* Fix OCA#321, choked on cached writes
Declare as many options prefixed with this string as you need for binding
a field value with an HTML node attribute (disabled, class, style...)
called as the `<name>` passed in the option.

NOTE: This doesn't prevent to require to fill the full matrix with
all the combination records.
…s_clickable attrs

XML attributes for the widget that allows to configure if the axis will be clickable
or not in case the source field is a many2one field.
hbrunn and others added 18 commits October 8, 2025 16:22
<=v15 used to pass the list view field node to the rendering widget,
enabling full configurability of the value field
…ckable)

it was possible to make the axis fields links to the record if they were
many2one fields
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: web-18.0/web-18.0-web_widget_x2many_2d_matrix
Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_widget_x2many_2d_matrix/
Currently translated at 100.0% (7 of 7 strings)

Translation: web-18.0/web-18.0-web_widget_x2many_2d_matrix
Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_widget_x2many_2d_matrix/it/
use components to render axes and aggregates, allowing custom widgets
there
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: web-18.0/web-18.0-web_widget_x2many_2d_matrix
Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_widget_x2many_2d_matrix/
Currently translated at 100.0% (8 of 8 strings)

Translation: web-18.0/web-18.0-web_widget_x2many_2d_matrix
Translate-URL: https://translation.odoo-community.org/projects/web-18-0/web-18-0-web_widget_x2many_2d_matrix/it/

@hbrunn hbrunn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot migration web_widget_x2many_2d_matrix

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Nov 14, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 14, 2025
46 tasks
@mav-adhoc

Copy link
Copy Markdown

LGTM

Comment on lines +29 to +30
column.text = column.value[1];
column.value = column.value[0];

@ferran-S73 ferran-S73 Feb 17, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@bizzappdev This isn't working as expected. Since v19 Odoo stores m2o fields in javascript as a dict instead of the usual tuple with display name and id.

The correct way to access this data should be as follows

Suggested change
column.text = column.value[1];
column.value = column.value[0];
column.text = column.value.display_name;
column.value = column.value.id;

Please also check the rest of the module to make sure if this change is needed elsewhere

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

good catch! please also extend the tests to check this

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi, I opened a new PR fixing this issue as we needed this module. Feel free to use my fixed code in your PR if you wish

Comment on lines +29 to +30
column.text = column.value[1];
column.value = column.value[0];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

good catch! please also extend the tests to check this

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.