audrius/contract
1
0
Fork 1
mirror of https://github.com/OCA/contract.git synced 2025-02-13 17:57:24 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] contract: consistency between contract count and contract action

Browse Source 
On the commercial entity, the count shows all contracts of child partners.
However clicking on the action might show no contract, as the action only
restricted the domain to the partner itself.
We extract the domain into a function to share it in both cases.
This commit is contained in:
nans
2021-02-16 13:35:39 +01:00
committed by Jean-Charles Drubay
parent 63b0677098
commit e80d79a1e6
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
contract/models/res_partner.py
View File

@@ -17,10 +17,14 @@ class ResPartner(models.Model):
comodel_name="contract.contract", inverse_name="partner_id", string="Contracts", comodel_name="contract.contract", inverse_name="partner_id", string="Contracts",
) )
def _get_partner_contract_domain(self):
self.ensure_one()
return [("partner_id", "child_of", self.ids)]
def _compute_contract_count(self): def _compute_contract_count(self):
contract_model = self.env["contract.contract"] contract_model = self.env["contract.contract"]
fetch_data = contract_model.read_group( fetch_data = contract_model.read_group(
[("partner_id", "child_of", self.ids)], self._get_partner_contract_domain(),
["partner_id", "contract_type"], ["partner_id", "contract_type"],
["partner_id", "contract_type"], ["partner_id", "contract_type"],
lazy=False, lazy=False,
@@ -51,7 +55,7 @@ class ResPartner(models.Model):
res = self._get_act_window_contract_xml(contract_type) res = self._get_act_window_contract_xml(contract_type)
action_context = {k: v for k, v in self.env.context.items() if k != "group_by"} action_context = {k: v for k, v in self.env.context.items() if k != "group_by"}
action_context["search_default_partner_id"] = self.id action_context["domain"] = self._get_partner_contract_domain()
action_context["default_partner_id"] = self.id action_context["default_partner_id"] = self.id
action_context["default_pricelist_id"] = self.property_product_pricelist.id action_context["default_pricelist_id"] = self.property_product_pricelist.id
res["context"] = action_context res["context"] = action_context