[16.0][REF] l10n_br_account, fiscal: Aperfeiçoamento nos lançamentos contábeis #3558
Conversation
|
Hi @renatonlima, @rvalyi, |
antoniospneto
requested review from
rvalyi,
marcelsavegnago and
felipemotter
and removed request for
rvalyi and
marcelsavegnago
| "amount_currency": 0.0, | ||
| "debit": 0.0, | ||
| "amount_currency": 206.5, | ||
| "debit": 206.5, |
There was a problem hiding this comment.
@antoniospneto @marcelsavegnago na v14 tava 0 para esses 2 valores. Tava errado na v14.0 mesmo?
https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account/tests/test_account_move_lc.py#L692
There was a problem hiding this comment.
Tava errado mesmo na v14, até tava conversando com o @marcelsavegnago sobre isso hoje, essa é uma nota fiscal de remessa com lançamento de imposto, ela tem cobrança de imposto , mas ao mesmo tempo não pode gerar financeiro, não pode ter contas a receber ou apagar no caso, e aí agora eu zerei o financeiro, mas de alguma forma ainda precisamos balancear os lançamentos, por isso joguei a diferença pra conta da receita/despesa, dessa forma o lançamento fica balanceado sem precisar gerar o financeiro, que é o correto, no caso uma nota fiscal de remessa com imposto a contra partida para os impostos é uma despesa para a empresa.
There was a problem hiding this comment.
OK, vc jogou a contra-partida dos impostos aqui e tirou das contas a receber...
There was a problem hiding this comment.
so para deixar claro: na 14, se as taxas são dedutíveis, os lançamentos de remessas estão OK. Mas se as taxas não são dedutíveis, os lançamento estão errados como neste caso (sera bom a gente botar algum comentário sobre isso na v14). E ai o Antonio solucionou esse caso tb aqui neste PR para a v16 (isso tava OK na v12 tb mas tinha regredido na v14)
| # contrapartida dos impostos para manter o balanço contábil | ||
| # equilibrado. Na versão 14 do Odoo, essa diferença era | ||
| # automaticamente alocada às contas associadas aos termos | ||
| # de pagamento. |
There was a problem hiding this comment.
@antoniospneto vc poderia apenas passar este comentário para Inglês? (nos modulos mais centrais é bom botar tudo em inglês). O restante me parece bem OK...
marcelsavegnago
requested review from
douglascstd,
WesleyOliveira98 and
mileo
|
This PR has the |
|
/ocabot merge minor |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at c0c83ff. Thanks a lot for contributing to OCA. ❤️ |
Durante a migração do módulo
l10n_br_sale_stock, identifiquei falhas na contabilização de faturas em algumas situações específicas. Esta PR tem como objetivo aprimorar os lançamentos contábeis, garantindo maior aderência aos casos específicos da legislação brasileira.Além disso, foi necessário ajustar alguns testes unitários, pois estavam apresentando falsos positivos. Por exemplo, anteriormente, contas a receber/pagar estavam sendo geradas para notas fiscais de remessa, o que é incorreto, já que essas notas não devem impactar o financeiro. Agora, o comportamento está alinhado ao esperado, garantindo maior precisão no processamento.
Essa PR é a terceira extração de #3532