Serialização do grupo de pagamentos #318
Conversation
ADD: `xml_writer.py` - responsavel por gerar as TAG's do XML ADD: Adicionado lista de `pagamentos` à classe NotaFiscal, removido campo `tipo_pagamento` ADD: Serialização do grupo de pagamentos TEST: `test_nfce_serializacao.py` - adicionado validação do total pago. (vPag-vTroco = vNF) issue: TadaSoftware#317
REFACTORY: remove unused import
REFACTORY: python 3.8 e 3.9 compatibility, remove unsupported operator
There was a problem hiding this comment.
Muito boa adição! Eu estava enrolando a um tempo pra fazer isso xD.
Só uma sugestão importante, que creio que vai facilitar a PR ser aprovada e também facilitar a todos que desejam manter o PyNFE atualizados (Como eu, que quero essa atualização mas se eu subir agora vou ter um downtime até implementar o adicionar_pagamento, já que não posso implementar antes de atualizar pela função não existir)
| pag = etree.SubElement(raiz, "pag") | ||
| detpag = etree.SubElement(pag, "detPag") | ||
| if ( | ||
| str(nota_fiscal.finalidade_emissao) == "3" | ||
| or str(nota_fiscal.finalidade_emissao) == "4" | ||
| ): | ||
| etree.SubElement(detpag, "tPag").text = "90" | ||
| etree.SubElement(detpag, "vPag").text = "{:.2f}".format(0) | ||
| else: | ||
| etree.SubElement(detpag, "tPag").text = str( | ||
| nota_fiscal.tipo_pagamento | ||
| ).zfill(2) | ||
| etree.SubElement(detpag, "vPag").text = "{:.2f}".format( | ||
| nota_fiscal.totais_icms_total_nota | ||
| ) | ||
| if nota_fiscal.tipo_pagamento == 3 or nota_fiscal.tipo_pagamento == 4: | ||
| cartao = etree.SubElement(detpag, "card") | ||
| """ Tipo de Integração do processo de pagamento com | ||
| o sistema de automação da empresa: | ||
| 1=Pagamento integrado com o sistema de automação da empresa | ||
| 2= Pagamento não integrado com o sistema de automação da empresa | ||
| """ | ||
| etree.SubElement(cartao, "tpIntegra").text = "2" | ||
| # etree.SubElement(cartao, 'CNPJ').text = '' | ||
| # # Informar o CNPJ da Credenciadora de cartão de crédito / débito | ||
| # etree.SubElement(cartao, 'tBand').text = '' | ||
| # # 01=Visa 02=Mastercard 03=American Express 04=Sorocred | ||
| # 05=Diners Club 06=Elo 07=Hipercard 08=Aura 09=Caba 99=Outros | ||
| # etree.SubElement(cartao, 'cAut').text = '' | ||
| # # Identifica o número da autorização da transação da operação | ||
| # com cartão de crédito e/ou débito | ||
| # troco | ||
| # etree.SubElement(pag, 'vTroco').text = str('') |
There was a problem hiding this comment.
Teria como fazer essa alteração não ser uma breaking change? Como verificar se existe o tipo_pagamento preenchido, garantindo que todos que utilizam atualmente a PyNFE possam migrar gradativamente.
De preferencia, pode também botar um warning no console avisando que usar o tipo_pagamento ficará depreciado e que usaremos somente o adicionar_pagamento.
Dessa forma também nos testes devemos ter por enquanto uma validação que garanta que a forma antiga e nova funcionam.
Serialização do grupo de pagamentos (issue: #317)
ADD: métodos de formatação de Decimais
ADD:
xml_writer.py- responsável por gerar as TAG's do XMLADD: Adicionado lista de
pagamentosà classeNotaFiscal, removido campotipo_pagamentoADD: Serialização do grupo de pagamentos
TEST:
test_nfce_serializacao.py- adicionado validação do total pago. (vPag-vTroco = vNF)