angular-ui/ui-grid

Since that the treeView module uses treeBase module, saveState module is broken

Open

#4.742 geöffnet am 20. Nov. 2015

Auf GitHub ansehen
 (10 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (2.496 Forks)batch import
good first issuegrid-groupinggrid-tree-viewtype: bug

Repository-Metriken

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

Beschreibung

Hi,

I wanted to implement treeView on my grid, and i wanted to restore tree state after refreshing data.

In the saveState module, in saveTreeView and restoreTreeView functions, there is this condition : if ( !grid.api.treeView || !grid.options.saveTreeView ){ return {}; }

It seems obsolete since the treeView module uses treeBase module , isn't it ? If i remove the condition, of course i get this error log : TypeError: Cannot read property 'getTreeView' of undefined

I missed something or i'm true ?

Contributor Guide

Research-Richtung
Untersuchen Sie, warum das Speichern des Baumansichtszustands fehlschlägt. Die Bedingung, die grid.api.treeView überprüft, ist veraltet, da das treeView Modul treeBase verwendet, aber getTreeView ist undefiniert. Stellen Sie fest, ob treeBase eine andere API bereitstellt oder ob die Bedingung aktualisiert werden sollte, um treeBase zu überprüfen.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65