[16.0][FIX] l10n_br_sale: Sobrescrever o carregamento de impostos para pedidos em empresas brasileiras

[antoniospneto]

Sobrescreve o método nativo responsável por carregar os impostos nas linhas do pedido. Essa alteração garante que, para empresas brasileiras, os impostos fiscais sejam carregados corretamente desde a criação do pedido, respeitando as particularidades fiscais do Brasil.

Extraido da PR #3532 [2]

[OCA-git-bot]

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

[marcelsavegnago]

LGTM
cc @kaynnan @WesleyOliveira98

[rvalyi]

apenas para dar uma perspectiva: como até a v10 os calculos fiscais eram embutidos no módulo l10n_br_account(_product), alguns metodos tipo compute tinham o mesmo nome do que no módulo account (propositalmente). Na v12 como a geração dos account.move(.line) era separada dos account.invoice(.line) tinha uma certa separação e menos risco de conflito. Depois do merge dos account.invoice(.line) no account.move(.line) na v13 de repente aparesceram mais possibilidades de conflitos.

Alem disso no proprio Odoo o nome dos métodos vai mudando de versão em versão e tem conflitos que surgiram por conta disso e ja tivemos que fazer mudanças do tipo na v16. Por examplo update_taxes -> update_fiscal_taxes.

[marcelsavegnago]

apenas para dar uma perspectiva: como até a v10 os calculos fiscais eram embutidos no módulo l10n_br_account(_product), alguns metodos tipo compute tinham o mesmo nome do que no módulo account (propositalmente). Na v12 como a geração dos account.move(.line) era separada dos account.invoice(.line) tinha uma certa separação e menos risco de conflito. Depois do merge dos account.invoice(.line) no account.move(.line) na v13 de repente aparesceram mais possibilidades de conflitos.

Alem disso no proprio Odoo o nome dos métodos vai mudando de versão em versão e tem conflitos que surgiram por conta disso e ja tivemos que fazer mudanças do tipo na v16. Por examplo update_taxes -> update_fiscal_taxes.

Visando evitar possiveis conflitos, o uso de algum prefixo faz sentido? e para campos ?

[rvalyi]

apenas para dar uma perspectiva: como até a v10 os calculos fiscais eram embutidos no módulo l10n_br_account(_product), alguns metodos tipo compute tinham o mesmo nome do que no módulo account (propositalmente). Na v12 como a geração dos account.move(.line) era separada dos account.invoice(.line) tinha uma certa separação e menos risco de conflito. Depois do merge dos account.invoice(.line) no account.move(.line) na v13 de repente aparesceram mais possibilidades de conflitos.
Alem disso no proprio Odoo o nome dos métodos vai mudando de versão em versão e tem conflitos que surgiram por conta disso e ja tivemos que fazer mudanças do tipo na v16. Por examplo update_taxes -> update_fiscal_taxes.

Visando evitar possiveis conflitos, o uso de algum prefixo faz sentido? e para campos ?

me equivoquei, esse meu comentário era para esse outro PR #3549 e confundi o PR no celular.

[rvalyi]

/ocabot merge patch

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-3550-by-rvalyi-bump-patch, awaiting test results.

[OCA-git-bot]

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