From 49113d7dab5e4d5852b06a879eada1259c171eb7 Mon Sep 17 00:00:00 2001
From: Felipe Prenholato <felipeprenholato@gmail.com>
Date: Thu, 26 Apr 2018 14:57:30 -0300
Subject: [PATCH 1/3] Better templates, for real.

---
 .../nfse/paulistana/templates/EnvioLoteRPS.xml     | 29 ++++++--
 pytrustnfe/nfse/paulistana/templates/EnvioRPS.xml  | 80 +++++++++++++---------
 2 files changed, 71 insertions(+), 38 deletions(-)

diff --git a/pytrustnfe/nfse/paulistana/templates/EnvioLoteRPS.xml b/pytrustnfe/nfse/paulistana/templates/EnvioLoteRPS.xml
index 8e9f34d..e7003de 100644
--- a/pytrustnfe/nfse/paulistana/templates/EnvioLoteRPS.xml
+++ b/pytrustnfe/nfse/paulistana/templates/EnvioLoteRPS.xml
@@ -32,28 +32,43 @@
     <CodigoServico>{{ rps.codigo_atividade }}</CodigoServico>
     <AliquotaServicos>{{ rps.aliquota_atividade }}</AliquotaServicos>
     <ISSRetido>{{ rps.iss_retido | default('false') }}</ISSRetido>
+    {% if rps.tomador.tipo_cpfcnpj != 3 %}
     <CPFCNPJTomador>
-        {% if rps.tomador.tipo_cpfcnpj == 1 -%}
-           <CPF>{{ rps.tomador.cpf_cnpj }}</CPF>
-        {% endif %}
-        {% if rps.tomador.tipo_cpfcnpj == 2 -%}
-           <CNPJ>{{ rps.tomador.cpf_cnpj }}</CNPJ>
-        {% endif %}
+      {% if rps.tomador.tipo_cpfcnpj == 1 -%}
+      <CPF>{{ rps.tomador.cpf_cnpj }}</CPF>
+      {% endif %}
+      {% if rps.tomador.tipo_cpfcnpj == 2 -%}
+      <CNPJ>{{ rps.tomador.cpf_cnpj }}</CNPJ>
+      {% endif %}
     </CPFCNPJTomador>
+    {% endif %}
     {% if rps.tomador.inscricao_municipal -%}
     <InscricaoMunicipalTomador>{{ rps.tomador.inscricao_municipal }}</InscricaoMunicipalTomador>
     {% endif %}
     <RazaoSocialTomador>{{ rps.tomador.razao_social|normalize|escape }}</RazaoSocialTomador>
+    {% if rps.tomador.cidade %}
     <EnderecoTomador>
+      {% if rps.tomador.tipo_logradouro %}
       <TipoLogradouro>{{ rps.tomador.tipo_logradouro }}</TipoLogradouro>
+      {% endif %}
+      {% if rps.tomador.logradouro %}
       <Logradouro>{{ rps.tomador.logradouro|normalize|escape }}</Logradouro>
+      {% endif %}
+      {% if rps.tomador.numero %}
       <NumeroEndereco>{{ rps.tomador.numero }}</NumeroEndereco>
+      {% endif %}
+      {% if rps.tomador.complemento %}
       <ComplementoEndereco>{{ rps.tomador.complemento|normalize|escape }}</ComplementoEndereco>
+      {% endif %}
+      {% if rps.tomador.bairro %}
       <Bairro>{{ rps.tomador.bairro }}</Bairro>
+      {% endif %}
       <Cidade>{{ rps.tomador.cidade }}</Cidade>
       <UF>{{ rps.tomador.uf }}</UF>
-      <CEP>{{ rps.tomador.cep }}</CEP>
+      {% if rps.tomador.cep %}<CEP>{{ rps.tomador.cep }}</CEP>{% endif %}
     </EnderecoTomador>
+    {% endif %}
+    {% if rps.tomador.email %}<EmailTomador>{{ rps.tomador.email }}</EmailTomador>{% endif %}
     <Discriminacao>{{ rps.descricao|normalize|escape }}</Discriminacao>
   </RPS>
   {% endfor %}
diff --git a/pytrustnfe/nfse/paulistana/templates/EnvioRPS.xml b/pytrustnfe/nfse/paulistana/templates/EnvioRPS.xml
index d828534..35e08cb 100644
--- a/pytrustnfe/nfse/paulistana/templates/EnvioRPS.xml
+++ b/pytrustnfe/nfse/paulistana/templates/EnvioRPS.xml
@@ -6,46 +6,64 @@
   </Cabecalho>
   {% for rps in nfse.lista_rps -%}
   <RPS xmlns="">
-      <Assinatura>{{ rps.assinatura }}</Assinatura>
+    <Assinatura>{{ rps.assinatura }}</Assinatura>
     <ChaveRPS>
-        <InscricaoPrestador>{{ rps.prestador.inscricao_municipal }}</InscricaoPrestador>
-        <SerieRPS>{{ rps.serie }}</SerieRPS>
-        <NumeroRPS>{{ rps.numero }}</NumeroRPS>
+      <InscricaoPrestador>{{ rps.prestador.inscricao_municipal }}</InscricaoPrestador>
+      <SerieRPS>{{ rps.serie }}</SerieRPS>
+      <NumeroRPS>{{ rps.numero }}</NumeroRPS>
     </ChaveRPS>
-    <TipoRPS>RPS-M</TipoRPS>
+    <TipoRPS>{{ rps.tipo_rps | default('RPS') }}</TipoRPS>
     <DataEmissao>{{ rps.data_emissao }}</DataEmissao>
     <StatusRPS>N</StatusRPS>
-    <TributacaoRPS>T</TributacaoRPS>
-    <ValorServicos>{{ nfse.total_servicos }}</ValorServicos>
-    <ValorDeducoes>{{ nfse.total_deducoes }}</ValorDeducoes>
-    <ValorPIS>{{ rps.valor_pis }}</ValorPIS>
-    <ValorCOFINS>{{ rps.valor_cofins }}</ValorCOFINS>
-    <ValorINSS>{{ rps.valor_inss }}</ValorINSS>
-    <ValorIR>{{ rps.valor_pis }}</ValorIR>
-    <ValorCSLL>{{ rps.valor_csll }}</ValorCSLL>
+    <TributacaoRPS>{{ rps.tributacao_rps | default('T') }}</TributacaoRPS>
+    <ValorServicos>{{ rps.valor_servico }}</ValorServicos>
+    <ValorDeducoes>{{ rps.valor_deducao }}</ValorDeducoes>
+    <ValorPIS>{{ rps.valor_pis | default('0.00') }}</ValorPIS>
+    <ValorCOFINS>{{ rps.valor_cofins | default('0.00') }}</ValorCOFINS>
+    <ValorINSS>{{ rps.valor_inss | default('0.00') }}</ValorINSS>
+    <ValorIR>{{ rps.valor_ir | default('0.00') }}</ValorIR>
+    <ValorCSLL>{{ rps.valor_csll | default('0.00') }}</ValorCSLL>
     <CodigoServico>{{ rps.codigo_atividade }}</CodigoServico>
     <AliquotaServicos>{{ rps.aliquota_atividade }}</AliquotaServicos>
-    <ISSRetido>false</ISSRetido>
+    <ISSRetido>{{ rps.iss_retido | default('false') }}</ISSRetido>
+    {% if rps.tomador.tipo_cpfcnpj != 3 %}
     <CPFCNPJTomador>
-        {% if rps.tomador.tipo_cpfcnpj == 1 -%}
-        <CPF>{{ rps.tomador.cpf_cnpj }}</CPF>
-        {% endif %}
-        {% if rps.tomador.tipo_cpfcnpj == 2 -%}
-        <CNPJ>{{ rps.tomador.cpf_cnpj }}</CNPJ>
-        {% endif %}
+      {% if rps.tomador.tipo_cpfcnpj == 1 -%}
+      <CPF>{{ rps.tomador.cpf_cnpj }}</CPF>
+      {% endif %}
+      {% if rps.tomador.tipo_cpfcnpj == 2 -%}
+      <CNPJ>{{ rps.tomador.cpf_cnpj }}</CNPJ>
+      {% endif %}
     </CPFCNPJTomador>
-    <RazaoSocialTomador>{{ rps.tomador.razao_social }}</RazaoSocialTomador>
+    {% endif %}
+    {% if rps.tomador.inscricao_municipal -%}
+    <InscricaoMunicipalTomador>{{ rps.tomador.inscricao_municipal }}</InscricaoMunicipalTomador>
+    {% endif %}
+    <RazaoSocialTomador>{{ rps.tomador.razao_social|normalize|escape }}</RazaoSocialTomador>
+    {% if rps.tomador.cidade %}
     <EnderecoTomador>
-        <TipoLogradouro>{{ rps.tomador.tipo_logradouro }}</TipoLogradouro>
-        <Logradouro>{{ rps.tomador.logradouro }}</Logradouro>
-        <NumeroEndereco>{{ rps.tomador.numero }}</NumeroEndereco>
-        <ComplementoEndereco>{{ rps.tomador.complemento }}</ComplementoEndereco>
-        <Bairro>{{ rps.tomador.bairro }}</Bairro>
-        <Cidade>{{ rps.tomador.cidade }}</Cidade>
-        <UF>{{ rps.tomador.uf }}</UF>
-        <CEP>{{ rps.tomador.cep }}</CEP>
+      {% if rps.tomador.tipo_logradouro %}
+      <TipoLogradouro>{{ rps.tomador.tipo_logradouro }}</TipoLogradouro>
+      {% endif %}
+      {% if rps.tomador.logradouro %}
+      <Logradouro>{{ rps.tomador.logradouro|normalize|escape }}</Logradouro>
+      {% endif %}
+      {% if rps.tomador.numero %}
+      <NumeroEndereco>{{ rps.tomador.numero }}</NumeroEndereco>
+      {% endif %}
+      {% if rps.tomador.complemento %}
+      <ComplementoEndereco>{{ rps.tomador.complemento|normalize|escape }}</ComplementoEndereco>
+      {% endif %}
+      {% if rps.tomador.bairro %}
+      <Bairro>{{ rps.tomador.bairro }}</Bairro>
+      {% endif %}
+      <Cidade>{{ rps.tomador.cidade }}</Cidade>
+      <UF>{{ rps.tomador.uf }}</UF>
+      {% if rps.tomador.cep %}<CEP>{{ rps.tomador.cep }}</CEP>{% endif %}
     </EnderecoTomador>
-    <Discriminacao>{{ rps.descricao }}</Discriminacao>
-  {% endfor %}
+    {% endif %}
+    {% if rps.tomador.email %}<EmailTomador>{{ rps.tomador.email }}</EmailTomador>{% endif %}
+    <Discriminacao>{{ rps.descricao|normalize|escape }}</Discriminacao>
   </RPS>
+  {% endfor %}
 </PedidoEnvioRPS>

From d616b3b9a6f0e7fe3f061bbdd938b9cb5cc0ef3a Mon Sep 17 00:00:00 2001
From: Felipe Prenholato <felipeprenholato@gmail.com>
Date: Tue, 15 May 2018 14:05:29 -0300
Subject: [PATCH 2/3] Fix case name, bump version to 0.9.18.post1

---
 setup.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/setup.py b/setup.py
index 4f68dcf..ea9d46b 100644
--- a/setup.py
+++ b/setup.py
@@ -2,11 +2,11 @@
 from setuptools import setup, find_packages
 
 
-VERSION = "0.9.18"
+VERSION = "0.9.18.post1"
 
 
 setup(
-    name="PyTrustNFe3",
+    name="pytrustnfe3",
     version=VERSION,
     author="Danimar Ribeiro",
     author_email='danimaribeiro@gmail.com',

From 7fa4f4602f85ede83d987110313e22ae3b4264c8 Mon Sep 17 00:00:00 2001
From: Felipe Prenholato <felipeprenholato@gmail.com>
Date: Wed, 23 May 2018 14:51:35 -0300
Subject: [PATCH 3/3] Add missing deps, bump version to 0.9.18.post2

---
 setup.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/setup.py b/setup.py
index ea9d46b..4ded017 100644
--- a/setup.py
+++ b/setup.py
@@ -2,7 +2,7 @@
 from setuptools import setup, find_packages
 
 
-VERSION = "0.9.18.post1"
+VERSION = "0.9.18.post2"
 
 
 setup(
@@ -44,6 +44,8 @@ later (LGPLv2+)',
     description='PyTrustNFe é uma biblioteca para envio de NF-e',
     long_description=open('README.md', 'r').read(),
     install_requires=[
+        'urllib3',
+        'xmlsec==1.3.3',  # apt update;apt install libxmlsec1-dev pkg-config -y
         'Jinja2 >= 2.8',
         'pyOpenSSL >= 16.0.0, < 18',
         'signxml >= 2.4.0',