Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] mrp_bom_reference_selection: Fixed onchange error closes #62 #63

Merged
merged 2 commits into from Dec 10, 2015
Merged

[FIX] mrp_bom_reference_selection: Fixed onchange error closes #62 #63

merged 2 commits into from Dec 10, 2015

Conversation

oihane
Copy link
Contributor

@oihane oihane commented Dec 9, 2015

No description provided.

@@ -49,23 +50,22 @@ def onchange_product_id(self, product_id):

return res

@api.one
@api.depends('product_id')
def _get_child_bom_lines(self):
Copy link
Member

Choose a reason for hiding this comment

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

call it compute...

@oihane
Copy link
Contributor Author

oihane commented Dec 9, 2015

@rafaelbn @pedrobaeza this fixes error reported by @anajuaristi @ybofr in the PR of mrp_hook and mrp_operations_extension

@pedrobaeza
Copy link
Member

👍

Fast-tracking for solving the current problem.

pedrobaeza added a commit that referenced this pull request Dec 10, 2015
[FIX] mrp_bom_reference_selection: Fixed onchange error closes #62
@pedrobaeza pedrobaeza merged commit d9a321b into OCA:8.0 Dec 10, 2015
@pedrobaeza pedrobaeza deleted the 8.0-fix-mrp_bom_reference_selection branch December 10, 2015 15:27
JoanSForgeFlow pushed a commit to ForgeFlow/manufacture that referenced this pull request Jul 26, 2021
* Show wizards results
* Fix refunds
* Fixes OCA#61
* Fixes OCA#63.
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.

None yet

2 participants