[14.0][FIX] l10n_br_fiscal: Os valores do frete, seguro e 'outros' não estão sendo inseridos no valor total da fatura

[corredato]

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]

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

[rvalyi]

@corredato eu nao olhei ainda o que vc procurou resolver exatamente, mas de uma forma geral esse codigo era bem repetitivo e eu tinha começado a fatorizar ele melhor nesta PR #3027

Eu imagino que teria que fazer seus corretivos por cima dessa fatorização do codigo. Tb seria legal algum teste para garantir que o que vc esta corrigindo não vai passar a quebrar de novo, na hora das migrações por exemplo.

cc @DiegoParadeda @marcelsavegnago @antoniospneto @renatonlima @mbcosta

[corredato]

@corredato eu nao olhei ainda o que vc procurou resolver exatamente, mas de uma forma geral esse codigo era bem repetitivo eu eu tinha começado a fatorizar ele melhor nesta PR #3027

Eu imagino que teria que fazer seus corretivos por cima dessa fatorização do codigo. Tb seria legal algum teste para garantir que o que vc esta corrigindo não vai passar a quebrar de novo, na hora das migrações por exemplo.

cc @DiegoParadeda @marcelsavegnago @antoniospneto @renatonlima @mbcosta

Resumindo, esses três inverses quando eram chamados, não estavam alterando o valor total da fatura. Eles inserem um valor nos respectivos campos das linhas (frete, seguro e outros custos), porém esse valor adicional ia para o total, o balanceamento nos itens do diário ficavam errados e consequentemente o valor devido também. Vou falar com o @DiegoParadeda pra avaliar qual solução vai ser melhor, valeu.