[14.0][RFC] l10n_br_nfe: refactor nfce qrcode methods #3542
Conversation
|
Hi @rvalyi, @renatonlima, |
|
@DiegoParadeda pode dar uma olhada nos testes? |
DiegoParadeda
force-pushed
the
fix/nfce-qrcode
branch
2 times, most recently
from
b27f38b to
3c7d88e
Compare
|
|
||
| copy_invoice.fiscal_document_id.processador_edoc = self.processador_edoc | ||
| copy_invoice.processador_edoc = self.processador_edoc | ||
|
|
There was a problem hiding this comment.
Porque pular linhas antes e depois? Não me parece que essas duas linhas seriam um bloco logico espcial, logo não acho bom pular linhas antes e depois. Eu perdi algo?
|
Pessoal esse PR, da forma que esta no momento resolve os problemas de transmissão. Mas os testes não estão passando, vamos rever isso com calma nas próximas semanas. Quem estiver com problemas em produção pode usar dessa forma por enquanto. |
|
@mileo @DiegoParadeda por se tratar de código compartilhado entre NFe, NFCe, CTe e outros edoc, eh bastante importante acertar esse PR... |
|
O erro agora provavelmente esta relacionado a este PR: https://github.com/OCA/l10n-brazil/pull/3521/files |
DiegoParadeda
force-pushed
the
fix/nfce-qrcode
branch
2 times, most recently
from
6525334 to
7bced94
Compare
| processor = record.processador_edoc | ||
|
|
||
| # TODO: Avaliar a possibilidade de remover esse flush e invalidate | ||
| record.flush() | ||
| record.invalidate_cache() | ||
| inf_nfe = record._build_binding("nfe", "40") | ||
|
|
||
| if hasattr(record, "move_ids") and record.move_ids: | ||
| record.move_ids.processador_edoc = processor | ||
| record.processador_edoc = processor | ||
|
|
There was a problem hiding this comment.
poderia explicar um pouco qual a necessidade desse código?
DiegoParadeda
force-pushed
the
fix/nfce-qrcode
branch
3 times, most recently
from
dd2d0da to
2fb1c1a
Compare
RFC #3522 + fix duplicate inf.supl
Esse PR corrige um bug atual na geração na NFCe que resulta em falha de schema de XML por falta de inf.supl