[14.0][REF] l10n_br_stock_account: Adaptando o código para os casos Sem Operação Fiscal ou Multi-Localizações - Githubissues

OCA / l10n-brazil

Localização brasileira oficial do Odoo.

https://odoo-community.org/psc-teams/brazil-66

GNU Affero General Public License v3.0

235 stars 244 forks source link

[14.0][REF] l10n_br_stock_account: Adaptando o código para os casos Sem Operação Fiscal ou Multi-Localizações #3144

Closed mbcosta closed 2 months ago

mbcosta commented 2 months ago

Adaptando o código para os casos Sem Operação Fiscal ou Multi-Localizações, PR simples basicamente:

chamando onchange_product_id_fiscal da stock.move apenas quando o stock.picking tiver a Operação Fiscal
O método default da Operação Fiscal do stock.picking retornar False quando o Pedido de Vendas ou Compras não tiver Operação Fiscal para evitar que na criação da Fatura a partir do stock.picking acabe considerando ser um caso "Com Operação Fiscal". Isso está sendo feito no l10n_br_stock_account e não no l10n_br_purchase_stock ou l10n_br_sale_stock porque até onde entendi esses métodos default para serem chamados em outros módulos precisam sobre escrever o campo e parece que deixa de chamar o método original mesmo com o super e também acabaria duplicando código, talvez exista uma forma, por enquanto o attrs parece atender
Nos testes removi a chamada dos onhanges _onchange_fiscal_operation_id e _onchange_fiscal_operation_line_id por já serem chamados pelo _onchange_product_id_fiscal

cc @rvalyi @renatonlima @marcelsavegnago @mileo

rvalyi commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-3144-by-rvalyi-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3144-by-rvalyi-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

rvalyi commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

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

OCA-git-bot commented 2 months ago

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3144-by-rvalyi-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

rvalyi commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

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

OCA-git-bot commented 2 months ago

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