-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pagenator error #44257
Comments
It is not clear to me, if you are describing a problem in Joomla itself or in a third-party component. Security versions 4.4.7 and 5.1.3 introduced changes in the war pagination works. Here is an explanation and the way extensions can add parameters: https://docs.joomla.org/J5.x:Pagination_class_removes_query_parameters |
Hello In Last Joomla versions the Pagenator API start work ONLY with parameters from this list
The if I use any other parameter "catid", "name", ...... The API with Joomla Pagenator will remove it. Please turn OFF Router and create link to page with Pagenator - and add to that link any parameter not in your list(which I show before) - and Pagenator will not support it. Please or fix this or return back old Pagenator file Regards This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44257. |
May be setAdditionalUrlParam('month', $state->get('filter.month')); Is a good solution - only as result in configure able "Search forms" - will need set a lot additional settings - so more code, more complicate component, more errors, low site speed .... and as me seem this "That behavior create a cache poisoning attack vector" must resolve with another methods Regards This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44257. |
It is still not clear if you are having a problem with Joomla Core or a 3rd part extension. Please let us know. |
Hello
Joomlla 4.4.8 and Joomla 5.1.4
Steps to reproduce the issue
any url where use pagenator, created like describer there:
https://ordasoft.com/Forum/BookLibary-Component-Support/49107-Navigation-Link-Issues-on-Next-Page.html
Pagenator not save in URL some parameter which not exist in your params list, in my case "catid"
Pagenator remove it
Expected result
Must save it. Any Parameter must exist in pagenator
Actual result
"catid" removed from Pagenator. And any other. You save only "default list"
System information (as much as possible)
Error in file:
libraries/src/Pagination/Pagination.php
You save only parameters in url which exist in
In old Joomla version
libraries/src/Pagination/Pagination.php
work good
if I will use:
$defaultUrlParams = [
'format' => 'CMD',
'option' => 'CMD',
'controller' => 'CMD',
'view' => 'CMD',
'layout' => 'STRING',
'task' => 'CMD',
'template' => 'CMD',
'templateStyle' => 'INT',
'tmpl' => 'CMD',
'tpl' => 'CMD',
'catid' => 'STRING',
'id' => 'STRING',
'Itemid' => 'INT',
];
catid - will exist in pagenator URL
Additional comments
Regards
Andrew
The text was updated successfully, but these errors were encountered: