l10n-brazil icon indicating copy to clipboard operation
l10n-brazil copied to clipboard

[RFC] Avaliar a possibilidade buscar os dados fiscais através de partner_id.commercial_partner_id

Open marcelsavegnago opened this issue 2 years ago • 7 comments

Hoje todos os dados fiscais inclusive os calculos dos impostos dependem diretamente do partner_id.campox , y ou z

Acho que pode ser interessante mudar para partner_id.commercial_partner_id.campox, y ou z

Nativamente se o parceiro tem um parent_id o comercial_partner_id é o parent_id e se nao tem o parent_id o comercial_partner_id é ele mesmo https://github.com/odoo/odoo/blob/a362257ef49a7e074ff2badd5f969777cb2d9585/odoo/addons/base/models/res_partner.py#L293

    @api.depends('is_company', 'parent_id.commercial_partner_id')
    def _compute_commercial_partner(self):
        for partner in self:
            if partner.is_company or not partner.parent_id:
                partner.commercial_partner_id = partner
            else:
                partner.commercial_partner_id = partner.parent_id.commercial_partner_id

Essa mudança vai além do fiscal e mexe tbm no nfe e outros


Referências enviadas pelo Raphael https://www.slideshare.net/RaphalValyi/openerp-contact-id-solution-to-the-contacts-issue-in-v7 https://github.com/OCA/sale-workflow/tree/14.0/sale_commercial_partner https://github.com/OCA/purchase-workflow/tree/14.0/purchase_commercial_partner

marcelsavegnago avatar Aug 05 '22 17:08 marcelsavegnago

Realmente @marcelsavegnago , para quem não usa o l10n_br_sale_stock ou o l10n_br_purchase_stock, esse problema pode ser serio. Eu fiz um PR para resolver isso https://github.com/OCA/l10n-brazil/pull/2075

rvalyi avatar Aug 09 '22 21:08 rvalyi

There hasn't been any activity on this issue in the past 6 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 issue to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Oct 15 '23 12:10 github-actions[bot]

Resolvido no https://github.com/OCA/l10n-brazil/pull/2520

mileo avatar Oct 15 '23 22:10 mileo

@marcelsavegnago conseguiu validar se o #2520 resolve ?

antoniospneto avatar Dec 12 '23 01:12 antoniospneto

@antoniospneto não testei mas é isso ai

marcelsavegnago avatar Dec 12 '23 15:12 marcelsavegnago