[14.0][FIX] l10n_br_account: price_total and price_subtotal values #3037
Conversation
|
Hi @rvalyi, @renatonlima, |
marcelsavegnago
force-pushed
the
14.0-l10n_br_account-fix-price_total-and-price_subtotal-account_move_line
branch
3 times, most recently
from
66ff0b2
to
97de7e7
Compare
marcelsavegnago
force-pushed
the
14.0-l10n_br_account-fix-price_total-and-price_subtotal-account_move_line
branch
8 times, most recently
from
470b4c4
to
cb7a534
Compare
marcelsavegnago
requested review from
rvalyi,
renatonlima,
felipemotter,
douglascstd and
antoniospneto
and removed request for
renatonlima
|
Pessoal, esta PR resolve os calculos do price_total e price_subtotal. Sugiro testar funcionalmente tanto criando diretamente um account.move ou através de pedidos de venda, compra e picking. |
|
@rvalyi foi ótimo a inclusão dos testes que vc adicionou um tempo atrás no l10n_br_account. Isso tem ajudado muito a identificar os impactos de qq mudança no módulo. Parabens. |
marcelsavegnago
changed the title
|
@marcelsavegnago consegue explicar melhor como chegar no erro? isso seria, como reportado na issue, quando altera o ipi de uma linha? |
|
@felipemotter a ajuda de vc nessa revisão vai ser bem util... |
|
O primeiro problema que identifiquei foi o valor do campo price_total dos invoie_line_ids. Isto ocorre principalmente quando tem IPI, Redução e Retenção. (Percebi comportamento diferente durante a criação e apenas na escrita) O segundo problema que identifiquei foi que as demais linhas do lançamento contábil não estavam sendo preenchidos os campos price_unit, price_subtotal e price_total. Tem alguns prints aqui E sim, sobre o valor do price_total do item da fatura, sim um exemplo é o da issue #3035 Se precisar de mais informações me diga. |
|
olhei rapidamente. Na parte tecnica não vi nada de chocante até agora. Agora é bom analisar fundo a questão funcional... |
|
Gente, prometo que amanha pego pra testar, se é o que estou pensando, ja havíamos notado esse problema, mas nunca fui a fundo do que estava gerando. |
There was a problem hiding this comment.
[FUNCTIONAL REVIEW]
Verificação da alteração de IPI na linha da fatura de compras - OK conforme esperado
https://github.com/OCA/l10n-brazil/assets/20867090/1ef63f20-0a15-4d68-a450-19cedb7584d7
Não sei se teve interferência desta PR, mas ao salvar a fatura, os impostos selecionados na linha da fatura são apagados, ficando os registros vazios:
https://github.com/OCA/l10n-brazil/assets/20867090/2559f8b9-4244-4e3a-a2b6-6c6cc9e0ea59
Esse foi o teste no Runboat da OCA: O comportamento de apagar os registros de impostos na linha não é verificado:
https://github.com/OCA/l10n-brazil/assets/20867090/e082dc08-ebdb-4152-bc1c-49064ba99a85
Outra característica é:
Se Salvar a fatura antes de Postar, os lançamentos de impostos permanecem no lançamento de diário
Mas se Postar (sem salvar antes), o lançamento de diário não aparecem impostos
Nas duas situações, os campos de impostos ficam vazios na linha da fatura.
Verificado campos preenchidos -
OCA - l10n_br
Neste PR:
Este problema reportado pelo douglas está sendo tratado na PR 3076... mesclando ela eu faço o rebase aqui.. se puder priorizar a 3076 agora eu agradeço. |
… and price_subtotal values
marcelsavegnago
force-pushed
the
14.0-l10n_br_account-fix-price_total-and-price_subtotal-account_move_line
branch
from
cb7a534
to
88cfea6
Compare
|
@douglascstd o problema que estava acontecendo foi resolvido na PR #3077. Agradeço se puder revisar novamente. |
|
Pessoal, comecei a revisar um pouco, todo cuidado é pouco... Uma coisa positiva que eu vi é que os valores price_unit, price_subtotal e price_total dos tax_vals* e term_vals* nos testes ficaram com os valores do debito ou do credito da linha, assim como esta nos testes originais no modulo account: Nisso eu estou OK com o diff na pasta l10n_br_account/tests e novamente nada me chocou nas outras partes apesar que eu preciso revisar mais fundo. Eu pretendo revisar as outras coisas em breve, mas é bom que as pessoas competentes revisam mesmo... |
|
/ocabot merge minor |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at ec52898. Thanks a lot for contributing to OCA. ❤️ |
marcelsavegnago
deleted the
14.0-l10n_br_account-fix-price_total-and-price_subtotal-account_move_line
branch
Esta PR trata a questão dos campos price_unit, price_total e price_subtotal das linhas do movimento contábil. Aqui issue apresentando o caso #3035