-
-
Notifications
You must be signed in to change notification settings - Fork 397
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
[16.0][FIX] account_reconcile_oca: UnboundLocalError: local variable … #595
Conversation
Hi @etobella, |
dc77af2
to
82c07d5
Compare
Can you squash your commits? |
…'balance' referenced before assignment
82c07d5
to
7e543c0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested this and it works. Please merge this fix as soon as possible since other modules are depending on this fix (and they are failing now). TIA
I don't feel comfortable with this change. Which is the definition of the model that is giving you this error? |
@etobella this is something you fixed when porting the module to V16: #500 (comment) But then the bug re-appeard after your PR: #580 So the problem is standard odoo method |
Don't make a verbatim copy of the method, but overwrite |
This is a quick&dirty fix and i don't think I will have time to change it. If anyone will take over and propose a better solution I will be happy. Shall I close this one? |
Hi, |
Thank you @ACheung-FactorLibre |
…'balance' referenced before assignment
Fixes #592, by applying the patch proposed here: odoo/odoo#123626
This is a quick&dirty fix, waiting for better ideas on how to fix it.