angular-ui/ui-grid

paginationApi.getLastRowIndex and grid.options.totalItems wrong when filtered and data externally changed

Open

#6 897 ouverte le 27 nov. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (2 496 forks)batch import
help wanted

Métriques du dépôt

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

Description

I am using paginationApi.getLastRowIndex() and grid.options.totalItems inside a custom pagination template, done via $templateCache.put('ui-grid/pagination',.

That works in general. However, the results are wrong if two conditions happen:

  • A column filter is active
  • While the column filter is active, the data is externally changed, rendering the filter condition false for at least one row.

The actual result displayed is, in this case, the "maximum" (unfiltered) number of items - as if the filter was not there.

I have created a Plunker: http://next.plnkr.co/edit/i32TV4PrYIJby8Uy?preview

Steps for reproducing error there:

  • Filter for City: "Tokio"

  • Click on "Transfer Maria from Tokio to Paris"

  • Pagination footer wrong

Guide contributeur

Direction de recherche
Inspectez l'exemple Plunker et tracez comment les compteurs de pagination sont recalculés lorsque les données externes changent alors qu'un filtre est actif. Examinez la logique de pagination interne de la grille (par exemple, getLastRowIndex et totalItems) pour voir s'ils réévaluent correctement les données filtrées.
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
AngularJSui grid
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75