sjelfull / craft-audit

Audit log for Craft 3
https://superbig.co
Other
20 stars 11 forks source link

Pagination URLS broken in the admin #52

Closed aj-adl closed 5 years ago

aj-adl commented 5 years ago

Pagination is broken within the Audit admin screen, in that links all print the same (incorrect) url.

Issue with Pagination for this plugin on craft admin screen

This is because the pattern required by JasonGrimes\Paginator is being fed through a Craft function that encodes any params.

As a result, the Url Pattern supplied to the Paginator doesn't contain any identifiable tokens for the paginator to replace, resulting in all pagination links pointing to /admin/audit?page=%28%3Anum%29, which is page=(:num) URI encoded via the http_build_query() function in UrlHelper::_createURL().

PR supplied - #51