DANFE: Garantir que NFE com nota de referncia, tenham informações complementares

[vieira-paulo]

Em algumas NFE's que são de devolução por exemplo, não contém a tag infAdic, o que faz com que a caixa de texto de informações adicionais viesse vazia, porém é necessário que nesse campo venha as informações da nota de referência, dito isso, fiz uma mudança nessa parte do código, onde verifica a não existencia da tag infAdic e dessa forma pego o valor contido em NFref, como esse valor é a chave de acesso da nota de referencia, eu distrincho ele, pegando o numero da nota, data,modelo, série e CNP do emitente, ainda adiciono uma verificação para identificar se é um CNPJ ou CPF e tratar de forma dinâmica.
Fiz essa alteração no arquivo do meu sistema, e está funcionando perfeitamente, A mudança ajuda muito os setores contábeis a validarem as notas fiscais, Abaixo deixo o meu código:

def _is_valid_cpf(self, cpf):
        """Valida um CPF usando os dígitos verificadores."""
        cpf = re.sub(r"\\\\D", "", cpf)  #

        if len(cpf) != 11 or cpf in (c * 11 for c in "0123456789"):
            return False  

        
        soma = sum(int(cpf[i]) * (10 - i) for i in range(9))
        resto = soma % 11
        dv1 = 0 if resto < 2 else 11 - resto

        
        soma = sum(int(cpf[i]) * (11 - i) for i in range(10))
        resto = soma % 11
        dv2 = 0 if resto < 2 else 11 - resto

        return cpf[-2:] == f"{dv1}{dv2}"


    def _extract_cnp_emitente(self, ref_nfe):
        """Extrai e valida se o identificador do emitente é um CPF ou CNPJ."""
        cpf_candidato = ref_nfe[9:20]


        if self._is_valid_cpf(cpf_candidato):
            return cpf_candidato

        return ref_nfe[6:20]  


    def _get_additional_data_content(self):
        fisco = ""
        obs = ""

        if hasattr(self, 'inf_adic') and self.inf_adic:
            fisco = extract_text(self.inf_adic, "infAdFisco")
            obs = extract_text(self.inf_adic, "infCpl") or ""

        if not fisco and hasattr(self, 'nf_ref') and self.nf_ref:
            ref_nfe = extract_text(self.nf_ref, "refNFe")

            cnp_emitente = self._extract_cnp_emitente(ref_nfe)
            modelo = ref_nfe[20:22]
            serie = ref_nfe[22:25]
            numero = ref_nfe[25:34]
            data = ref_nfe[2:6]

            fisco = (
                f"Dados da nota de referência:\\\\n"
                f"Modelo: {modelo}\\\\nSérie: {serie}\\\\nNúmero: {numero}\\\\n"
                f"{'CPF' if len(cnp_emitente) == 11 else 'CNPJ'} do Emitente: {cnp_emitente}\\\\n"
                f"Data de Emissão: {data[2:4]}/20{data[0:2]}\\\\n"
                f"Chave de Acesso: {ref_nfe}"
            )

        if fisco:
            obs = f"{obs} {fisco}".strip()

        obs = " ".join(re.split(r"\\\\s+", obs.strip(), flags=re.UNICODE))
        return obs
        
        Ele substitui o trecho:
        
            def _get_additional_data_content(self):
        fisco = extract_text(self.inf_adic, "infAdFisco")
        obs = extract_text(self.inf_adic, "infCpl")
        dest_end, cpl, cpl_truncado = self._get_dest_end_text(self.dest)
        if cpl_truncado:
            obs += "Complemento do destinatário: " + cpl + "."
        if fisco:
            obs = f"{obs} {fisco}\n"
        obs = " ".join(re.split(r"\s+", obs.strip(), flags=re.UNICODE))
        return obs

E claro, é necessaŕio criar: self.nf_ref = root.find(f"{URL}NFref")

[CristianoMafraJunior]

Em algumas NFE's que são de devolução por exemplo, não contém a tag infAdic, o que faz com que a caixa de texto de informações adicionais viesse vazia, porém é necessário que nesse campo venha as informações da nota de referência, dito isso, fiz uma mudança nessa parte do código, onde verifica a não existencia da tag infAdic e dessa forma pego o valor contido em NFref, como esse valor é a chave de acesso da nota de referencia, eu distrincho ele, pegando o numero da nota, data,modelo, série e CNP do emitente, ainda adiciono uma verificação para identificar se é um CNPJ ou CPF e tratar de forma dinâmica. Fiz essa alteração no arquivo do meu sistema, e está funcionando perfeitamente, A mudança ajuda muito os setores contábeis a validarem as notas fiscais, Abaixo deixo o meu código:

def _is_valid_cpf(self, cpf): """Valida um CPF usando os dígitos verificadores.""" cpf = re.sub(r"\D", "", cpf) #

    if len(cpf) != 11 or cpf in (c * 11 for c in "0123456789"):
        return False  

    
    soma = sum(int(cpf[i]) * (10 - i) for i in range(9))
    resto = soma % 11
    dv1 = 0 if resto < 2 else 11 - resto

    
    soma = sum(int(cpf[i]) * (11 - i) for i in range(10))
    resto = soma % 11
    dv2 = 0 if resto < 2 else 11 - resto

    return cpf[-2:] == f"{dv1}{dv2}"


def _extract_cnp_emitente(self, ref_nfe):
    """Extrai e valida se o identificador do emitente é um CPF ou CNPJ."""
    cpf_candidato = ref_nfe[9:20]


    if self._is_valid_cpf(cpf_candidato):
        return cpf_candidato

    return ref_nfe[6:20]  


def _get_additional_data_content(self):
    fisco = ""
    obs = ""

    if hasattr(self, 'inf_adic') and self.inf_adic:
        fisco = extract_text(self.inf_adic, "infAdFisco")
        obs = extract_text(self.inf_adic, "infCpl") or ""

    if not fisco and hasattr(self, 'nf_ref') and self.nf_ref:
        ref_nfe = extract_text(self.nf_ref, "refNFe")

        cnp_emitente = self._extract_cnp_emitente(ref_nfe)
        modelo = ref_nfe[20:22]
        serie = ref_nfe[22:25]
        numero = ref_nfe[25:34]
        data = ref_nfe[2:6]

        fisco = (
            f"Dados da nota de referência:\\\\n"
            f"Modelo: {modelo}\\\\nSérie: {serie}\\\\nNúmero: {numero}\\\\n"
            f"{'CPF' if len(cnp_emitente) == 11 else 'CNPJ'} do Emitente: {cnp_emitente}\\\\n"
            f"Data de Emissão: {data[2:4]}/20{data[0:2]}\\\\n"
            f"Chave de Acesso: {ref_nfe}"
        )

    if fisco:
        obs = f"{obs} {fisco}".strip()

    obs = " ".join(re.split(r"\\\\s+", obs.strip(), flags=re.UNICODE))
    return obs
    
    Ele substitui o trecho:
    
        def _get_additional_data_content(self):
    fisco = extract_text(self.inf_adic, "infAdFisco")
    obs = extract_text(self.inf_adic, "infCpl")
    dest_end, cpl, cpl_truncado = self._get_dest_end_text(self.dest)
    if cpl_truncado:
        obs += "Complemento do destinatário: " + cpl + "."
    if fisco:
        obs = f"{obs} {fisco}\n"
    obs = " ".join(re.split(r"\s+", obs.strip(), flags=re.UNICODE))
    return obs

E claro, é necessaŕio criar: self.nf_ref = root.find(f"{URL}NFref")

@vieira-paulo Seja bem-vindo ao projeto, sobre essa alteração você consegue abrir uma PR (Pull Request) para analisar melhor a alteração?

[vieira-paulo]

@CristianoMafraJunior Boa tarde! Claro, posso sim, você libera a autorização de scrita para que eu possa fazer o Git Push e subir a Branch?

[CristianoMafraJunior]

@CristianoMafraJunior Boa tarde! Claro, posso sim, você libera a autorização de scrita para que eu possa fazer o Git Push e subir a Branch?

Não precisa liberar autorização, você deve fazer um Fork do repositório através desse fork criar uma branch nova e fazer o push e abrir a PR.

[antoniospneto]

Acho válida sua sugestão! É interessante exibir essas informações nas complementares, já que não existe um campo específico na DANFE para isso.

Porém, não vejo necessidade em validar o CPF ou CNPJ, pois partimos da premissa que essas informações já chegam corretas no XML emitido pela SEFAZ.

Além disso, talvez detalhar tanto assim não seja necessário. Uma abordagem mais simples poderia ser exibir apenas a chave da nota referenciada. Por exemplo:

NFe Ref: 35200159594315000157550010000000022062777169

Outro ponto importante seria adicionar uma configuração no sistema, permitindo ao usuário ativar ou desativar essa funcionalidade conforme a necessidade dele. Dessa forma, oferecemos flexibilidade sem comprometer a simplicidade do código.

O que acha?

[vieira-paulo]

A Validação do CPF e CNPJ é mais por questão visual, pois me incomoda. Se não faço essa validação um CPF irá vir com 2 zeros a sua esquerda.
Quanto a questão de detalhes também é por opção pessoal, mas concordo que seria bom permitir ativar ou desativar a funcionalidade, possivelmente até escolher quais campos o usuário quer que eseja exibido

[antoniospneto]

ah mas então é formatação, não validação, a biblioteca já tem uma função para isso veja:

BrazilFiscalReport/brazilfiscalreport/utils.py

Line 52 in 5ed1d7e

def format_cpf_cnpj(doc):

[vieira-paulo]

Verdade, deixei passar despercebido

[antoniospneto]

Fechando a discussão pois a melhoria está sendo discutida na PR #71