[18.0][MIG] onchange_helper: Migration to 18.0 - #3052
Merged
Merged
Conversation
other improvement: set onchange_specs to all fields as _onchange_spec() retrun onchange fields for default view return field value if it's set in onchange_fields (usuful to get default value for a field)
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-tools-12.0/server-tools-12.0-onchange_helper Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-onchange_helper/
Currently translated at 100.0% (1 of 1 strings) Translation: server-tools-12.0/server-tools-12.0-onchange_helper Translate-URL: https://translation.odoo-community.org/projects/server-tools-12-0/server-tools-12-0-onchange_helper/zh_CN/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-tools-16.0/server-tools-16.0-onchange_helper Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-onchange_helper/
Currently translated at 100.0% (1 of 1 strings) Translation: server-tools-16.0/server-tools-16.0-onchange_helper Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-onchange_helper/es/
Currently translated at 100.0% (1 of 1 strings) Translation: server-tools-16.0/server-tools-16.0-onchange_helper Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-onchange_helper/it/
Without explicit web dependency all tests fail like this:
File "/home/odoo/src/odoo/odoo/models.py", line 6905, in onchange
raise NotImplementedError("onchange() is implemented in module 'web'")
NotImplementedError: onchange() is implemented in module 'web'
…ed by As the onchange method can be a little bulky, we don't call it if fields passed as arguments are not in _onchange_methods
thienvh332
marked this pull request as ready for review
October 8, 2024 05:24
Contributor
|
/ocabot migration onchange_helper |
Contributor
|
/ocabot merge nobump |
Contributor
|
Hey, thanks for contributing! Proceeding to merge this for you. |
OCA-git-bot
added a commit
that referenced
this pull request
Oct 10, 2024
Contributor
|
It looks like something changed on |
Contributor
|
Congratulations, your PR was merged at e2b7cfa. Thanks a lot for contributing to OCA. ❤️ |
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
20 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.
No description provided.