Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][FIX] l10n_br_sale: Fatura criada a partir do Pedido de Vendas deve usar o Partner do campo partner_invoice_id #2864

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Jan 16, 2024

Invoice should be create with partner_invoice_id not with partner_id.

Fatura criada a partir do Pedido de Vendas deve usar o Partner do campo partner_invoice_id, PR simples que mantem o comportamento padrão do modulo Sale, originalmente esse commit estava no PR #2849 mas para facilitar a revisão estou extraindo de lá, o problema pode ser visto na tela com os Dados de Demonstração:

Apesar do campo partner_invoice_id ter um Partner diferente do campo partner_id

image

a Fatura é criada com o valor do partner_id

image

Esse comportamento não é o padrão do modulo Sale e isso pode ser visto criando um Pedido que não tem Operação Fiscal, que pode ser o Caso Internacional

image

image

Isso acontece porque o dicionario de dados chamado pelo método Fiscal esta trazendo o valor do partner_id https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_sale/models/sale_order.py#L185 e quando é feito o update do dicionario de dados do metodo original o valor é substituido( O método ._prepare_br_fiscal_dict deveria resolver essa questão? Deixei um TODO sobre isso )

Com esse PR a Localização passa a ter o mesmo comportamento do modulo original, permitindo os Casos de Uso onde o partner_invoice_id pode ser diferente do partner_id

image

image

cc @renatonlima @rvalyi @marcelsavegnago @mileo

@OCA-git-bot
Copy link
Contributor

Hi @renatonlima,
some modules you are maintaining are being modified, check this out!

result.update(self._prepare_br_fiscal_dict())
# TODO: O metodo prepare_br_fiscal_dict retorna o partner_id
# principal e não está considerando os casos do partner_invoice_id
# diferente do partner, deveria?
Copy link
Member

@marcelsavegnago marcelsavegnago Jan 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mbcosta entendo que vamos ter pedidos desta forma. Neste caso entendo que os dados de endereço para a fatura deve ser do partner_invocie_id (se houver dado) e os dados fiscais deve pegar do parceiro comercial

image

no modo nativo o parceiro que vai para a invoice é partner_invoice_id

image

@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

O Neto está trabalhando com uma proposta para o fiscal tbm.. #2520

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No modo nativo o VAT nem é editável e entendo que o CNPJ/CPF tbm nao deve ser

image

@mileo
Copy link
Member

mileo commented Jul 30, 2024

#3185 (review)

@marcelsavegnago
Copy link
Member

@mbcosta tem mais alguma coisa para fazer nesta PR ?

@mbcosta
Copy link
Contributor Author

mbcosta commented Sep 27, 2024

valeu @marcelsavegnago nesse PR acabava faltando o mapeamento do res.partner Fiscal e campos related mas como isso estava no PR #2849 e houve o merge acredito que agora esse PR passou a ser desnecessário, vou fechar o PR, se tiver problemas e só avisar

@mbcosta mbcosta closed this Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants