angular-ui/ui-grid

Treebase getRowChildren returning wrong results

Open

#5 632 ouverte le 18 août 2016

Voir sur GitHub
 (0 commentaires) (0 réactions) (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

The getRowChildren() function on the treebase control seems to be returning unexpected rows when a row processor is registered for filtering. The issue can be reproduced in the following plunker. http://plnkr.co/edit/xHfc63yZWdgr1O3qYjIm

On the plunker, type "josh" and press the "Log First Group Visible Rows" button. Notice that in the console, 3 rows will be logged, 2 of which actually belong to the first group, but you will notice that there is an entity in there that isn't even a part of this group, yet getRowChildren is stating that this is a child of that row.

Guide contributeur

Direction de recherche
Reproduisez le problème en utilisant le lien Plunker fourni. Concentrez vous sur la fonction getRowChildren dans le contrôle treebase, en particulier sur la façon dont les processeurs de lignes affectent les lignes visibles. Tracez la logique qui détermine les enfants et identifiez pourquoi une ligne d'un groupe différent est incluse par erreur. Vérifiez si l'enregistrement du processeur de lignes ou la logique de filtrage interfère avec le mappage parent enfant.
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptAngular
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55