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

Correção de bug na formatação de CPF + Criação de novo parâmetro #98

Closed
wants to merge 2 commits into from

Conversation

adilsonjuniordev
Copy link

Ao utilizar o formatador de CPF CpfInputFormatter(), percebi que o mesmo não estava se portando de maneira correta.
Fiz o teste utilizando iOS 17, com a versão 1.14.3 do package, e o comportamento foi o seguinte:

brasil_fields_bug

Efetuei a correção, e também já criei um teste, o qual pode ser revisado também, case seja necessário.

Aproveitando a correção, tomei a liberdade de adicionar um parâmetro nomeado opcional no formatter:

bool deleteSeparatorsAutomatically = true;

Ele é opcional para não gerar nenhum break change, e o objetivo é controlar o seguinte caso:
Caso seja true(default): irá apagar os delimitadores (.) (-) automaticamente ao apagar os caracteres.
Caso seja false: NÃO irá apagar os delimitadores (.) (-) automaticamente ao apagar os caracteres.

Caso o PR seja aprovado, eu me disponho a atualizar a documentação também.

Atenciosamente,
Adilson Junior.

…l) para habilitar o apagamento automático dos delimitadores ao apagar caracteres do campo
…l) para habilitar o apagamento automático dos delimitadores ao apagar caracteres do campo
@rubensdemelo
Copy link
Collaborator

@adilsonjuniordev este comportamento acontece quando o formatter FilteringTextInputFormatter.digitsOnly não é utilizado. Verifique o seu código e depois atualiza aqui.

@adilsonjuniordev
Copy link
Author

@rubensdemelo Obrigado pelo retorno. Eu havia tentado com o FilteringTextInputFormatter.digitsOnly sim, porém não estava formatando corretamente. Agora fazendo um teste mais preciso, percebi que a ordem dos formatter importa.... Não havia me atentado para isso, o FilteringTextInputFormatter.digitsOnly deve vir antes de CpfInputFormatter().

Obrigado e pode desconsiderar o meu PR, eu estava tentando ajudar mas não tinha percebido este detalhe.

Abraço!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants