- Issue created by @euk
- π¬π§United Kingdom aaron.ferris
Perhaps an admin configurable 'expiry time' setting, where we automatically trim items that are older than 'x'?
- πΊπΈUnited States euk
I am working on refactoring of the report page. It is a bit sub-optimal from the coding and usability perspective:
- Instead of looping trough results to convert each of them into an array, PDO::FETCH_ASSOC could be used to get results as arrays in the first place.
- from the usability perspective, unlikely anyone is interested too see hundreds results on a single page
- it needs a pager paging
- it needs filters - no way to only see results you need
- the results are split into two tables, where it should be one table with filters
- etc.It is easier to refactor and get a good UI.
- Merge request !6Issue #3518878: added pagination to the report page, cleaned up code. β (Open) created by euk
- πΊπΈUnited States euk
I refactored the report page a bit:
- Added automated Pagination and Sort/Order support viaPagerSelectExtender
andTableSortExtender
- Added limit to display 25 records per page,
- refactored and cleaned up the code a bit.This is ready for review and testing.
I am going to create another ticket for adding filtering support.
- πΊπΈUnited States euk
I have changed the ticket from the bug report to feature request. MR7 now contains full refactoring including filters. MR6 contains only report refactoring for performance.