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] Allow default values for payment mode in multi-company env #666

Closed
wants to merge 1 commit into from
Closed

[FIX] Allow default values for payment mode in multi-company env #666

wants to merge 1 commit into from

Conversation

CasVissers-360ERP
Copy link

If using ir.default for payment mode in a multi-company environment the payment mode of a sister company could be loaded if superuser (uid 1) was active in this sister company. Resulting in the error message:
_("The company of the invoice %s does not match with that of the payment mode") % rec.name)

This PR fixes this issue.

@pedrobaeza pedrobaeza added this to the 12.0 milestone Jan 8, 2020
@pedrobaeza
Copy link
Member

Please indicate steps to reproduce the problem. I don't think populating company_id in the refund fields is the solution to your problem.

@CasVissers-360ERP
Copy link
Author

@pedrobaeza I would like to create a runbot video, but OCA runbot seems not accessible?

@pedrobaeza
Copy link
Member

You can use the "Try me on runbot" button. Any way, I still prefer written steps than a hard to follow video. And as said, a technical explanation why this solves the problem is also welcome (included as comment/docstring of the method as well for future references).

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