correções e melhorias diversas #315
Conversation
FIX: Corrigido problema de exception com mensagem vazia quando o campo `cliente.inscricao_estadual`esta nulo e o `indidacador_ie` != 2 ou `not client.isento_icms`, tornando dificil a identificação doi problema. FIX: NFC-e deve permitir informar apenas o CPF e o Nome, no entanto não estava sendo gerado o nome do destinatario caso informado. FIX: removido validação incorreta onde não permitia PIS/COFINS para NFC-e ADDED: adicionado tag EXTIPI na serialização do XML REFACTORY: removido prop `pis_situacao_tributaria` não utilizada no objeto `NotaFiscalProduto` e adicionado `pis_modalidade` que é utilizado pela serialização, esta incongruencia gerava confusão na hora de informar os dados do pis.
REFACTORY: resolvido `TODO calcular impostos aproximados`, o sistema passará a totalizar os totais dos tributos informados para o produto. REFACTORY: alterado `totais_tributos_aproximado`para readonly, este valor não deve ser informado para não acumular duas vezes os tributos. TEST: Removido `totais_tributos_aproximado` para ficar de acordo com as alterações da NotaFiscal.
betogd
changed the title
| @@ -158,8 +158,7 @@ def _serializar_cliente( | |||
| cliente.numero_documento | |||
| ) | |||
| if not self._so_cpf: | |||
| if cliente.razao_social: | |||
| etree.SubElement(raiz, "xNome").text = cliente.razao_social | |||
| etree.SubElement(raiz, "xNome").text = cliente.razao_social | |||
There was a problem hiding this comment.
Isso aqui está errado, deve permitir preencher somente cpf sem nome.
| @@ -401,8 +405,8 @@ def _serializar_produto_servico( | |||
| # Lei da transparencia | |||
| # Tributos aprox por item | |||
| if produto_servico.valor_tributos_aprox: | |||
| etree.SubElement(imposto, "vTotTrib").text = str( | |||
| produto_servico.valor_tributos_aprox | |||
| etree.SubElement(imposto, "vTotTrib").text = "{:.2f}".format( | |||
There was a problem hiding this comment.
Valor total de tributos deve ser informado pelo emissor.
There was a problem hiding this comment.
ˆ Mas está certo, a unica coisa que ele fez foi colocar a formatação padrão utilizada em outros valores decimais.
|
Existe PIS e COFINS para NFC-e? |
| __totais_tributos_aproximado = Decimal() | ||
| @property | ||
| def totais_tributos_aproximado(self): | ||
| return self.__totais_tributos_aproximado |
There was a problem hiding this comment.
Não entendi, por que foi feito isso?
| # TODO calcular impostos aproximados | ||
| # self.totais_tributos_aproximado += obj.tributos |
There was a problem hiding this comment.
Não é a mesma coisa que antes?
| @@ -813,7 +815,7 @@ class NotaFiscalProduto(Entidade): | |||
| # - PIS | |||
| # - PIS | |||
| # - Situacao tributaria (obrigatorio - seleciona de lista) - PIS_TIPOS_TRIBUTACAO | |||
| pis_situacao_tributaria = str() | |||
There was a problem hiding this comment.
Muito bem observado, isso aqui estava com o nome errado kkkk
| @@ -158,8 +158,7 @@ def _serializar_cliente( | |||
| cliente.numero_documento | |||
| ) | |||
| if not self._so_cpf: | |||
| if cliente.razao_social: | |||
| etree.SubElement(raiz, "xNome").text = cliente.razao_social | |||
| etree.SubElement(raiz, "xNome").text = cliente.razao_social | |||
| @@ -190,7 +192,7 @@ def _serializar_cliente( | |||
| etree.SubElement(raiz, "indIEDest").text = "9" | |||
| elif ( | |||
| cliente.indicador_ie == 2 or cliente.isento_icms | |||
| ) or cliente.inscricao_estadual.upper() == "ISENTO": | |||
| ) or (cliente.inscricao_estadual or '').upper() == "ISENTO": | |||
There was a problem hiding this comment.
Acho que o certo é colocar um Throw se tentar colocar o inscricao_estadual como None OU no init da classe preencher ele como "" caso for None.
| @@ -401,8 +405,8 @@ def _serializar_produto_servico( | |||
| # Lei da transparencia | |||
| # Tributos aprox por item | |||
| if produto_servico.valor_tributos_aprox: | |||
| etree.SubElement(imposto, "vTotTrib").text = str( | |||
| produto_servico.valor_tributos_aprox | |||
| etree.SubElement(imposto, "vTotTrib").text = "{:.2f}".format( | |||
There was a problem hiding this comment.
ˆ Mas está certo, a unica coisa que ele fez foi colocar a formatação padrão utilizada em outros valores decimais.
Sim, são opcionais mas existem. |
|
As alterações solicitadas não foram respondidas. Estou fechando este PR por enquanto. |
existe a criterio da UF. |
FIX: Corrigido problema de exception com mensagem vazia quando o campo
cliente.inscricao_estadualesta nulo e oindidacador_ie!= 2 ounot client.isento_icms, tornando dificil a identificação doi problema.FIX: NFC-e deve permitir informar apenas o CPF e o Nome, no entanto não estava sendo gerado o nome do destinatario caso informado.
FIX: removido validação incorreta onde não permitia PIS/COFINS para NFC-e
ADDED: adicionado tag EXTIPI na serialização do XML
REFACTORY: removido prop
pis_situacao_tributarianão utilizada no objetoNotaFiscalProdutoe adicionadopis_modalidadeque é utilizado pela serialização, esta incongruencia gerava confusão na hora de informar os dados do pis.