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_fiscal: Os valores do frete, seguro e 'outros' não estão sendo inseridos no valor total da fatura #3484

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

corredato
Copy link
Contributor

Veja: #2666

Segui a ideia do @mileo de abrir uma PR para discutirmos a melhor maneira de consertar esse problema.

Resumindo: o inverse desses campos funcionam parcialmente, eles realizam a distribuição proporcional do valor inserido no cabeçalho nas linhas da fatura. Porém, essa inserção dos valores nas linhas apenas atribui um valor ao campo, e não chama todos os métodos necessários para computar o total da fatura, diferente de quando o valor na linha é alterado manualmente.

Nesse commit, adicionei a chamada de um método do core que entrava na pilha de chamadas quando o valor era alterado manualmente na linha, ele adiciona um valor na fatura, mas dá um erro de diferença de débito e crédito.

Vou continuar investigando, acredito que a solução será algo parecido com isso, alguém tem alguma sugestão?

@antoniospneto @rvalyi @marcelsavegnago @mbcosta

@OCA-git-bot
Copy link
Contributor

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

@DiegoParadeda DiegoParadeda force-pushed the fix/freight-insurance-other-values branch from 3c8cd01 to 4241648 Compare November 25, 2024 14:45
@DiegoParadeda DiegoParadeda force-pushed the fix/freight-insurance-other-values branch from 4241648 to 7804cb1 Compare November 25, 2024 17:03
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.

3 participants