search

geekcom / validator-docs

Validação de CPF, CNPJ, CNH, NIS, DDD, Título Eleitoral e Cartão Nacional de Saúde com Laravel.
MIT License
566 stars 96 forks source link

Fix Sanitization class to deal with null values #112

Closed nerijunior closed 3 years ago

nerijunior commented 3 years ago

O título do PR é auto explicativo. Testando no Laravel 8 e recebendo o campo em branco, o preg_replace falha porque não é string|array e sim null.

vicentimartins commented 3 years ago

@nerijunior é possível add testes ao seu PR? Sua solução parece funcionar bem, mas seria muito bom ter teste garantindo sua contrib.

nerijunior commented 3 years ago

vou adicionar!

nerijunior commented 3 years ago

@vicentimartins os testes estão feitos, o problema é que o CI não vai passar enquanto o PR #108 não foi aceito.

vicentimartins commented 3 years ago

@nerijunior muito bom Neri, obg por add o teste.

O #108 vai atualizar a versão do PHP para a lib, então sua checagem vai passar tranquilamente.