angular-ui/ui-grid

Sorting works inconsistently , when table column contains null/undefined/empty values

Open

#3.360 aberto em 24 de abr. de 2015

Ver no GitHub
 (15 comments) (1 reaction) (1 assignee)JavaScript (2.496 forks)batch import
good first issue

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

I observed that the handlenulls method present in all sort methods(exmaple : alphasort) is working inconsistently. It is unable to handle the empty values in present in the table while sorting. Note - I am using Sort Ascending and Sort Descending options and not the header options.

Scenario - When a column containing null values is sorted -

  1. Step 1 : Ascending order - null values are pushed at the bottom, values gets sorted correctly
  2. Step 2 : Descending order - nothing happens ,sorting does not work.

Guia do colaborador

Direção de pesquisa
Inspecione o método 'handlenulls' nas funções de ordenação alphaSort (ou similares) no código fonte do ui grid. Rastreie a lógica de ordenação para entender por que a ordenação decrescente falha após a ordenação crescente quando valores nulos estão presentes. Garanta que o tratamento de nulos funcione consistentemente para ambas as ordens, crescente e decrescente.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65