angular-ui/ui-grid

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

Open

#5.520 geöffnet am 5. Juli 2016

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
good first issue

Repository-Metriken

Stars
 (5.395 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Vergleichen Sie die Methoden getIntersectionValueFiltered und getCellDisplayValue im ui grid Quellcode, um die Änderung zu verstehen, die den $parse Fehler verursacht hat. Konzentrieren Sie sich darauf, wie Zellenwerte analysiert werden und ob die neue Methode ein anderes Wertformat annimmt.
Tech Stack
javascriptangular
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
AngularJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40