[TASK] Follow-up: Introduce native pagination for lists
- Switch the order of constructor arguments $currentPageNumber and $itemsPerPage of both the ArrayPaginator and QueryResultPaginator classes as the $currentPageNumber needs to be set on every object construction whereas the $itemsPerPage argument only needs to be changed for paginators that use a differen amount of items per page other than the default of 10. - Change visibility of properties of both the ArrayPaginator and the QueryResultPaginator from protected to private as both classes are marked final. Releases: master Resolves: #89754 Change-Id: If900c1b786aef7f155bbe0b06cdd167f59e8c9ff Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62418 Tested-by:TYPO3com <noreply@typo3.com> Tested-by:
Daniel Goerz <daniel.goerz@posteo.de> Tested-by:
Susanne Moog <look@susi.dev> Reviewed-by:
Daniel Goerz <daniel.goerz@posteo.de> Reviewed-by:
Susanne Moog <look@susi.dev>
Showing
- typo3/sysext/core/Classes/Pagination/ArrayPaginator.php 5 additions, 5 deletionstypo3/sysext/core/Classes/Pagination/ArrayPaginator.php
- typo3/sysext/core/Documentation/Changelog/master/Feature-89603-IntroduceNativePaginationForLists.rst 1 addition, 1 deletion...aster/Feature-89603-IntroduceNativePaginationForLists.rst
- typo3/sysext/core/Tests/Unit/Pagination/ArrayPaginatorTest.php 7 additions, 6 deletions.../sysext/core/Tests/Unit/Pagination/ArrayPaginatorTest.php
- typo3/sysext/extbase/Classes/Pagination/QueryResultPaginator.php 5 additions, 5 deletions...ysext/extbase/Classes/Pagination/QueryResultPaginator.php
- typo3/sysext/extbase/Tests/Functional/Pagination/QueryResultPaginatorTest.php 7 additions, 6 deletions.../Tests/Functional/Pagination/QueryResultPaginatorTest.php
Please register or sign in to comment