angular-ui/ui-grid

getTotalPages() is working wrong for Custom pagination

Open

#6246 aperta il 2 giu 2017

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (2496 fork)batch import
good first issue

Metriche repository

Star
 (5395 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hi guys, In this Pagination api method below

getTotalPages: function () { if (!grid.options.enablePagination) { return null; } if (grid.options.useCustomPagination) { return grid.options.paginationPageSizes.length; } return (grid.options.totalItems === 0) ? 1 : Math.ceil(grid.options.totalItems / grid.options.paginationPageSize); }

in this part:

if (grid.options.useCustomPagination) { return grid.options.paginationPageSizes.length; }

It brokes the pagination, since it will never return the real quantity of pages to use, but the quantity in the pageSizes array ....

Guida contributor

Direzione di ricerca
Ispeziona la funzione getTotalPages() nel servizio di paginazione. Il caso di paginazione personalizzata restituisce erroneamente la lunghezza dell'array paginationPageSizes. L'implementazione corretta dovrebbe calcolare il numero totale di pagine in base alla logica di paginazione personalizzata, probabilmente utilizzando totalItems e la dimensione della pagina o proprietà simili.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75