8000 Metodo zClean · Issue #869 · nfephp-org/nfephp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Metodo zClean #869

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
weguri opened this issue Sep 2, 2016 · 4 comments
Closed

Metodo zClean #869

weguri opened this issue Sep 2, 2016 · 4 comments
Labels
em progresso falha/bug

Comments

@weguri
Copy link
weguri commented Sep 2, 2016

O método zClean da classe ConvertNFe retorna a mesma variável de entrada.

protected function zClean($aCampos = array())
{
    foreach ($aCampos as $campo) {
        $campo = trim(preg_replace('/\s+/', ' ', $campo));
        if ($this->limparString) {
            $campo = Strings::cleanString($campo);
        }
    }
    return $aCampos;
}

Não deveria retornar um novo array?
@robmachado
Copy link
Contributor

Sim e é isso que ele retorna o ARRAY $aCampos
Não entendi sua colocação ?!?!

@weguri
Copy link
Author
weguri commented Dec 12, 2016

O meu exemplo de uso: var_dump(zClean([' teste ','ação','é agora ção ']));
a saída ficou igual a entrada.

Fiz sobrescrita no método para resolver o meu problema, da forma que está abaixo os caracteres de especiais e os espaços extras foram limpados:

protected function zClean($aCampos = array()) 
{
   $arrAuxiliar=[];
    foreach ($aCampos as $campo) {
        $campo = trim(preg_replace('/\s+/', ' ', $campo));
        if ($this->limparString) {
            $arrAuxiliar[] = Strings::cleanString($campo);
        }
    }
    return $arrAuxiliar;
}

@robmachado
Copy link
Contributor

OK .. já chequei o problema. Realmente é um BUG vou corrigir mas de outra forma. Pois no seu caso ainda tem um erro quando a propriedade $this->limparString for FALSE essa sua função vai retornar um array vazio.

@weguri
Copy link
Author
weguri commented Dec 12, 2016

Muito obrigado...

robmachado added a commit that referenced this issue Dec 12, 2016
@robmachado
Merge pull request #911 from robmachado/master
c97842c 
[UPD] ISSUE #869 Correção referente ao processo de limpeza dos campos…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
em progresso falha/bug
Projects
None yet
369E
Milestone
No milestone
Development

No branches or pull requests
2 participants
@weguri @robmachado
0