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] account: Calling oldname cash_basis_account instead of new field #28594

Closed
wants to merge 1 commit into from

Conversation

hbto
Copy link
Contributor

@hbto hbto commented Nov 12, 2018

Method _onchange_line_ids is calling oldname field cash_basis_account instead of new one cash_basis_account_id

https://youtu.be/hnqMc_LKUww

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added seen 🙂 CI 🤖 Robodoo has seen passing statuses labels Nov 12, 2018
@Yenthe666
Copy link
Collaborator

@qdp-odoo for your interest

@hbto
Copy link
Contributor Author

hbto commented Nov 13, 2018

@JulioSerna @moylop260 @nhomar can you check on this

By not merging this fix user will not be able to edit Journal Entries.

Regards.

@qdp-odoo
Copy link
Contributor

@robodoo r+

@robodoo
Copy link
Contributor

robodoo commented Nov 14, 2018

Merged, thanks!

@robodoo robodoo closed this Nov 14, 2018
@luisg123v luisg123v deleted the 12.0-cash_basis_account-hbto branch July 14, 2022 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants