-
-
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
Homepage pagination does not add other parameters #19579
Comments
@ztj1993 can you please try the latest rc version https://github.com/joomla/joomla-cms/releases/tag/3.8.5-rc? As this release contains patches to be released stable today. |
@zero-24 OK, I'm doing more tests, as if this was just the case on the front page, and the other menu pages wouldn't have happened that way. |
@zero-24 It is true that the test does not appear on the front page. |
@ztj1993 can you please clarify what you mean? Did the 3.8.5-rc package fixed the problem or not? |
@zero-24 The effect of this problem is not very big now, only a few special cases will be used. To implement the problem steps: If the first page to do filtering, this problem will have an impact! The above content for software translation! |
i have seen this bug, and added a workaround in our pagination code, When the paginated page is the home page then all URL variables that do not exist in the home page page menu item are lost, thus you can not e.g. paginate a filtered view The bug is in SiteRouter.php |
Why do you write "workarounds" instead of fixing the issue upstream and help everyone? |
Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/19579 |
closed as having Pull Request #19582 |
Today in the development, found that the home page can not add other parameters!
Desired results:
http://domain/?aaa=1
http://domain/?aaa=1&start=10
http://domain/?aaa=1&start=20
The result of the present:
http://domain/?aaa=1
http://domain/?start=10
http://domain/?start=20
Should it be like this?
The above content for software translation!
The text was updated successfully, but these errors were encountered: