Skip to content
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.

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

Erro na busca por CEP #2

Closed
dlimars opened this issue Nov 10, 2017 · 8 comments
Closed

Erro na busca por CEP #2

dlimars opened this issue Nov 10, 2017 · 8 comments
Labels

Comments

@dlimars
Copy link

dlimars commented Nov 10, 2017

Após uma busca pelo CEP: 18071-051, só me retornava "null" como resultado, após análise, constatei que o erro era causado porque o nome da rua é "Rua Constantino Verrone", que ao rodar o método validation, o mesmo acusava erro:

    private function validation($get)
    {
        return !((int)preg_match('/(erro)/', $get) === 1);
    } 

ou seja, todos os ceps que contiverem a palavra erro no endereço irão falhar

@dlimars
Copy link
Author

dlimars commented Nov 10, 2017

Outro exemplo é com o cep 97542080 cujo nome da rua é Rua Barão de Cerro Largo

@fulviocanducci
Copy link
Collaborator

fulviocanducci commented Nov 13, 2017

Caro @dlimars tudo bem, espero que sim! Agradeço a abertura do ISSUE!

Mas,

Eu acabei de testar o código inclusive tem uma página http://zipcodedemo.herokuapp.com/ que também funcionou corretamente o código. Aquele busca da palavra "erro" até vou verificar, mas, não é por causa disso que não trouxe o resultado, porque nos testes feitos funcionaram inclusive no site

Observe:

save12389

Um outro observação é que quando dá erro seria assim:

erro

Qualquer coisa me mande o seu código e como esta resgatando, por favor?

Muito Obrigado!

@dlimars
Copy link
Author

dlimars commented Nov 13, 2017

Opa @fulviocanducci , então, antes de postar, utilizei a página demo também, mas olha como estou utilizando:
screen shot 2017-11-13 at 10 47 14

e este é o resultado:
screen shot 2017-11-13 at 10 49 08

Quanto ao erro, eu resolvi colocar um dump antes do retorno do parser, da seguinte forma:
screen shot 2017-11-13 at 10 51 58

e olha o resultado:
screen shot 2017-11-13 at 10 53 04

Nota que o segundo parâmetro retorna false que é o que roda a validação.
Nota que o método validation($get) roda sobre o retorno inteiro da consulta e não apenas verificando o retorno { "erro" : "true" } como você comentou.

@fulviocanducci
Copy link
Collaborator

@dlimars já estou verificando porque realmente é um bug! Peço a compreensão e logo mais já posto a solução e a nova versão do Canducci Cep. e Muito obrigado pela observação!

Quando postar já te aviso.!

@fulviocanducci
Copy link
Collaborator

fulviocanducci commented Nov 13, 2017

@dlimars bug resolvido.! Nova versão

https://packagist.org/packages/canducci/cep#v2.0.2

Qualquer problemas, pode avisar e obrigado.

Uma pergunta você utiliza ele fora do Laravel?

@dlimars
Copy link
Author

dlimars commented Nov 13, 2017

Opa, obrigado @fulviocanducci, na verdade eu uso com Laravel, eu só não sou muito chegado nos Facades, pq?

@netdragoon
Copy link
Owner

@dlimars se for utilizar para Laravel utilize esse:

https://packagist.org/packages/canducci/zipcode

Ele otimiza o desempenho e só busca se não tiver no cache. é útil porque diminui o numero de requisições.

Se esse pacote deu certo se poderia comentar eu fiz vários testes e agora acredito que não terá problemas!

@dlimars
Copy link
Author

dlimars commented Nov 13, 2017

Opa @netdragoon @fulviocanducci deu certinho a correção, muito obrigado!

@dlimars dlimars closed this as completed Nov 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants