Yaltik/account-financial-tools
2
0
Fork 0
Code Pull Requests Activity

[FIX] account_asset_management: make asset company match move's

Browse Source 
To start, the context `company_id` is no longer used in v14. Instead, we use `with_company`.

OTOH, it doesn't make much sense because in case you have several companies enabled in your context, not necessarily your main one is gonna be the one that should be used for the asset. Instead, it should be the move company always, or consistency rules will fail.

Altered forward port of https://github.com/OCA/account-financial-tools/pull/1223.

@Tecnativa TT31311
This commit is contained in:
Jairo Llopis 2021-08-12 13:31:37 +01:00 committed by Rodrigo
parent 8dc37aa648
commit ca744428ef
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
account_asset_management/models/account_move.py
View File

@ -79,14 +79,10 @@ class AccountMove(models.Model):
"date_start": move.date,
"account_analytic_id": aml.analytic_account_id,
}
if self.env.context.get("company_id"):
vals["company_id"] = self.env["res.company"].browse(
self.env.context["company_id"]
)
asset_form = Form(
self.env["account.asset"].with_context(
create_asset_from_move_line=True, move_id=move.id
)
self.env["account.asset"]
.with_company(move.company_id)
.with_context(create_asset_from_move_line=True, move_id=move.id)
)
for key, val in vals.items():
setattr(asset_form, key, val)