angular-ui/ui-grid

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

Open

#5 520 ouverte le 5 juil. 2016

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)JavaScript (2 496 forks)batch import
good first issue

Métriques du dépôt

Stars
 (5 395 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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.

Guide contributeur

Direction de recherche
Comparez les méthodes getIntersectionValueFiltered et getCellDisplayValue dans le code source d'ui grid pour comprendre le changement qui a introduit l'erreur $parse. Concentrez vous sur la façon dont les valeurs des cellules sont analysées et si la nouvelle méthode suppose un format de valeur différent.
Stack technique
javascriptangular
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
AngularJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40