[FIX] *_operating_unit: add check_company in field OU
Step to test:
-
Add multi company and create OU
-
Go to invoice, po, so, exp or else > select Operating Unit field it will show all OU
This document created on YourCompany it should select OU that depend on YourCompany only
This PR fixed by add check_company=True in header field.
Exclude: line, compute field, wizard (may be it should add too. but I didn't test)
After add it
Hi @max3903, @ps-tubtim, some modules you are maintaining are being modified, check this out!
Too bad. It seems OU modules are not multicompany consistent :S
The changes are good to me, and I think we should merge this. Multicompany production instances may need to check their data before applying this change. I will wait for other contributors to give their opinion first.
I think having a commit per module is required in OCA, sorry for the pain.
@AaronHForgeFlow No problem. I think it’s the right thing to do. However, should we start fixing this from version 18?
Because so far, no one has multi ou in multi company.
I don't use this in a multicompany environment and I won't be affected. In any case I think it is best to fix the bug in any version.
I checked and it seems to be already solved in v17. I did not test v16.
It is ok to merge it and check if needed in v16
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.