[#8456] - Menu manager pagination broken in 3.5.0 beta
- Closed
- 17 Nov 2015
- Medium
- Build: master
- # 8456
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.
limistart is reset when the code thinks the filter states changed :)
Reverting #7487 DOES actually fix the problem for me, you just have to try twice right after making the code change in order for the session to pick up the change.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8456.
| Milestone |
Added: |
||
correction.. it works if I use isset() without the additional check.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8456.
I confirm
Using
if (isset($filters[$name]))
{
$new_state = $filters[$name];
}
solves this issue
Can you check that the original issue with the user manager is solved as well this way?
The user manager works fine for me using the version with isset() only.
Who wants to do the PR?
Article Manager displays OK Unpublished articles (state = 0)
No problem either for modules.
@Bakual @infograf768 @osignell please test #8467
Closing as we have a PR
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-17 18:34:59 |
| Closed_By | ⇒ | zero-24 |
Additional info:
The reason for this seems to be that the limitstart value is reset on line 709 in libraries/legacy/model/list.php because new_state is null and cur_state is "1".
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8456.