Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
On the reports page admin/reports/profiler/list
pagination is missing.
Comment | File | Size | Author |
---|---|---|---|
#12 | devel-add_pagination_to_webprofiler_reports_page-2755075-with_storage.patch | 5.98 KB | sardbaba |
|
Comments
Comment #2
sardbaba CreditAttribution: sardbaba as a volunteer commentedI think the absence of pagination is by design; anyway here's a patch which removes the select "Limit" from the form and adds the pagination to the results.
Comment #3
sardbaba CreditAttribution: sardbaba as a volunteer commentedComment #5
sardbaba CreditAttribution: sardbaba as a volunteer commentedRetry
Comment #6
sardbaba CreditAttribution: sardbaba as a volunteer commentedComment #7
lussolucaThanks for the patch!
We have to do that also for the file storage backend (Drupal\webprofiler\Profiler\FileProfilerStorage).
Comment #8
sardbaba CreditAttribution: sardbaba as a volunteer commentedOk, this patch should do the job also for the storage backend.
Maybe file reading can be improved to read only the range of lines of the current page but I can't find a way to do that.
Comment #9
sardbaba CreditAttribution: sardbaba as a volunteer commentedThere is just a little problem when searching by IP/URL with file storage, because the pager still shows all pages even if results are less than the maximum for the page (50). I'll try to fix this later.
Comment #10
sardbaba CreditAttribution: sardbaba as a volunteer commentednop
Comment #11
sardbaba CreditAttribution: sardbaba as a volunteer commentedComment #12
sardbaba CreditAttribution: sardbaba as a volunteer commentedHere is the full patch, with pagination on both database and file storage.
As said above maybe file reading can be improved to read only the range of lines of the current page but I can't find a way to do that.