angular-ui/ui-grid

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

Open

#4 742 ouverte le 20 nov. 2015

Voir sur GitHub
 (10 commentaires) (0 réactions) (1 assigné)JavaScript (2 496 forks)batch import
good first issuegrid-groupinggrid-tree-viewtype: bug

Métriques du dépôt

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

Description

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 ?

Guide contributeur

Direction de recherche
Enquêtez sur la raison pour laquelle la sauvegarde de l'état de la vue arborescente échoue. La condition vérifiant grid.api.treeView est obsolète car le module treeView utilise treeBase, mais getTreeView est indéfini. Déterminez si treeBase fournit une API différente ou si la condition doit être mise à jour pour vérifier treeBase.
Stack technique
javascript
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65