Skip to content
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

[3.9] Fix navigating to the first page in pagination when SEF is off #23042

Merged
merged 1 commit into from Nov 17, 2018

Conversation

csthomas
Copy link
Contributor

Pull Request for Issue #22983

Summary of Changes

Reverted a few lines added in the previous PR #19467
Changed the unit test method JPaginationTest::buildLink($url) to better emulate Router::build($url), where empty parameters such as &limitstart= are always removed from the URL.

Testing Instructions

Test #22983 when SEF is ON and OFF.

Expected result

You can back to the first page in pagination when SEF is OFF.

Actual result

When SEF is OFF the limitstart parameter remains in the link to the first page and you can not back to the first page.

Documentation Changes Required

No

@hendrikbehncke
Copy link
Contributor

Test successful on 3f30b88


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23042.

@Quy
Copy link
Contributor

Quy commented Nov 13, 2018

I have tested this item ✅ successfully on 3f30b88


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23042.

@Quy
Copy link
Contributor

Quy commented Nov 13, 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23042.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 13, 2018
@mbabker mbabker added this to the Joomla 3.9.1 milestone Nov 17, 2018
@mbabker mbabker merged commit 30ab125 into joomla:staging Nov 17, 2018
@joomla-cms-bot joomla-cms-bot added PR-staging Unit/System Tests and removed RTC This Pull Request is Ready To Commit labels Nov 17, 2018
@csthomas csthomas deleted the fix_nonsef_limitstart branch November 17, 2018 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants