From 73a7e02fc2cd88aeec2b80f6b604044380bc81e2 Mon Sep 17 00:00:00 2001 From: Sebastian Widmer Date: Fri, 17 May 2024 16:00:43 +0200 Subject: [PATCH] Odoo 16 Storage: Fix loading wrong partner in `get` (#206) `list` already loads correct partner --- apiserver/billing/odoostorage/odoo/odoo16/odoo16.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apiserver/billing/odoostorage/odoo/odoo16/odoo16.go b/apiserver/billing/odoostorage/odoo/odoo16/odoo16.go index fa327ab..9a7a68c 100644 --- a/apiserver/billing/odoostorage/odoo/odoo16/odoo16.go +++ b/apiserver/billing/odoostorage/odoo/odoo16/odoo16.go @@ -151,7 +151,7 @@ func (s *Odoo16Storage) get(ctx context.Context, name string) (company odooclien } cpp, err := session.FindResPartners( - odooclient.NewCriteria().AddCriterion(activeFilter).AddCriterion(odooclient.NewCriterion("id", "=", id)), + odooclient.NewCriteria().AddCriterion(activeFilter).AddCriterion(odooclient.NewCriterion("id", "=", accountingContact.ParentId.Get())), fetchPartnerFieldOpts) if err != nil { return odooclient.ResPartner{}, odooclient.ResPartner{}, fmt.Errorf("fetching parent %d of accounting contact %d failed: %w", accountingContact.ParentId.ID, id, err) @@ -211,7 +211,7 @@ func (s *Odoo16Storage) List(ctx context.Context) ([]billingv1.BillingEntity, er } mp, ok := companySet[int(p.ParentId.ID)] if !ok { - l.Info("could not load parent partner (maybe no longer active?)", "parent_id", p.ParentId.ID, "id", p.Id) + l.Info("could not load parent partner (maybe no longer active?)", "parent_id", p.ParentId.ID, "id", p.Id.Get()) continue } bes = append(bes, mapPartnersToBillingEntity(ctx, mp, p))