[14.0][FIX] l10n_br_sale: Mapear o campo ind_final no Pedido de Venda #3512
Conversation
|
Hi @renatonlima, |
|
O erro nos testes não parece ter relação com esse PR https://github.com/OCA/l10n-brazil/actions/runs/12089606467/job/33715182259?pr=3512#step:9:377 2024-11-29 19:11:52,331 535 CRITICAL odoo odoo.modules.module: Couldn't load module l10n_br_fiscal_certificate
2024-11-29 19:11:52,332 535 CRITICAL odoo odoo.modules.module: cannot import name 'verify' from 'OpenSSL.crypto' (unknown location)
File "/opt/odoo-venv/lib/python3.8/site-packages/signxml/__init__.py", line 7, in <module>
from .verifier import XMLVerifier, VerifyResult, SignatureConfiguration
File "/opt/odoo-venv/lib/python3.8/site-packages/signxml/verifier.py", line 13, in <module>
from OpenSSL.crypto import verify as openssl_verify
ImportError: cannot import name 'verify' from 'OpenSSL.crypto' (unknown location) |
tentei fazer um fix semelhante ao que fazemos na 15.0 e 16.0: #3513 |
|
@mbcosta deu certo, eu ja botei o fix na 14.0, se vc der um rebase em cima da 14.0 deve funcionar. |
mbcosta
force-pushed
the
14.0-FIX-sale_map_ind_final
branch
from
f170cb8
to
48a4939
Compare
|
valeu @rvalyi obrigado pela solução, agora parece estar tudo certo. Apenas para deixar claro para outros revisores que o teste desse PR é bem simples em um Novo Pedido de Venda ao selecionar um Cliente que tem a Operação de Consumo Final definida Não o campo no Pedido deve ser atualizado para esse valor. |
|
/ocabot merge patch |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 3739081. Thanks a lot for contributing to OCA. ❤️ |
No PR #2849 acabou passando um problema ao mapear o campo ind_final e foi aberto um issue sobre isso #3449
O erro acontece porque o método que faz esse mapeamento _onchange_partner_fiscal_id no módulo l10n_br_fiscal é chamando antes do método que preenche o campo partner_invoice_id onchange_partner_id no módulo sale e com isso o método que retorna o res.partner retorna False, para resolver eu apenas inclui o _onchange_partner_fiscal_id no sale.order para chamar antes o método que preenche o partner_invoice_id, deixei um comentário para deixar claro o problema e avaliar em futuras versões se é possível resolver com menos código.
cc @OCA/local-brazil-maintainers @DiegoParadeda