[14.0][FIX] l10n_br_account/l10n_br_fiscal: amount total,amount_price_gross when has tax withholding and IPI

[marcelsavegnago]

Correção/Ajustes dos calculos do valor total e do preço bruto preço bruto quando tem imposto retido. inclusive isso ja resolve o total de um pedido de compra com imposto retido subtrindo este o mesmo do montante.

Situação Atual:

Pedido de Compra:

Fatura (Operação Fiscal - NÃO Dedutível)

Linhas do Lançamento Contábil da Fatura (Operação Fiscal - NÃO Dedutível)

Fatura (Operação Fiscal - Imposto Dedutível)

Linhas do Lançamento Contábil da Fatura (Operação Fiscal - Imposto Dedutível)

Com os Ajustes desta PR:

Pedido de Compra:

Fatura (Operação Fiscal - NÃO Dedutível)

Linhas do Lançamento Contábil da Fatura (Operação Fiscal - NÃO Dedutível)

Fatura (Operação Fiscal - Imposto Dedutível)

Linhas do Lançamento Contábil da Fatura (Operação Fiscal - Imposto Dedutível)

[OCA-git-bot]

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

[marcelsavegnago]

PR verdinha com o runboat rodando agora. Por ora, deixei varios commits nesta PR para facilitar os testes mas podemos distribuir em outras PRs apesar da dependencia entre eles.

[marcelsavegnago]

@renatonlima, @rvalyi, @mbcosta, @mileo, @douglascstd, @antoniospneto, adicionei alguns prints na descrição do Pull Request para ilustrar melhor a questão. Inicialmente, notei que, no caso de operações fiscais não dedutíveis, o valor total a pagar da fatura estava incorreto. No entanto, decidi expandir a análise, observando que nem nos pedidos de compra nem nos de venda o imposto retido está sendo deduzido do total. Estou aberto a dúvidas, críticas ou sugestões que possam aprimorar nosso trabalho.

[marcelsavegnago]

Já movi alguns commits daqui para outras PRs:

#3019
#3020
#3021

[marcelsavegnago]

@rvalyi agora o teste é para ficar verde, mas veja que fix um commit no fiscal para não trazer definições de impostos quando a mesma estiver expirada. Acreditei que buscando apenas as aprovadas já atenderia mas pelo jeito não e isto causou erro em vários testes que não me aprofundei mas entendo que são casos que devem estar considerando definições de impostos ainda em racuscunho ou revisadas e não apenas as aprovadas.

2d2b7b2

cc @renatonlima

[marcelsavegnago]

Vou dar uma reorganiada no commit do test porque acabei mudando as coisas de lugar e vai dificultar a revisão.

[marcelsavegnago]

Vou dar uma reorganiada no commit do test porque acabei mudando as coisas de lugar e vai dificultar a revisão.

DONE

[rvalyi]

LGTM de um ponto de vista tecnico.

@renatonlima @douglascstd @felipemotter rola algum review funcional de vcs?

[marcelsavegnago]

@WesleyOliveira98 se tiver um tempo e puder fazer uma revisão funcional.. eu agradeço

[douglascstd]

[FUNCTIONAL REVIEW] APPROVED

Para o funcionamento dos impostos retidos era adotado o uso do imposto dedutível, onde apresentava corretamente os valores líquidos a pagar.
Com esta PR, não é mais necessário habilitar o dedutível para operar com impostos retidos.

Operação sem lançamento de imposto dedutível.

Alteração para aplicação do lançamento do imposto dedutível

Operação sem lançamento de imposto dedutível.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[rvalyi]

[FUNCTIONAL REVIEW] APPROVED

Para o funcionamento dos impostos retidos era adotado o uso do imposto dedutível, onde apresentava corretamente os valores líquidos a pagar.
Com esta PR, não é mais necessário habilitar o dedutível para operar com impostos retidos.

Operação sem lançamento de imposto dedutível.

Alteração para aplicação do lançamento do imposto dedutível

Operação sem lançamento de imposto dedutível.

show @douglascstd valeu pelos reviews funcionais sempre de qualidade!

[rvalyi]

/ocabot merge minor

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3016-by-rvalyi-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at d3d8d69. Thanks a lot for contributing to OCA. ❤️