This repository has been archived by the owner on Mar 11, 2019. It is now read-only.
Modificações no BRStrings para tratamento de hífens e espaços. #53
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
O método titleize estava gerando títulos diferente dos do método sobrescrito quando o texto continha hífen.
| @@ -15,19 +15,27 @@ | |||
| 'Érica da Silva' => 'érica da silva', | |||
| 'Íris Santos' => 'íris santos', | |||
| 'Paulo dos Santos' => 'paulo dos saNTos', | |||
| ' José da Silva ' => ' josé da silva ', | |||
| 'José da Silva' => ' josé da silva ', | |||
| 'Knut Schimidt-Nielsen' => 'Knut SCHIMIDT-NIELSEN', | |||
There was a problem hiding this comment.
Não tenho certeza se é este o comportamento correto para normalizar nomes próprios com hífens.
|
👍 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ao tentar substituir um método de formatação de nomes que tinha criado pelo do brazilian-rails, alguns testes não passaram, notadamente o "titleize". A diferença estava na manutenção do hífen, enquanto na versão original eles eram trocados por espaços.
Alterei o método titleize (e o nome_proprio, que acabou impactado) e adicionei testes de comparação com a versão original.
Já que estava com a mão na massa, substituí também chamadas gsub usadas nos métodos que alteram a instância da própria string (métodos com bang!).
Por fim, já que é a maneira gramaticamente correta, alterei o método nome_proprio para realizar a remoção de espaços extras.
Espero ajudar com esta contribuição.