Skip to content

Conversation

@SharkyKZ
Copy link
Contributor

Pull Request for Issue #22906.

Summary of Changes

Corrects pagination behavior when uninstalling extensions.

Testing Instructions

Install JCE (https://extensions.joomla.org/extension/jce/) or some other extension package with multiple (at least 6) extensions.
Go to Extensions -> Manage.
Set list limit to 5.
Search for JCE.
Click once on Type header to order by type so that JCE Extension Package ends up on first result page.
Go to page 2.
Uninstall the extensions on page 2.

Expected result

You are brought back to the first result page where remaining 5 extensions are shown.

Actual result

You remain on the second page and see this message:

There are no extensions installed matching your query.

Documentation Changes Required

No.

@Quy
Copy link
Contributor

Quy commented Jun 13, 2019

I have tested this item ✅ successfully on 5626fa7


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

1 similar comment
@alikon
Copy link
Contributor

alikon commented Jun 13, 2019

I have tested this item ✅ successfully on 5626fa7


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

@alikon
Copy link
Contributor

alikon commented Jun 13, 2019

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 13, 2019
@HLeithner HLeithner merged commit 496fb1e into joomla:staging Jun 14, 2019
@HLeithner
Copy link
Member

thx

@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Jun 14, 2019
@HLeithner HLeithner added this to the Joomla 3.9.9 milestone Jun 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants