angular-ui/ui-grid

regression on grid cell edit - $parse() throws errors

Fechada

#5.520 aberto em 5 de jul. de 2016

Ver no GitHub
 (2 comentários) (1 reação) (0 responsável)JavaScript (2.496 forks)batch import
good first issue

Métricas do repositório

Stars
 (5.395 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I have recently updated to ui-grid 3.2.1 from 3.1.1. After upgrade I started getting $parse() throwing error while parsing the grid values that was working perfectly fine before upgrade.

When I compared the code I found this change between the versions -

version 3.1.1 - (line 15117 in addAriaLiveRegion function) values.push(currentSelection[i].getIntersectionValueFiltered());

version 3.2.1 - (line 15208 in addAriaLiveregion function) values.push(grid.getCellDisplayValue(currentSelection[i].row, currentSelection[i].col));

If I disable the cellNav feature, the error vanishes. So I thought of filing a regression bug here.

Guia do colaborador

Direção de pesquisa
Compare os métodos getIntersectionValueFiltered e getCellDisplayValue no código fonte do ui grid para entender a mudança que introduziu o erro $parse. Concentre se em como os valores das células são analisados e se o novo método assume um formato de valor diferente.
Pilha de tecnologia
javascriptangular
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
AngularJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40