fix(lookup): corrige ordenação de campos numéricos e data#2785
Open
guilhermeverissimo14 wants to merge 1 commit intopo-ui:masterfrom
Open
fix(lookup): corrige ordenação de campos numéricos e data#2785guilhermeverissimo14 wants to merge 1 commit intopo-ui:masterfrom
guilhermeverissimo14 wants to merge 1 commit intopo-ui:masterfrom
Conversation
A pesquisa em campos lookup com valores numéricos ou data não estava ordenando corretamente. Ao tentar ordenar por valores numéricos como "mass" no exemplo do portal, a ordenação ficava incorreta tanto ascendente quanto descendente. A função sortValues foi ajustada para detectar strings numéricas e convertê-las para números antes da comparação, garantindo ordenação numérica ao invés de alfabética. Foram adicionados testes para validar a conversão de strings numéricas, strings vazias, decimais e números negativos. Fixes po-ui#2674
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
fix(lookup): corrige ordenação de campos numéricos e data
A pesquisa em campos lookup com valores numéricos ou data não estava ordenando corretamente. Ao tentar ordenar por valores numéricos como "mass" no exemplo do portal, a ordenação ficava incorreta tanto ascendente quanto descendente.
A função sortValues foi ajustada para detectar strings numéricas e convertê-las para números antes da comparação, garantindo ordenação numérica ao invés de alfabética. Foram adicionados testes para validar a conversão de strings numéricas, strings vazias, decimais e números negativos.
Fixes #2674
po-lookup
#2674
PR Checklist [Revisor]
Qual o comportamento atual?
Na pesquisa de um campo lookup, quando o valor é numérico ou data, a
ordenação não fica correta. Ao tentar ordenar pela coluna "mass" no
exemplo do portal (Star Wars films), a ordenação fica incorreta tanto
ascendente quanto descendente.
Isso ocorre porque a função sortValues estava fazendo comparação
alfabética de strings, onde "100" vem antes de "20" alfabeticamente,
mas numericamente deveria ser o contrário.
Qual o novo comportamento?
A função sortValues foi corrigida para detectar automaticamente strings
que representam valores numéricos puros e convertê-las para números
antes da comparação. Isso garante ordenação numérica correta.
Tratamentos adicionados:
Foram adicionados 9 novos testes unitários cobrindo todos os cenários.
Simulação
Esta correção pode ser validada no portal PO-UI: