angular-ui/ui-grid

uiGridValidateService.setValidator have newValue & oldValue param values reversed

Open

#5.059 aberto em 3 de fev. de 2016

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)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 was testing the sample in Plunkr and printed the newValue and oldValue parameters. Their values have been reversed. "NewValue" is showing old value and vice versa. Below is documented function with 2 alert statements.

uiGridValidateService.setValidator('startWith', function(argument) { return function(newValue, oldValue, rowEntity, colDef) { if (!newValue) { return true; // We should not test for existence here } else { alert('newValue ' + newValue); alert('oldValue ' + oldValue); return newValue.startsWith(argument); } }; }, function(argument) { return 'You can only insert names starting with: "' + argument + '"'; }

);

  • Nishat.

Guia do colaborador

Direção de pesquisa
Examine a função uiGridValidateService.setValidator para entender como newValue e oldValue são passados para o callback do validador. Verifique se os parâmetros estão trocados na implementação.
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.
Menos de 1 hora
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
JavaScriptAngularJS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70