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

Menu manager pagination broken in 3.5.0 beta #8467

Merged
merged 1 commit into from
Nov 18, 2015
Merged

Conversation

wojsmol
Copy link
Contributor

@wojsmol wojsmol commented Nov 17, 2015

Simple fix for #8456.

A copy of the original issue description

Steps to reproduce the issue

Install Joomla 3.5.0 beta
Open Administrator->Menus->Main menu
Add at least 5 new menu items
Change limit dropdown to "5"
Open search tools, select "Published".
Change to page 2, everything ok.
Change back to page 1, everything ok.
Open search tools again, select "- Select status-".
Try going to page 2 again.

Expected result

Page 2 shown

Actual result

Page 1 shown again

Additional comments

Please note that this is using 3.5.0 beta, so the changes regarding pagination included here do not fix this problem.

For details see that issue.

Simple fix for joomla#8456. For details see that issue.
@Bakual
Copy link
Contributor

Bakual commented Nov 17, 2015

I have tested this item ✅ successfully on 64f76a1


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

@izharaazmi
Copy link
Contributor

@wojsmol It would be better and easier to understand if you can copy the title & details here from the original issue. Also keeping the reference to that issue.

@osignell
Copy link
Contributor

I have tested this item ✅ successfully on 64f76a1


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

@infograf768
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 18, 2015
Bakual pushed a commit that referenced this pull request Nov 18, 2015
@Bakual Bakual merged commit c45e354 into joomla:staging Nov 18, 2015
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 18, 2015
@Bakual Bakual added this to the Joomla! 3.5.0 milestone Nov 18, 2015
@wojsmol wojsmol changed the title fix for joomla#8456 Menu manager pagination broken in 3.5.0 beta Nov 18, 2015
@wojsmol
Copy link
Contributor Author

wojsmol commented Nov 18, 2015

@izharaazmi Done

@wojsmol wojsmol deleted the patch-3 branch November 18, 2015 08:51
@izharaazmi
Copy link
Contributor

@wojsmol Thanks 👍

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.

None yet

6 participants