Skip to content

[18.0][FIX] edi_record_metadata_oca: rename get_metadata (ORM conflict)#258

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
camptocamp:18.0-fix-edi_record_metadata_oca-get_metadata
Apr 2, 2026
Merged

[18.0][FIX] edi_record_metadata_oca: rename get_metadata (ORM conflict)#258
OCA-git-bot merged 1 commit intoOCA:18.0from
camptocamp:18.0-fix-edi_record_metadata_oca-get_metadata

Conversation

@vvrossem
Copy link
Copy Markdown
Contributor

@vvrossem vvrossem commented Apr 1, 2026

get_metadata() is a built-in Odoo ORM method called by the "Debug > View Metadata" action. Defining it on edi.exchange.record overrode that behavior, causing a JS crash when opening the debug dialog:

"Cannot read properties of undefined (reading 'id')"

Renamed to edi_get_metadata() / edi_set_metadata() (for consistency) and updated all callers.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @simahawk,
some modules you are maintaining are being modified, check this out!

Copy link
Copy Markdown
Contributor

@SilvioC2C SilvioC2C left a comment

Choose a reason for hiding this comment

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

LGTM

Please squash the commits or simply use git cherry-pick to forward-port commits from #257

`get_metadata()` is a built-in Odoo ORM method called by the
"Debug > View Metadata" action. Defining it on `edi.exchange.record`
overrode that behavior, causing a JS crash when opening the debug
dialog:

  "Cannot read properties of undefined (reading 'id')"

Renamed to `edi_get_metadata()` / `edi_set_metadata()` (for consistency)
and updated all callers.
@vvrossem vvrossem force-pushed the 18.0-fix-edi_record_metadata_oca-get_metadata branch from 7af7cc0 to 42a5202 Compare April 2, 2026 07:41
@simahawk
Copy link
Copy Markdown
Contributor

simahawk commented Apr 2, 2026

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-258-by-simahawk-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e2e5cc7 into OCA:18.0 Apr 2, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at ed73890. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants