Configure -> Tasks "Per Page" pulldown doesn't do anything

[jrafanie]

Verified on chrome and firefox.

When you visit "my vm analysis tasks" or others in this section, the per page pulldown on both the top and bottom show 50:

If I change the top pulldown to 5:

3 issues:

• It still shows 50 items in the table content
• It still shows 1-50 of 126 next to the changed pulldown
• The bottom pulldown still shows the 50 in the pulldown, see below

[dclarizio]

@mzazrivec Please test this in 5.3 and if it is also present there, open BZs for both 5.3.z target and 5.4 target. Thx, Dan

[jrafanie]

@mzazrivec bump, can you see if this is still an issue? The tasks page was pretty hard to use with this bug.

[mzazrivec]

@jrafanie Yup, I was able to reproduce this problem just fine. Matter of fact, I'm working
on a fix for this right now. Though the problematic routine / method affects multiple pages
in our app, so the fix isn't going to be as easy as I thought ...

[jrafanie]

🍪 Thanks @mzazrivec

[mzazrivec]

I have a fix for this problem ready & working, though there's another (related) problem
showing on this page.

1. You hit the refresh button -> the page refreshes
2. You try to navigate the pages in the rendered list, though this finishes with an ISE:

Started GET "/miq_proxy/button?page=2&pressed=miq_task_reload"
ActionController::RoutingError (No route matches [GET] "/miq_proxy/button")
...

The routing error here is a bit miselading. We're not missing a route here, but the list view
is set to make incorrect request here. Correctly, it should be doing

GET /miq_proxy/change_tab?page=2&tab=2

[mzazrivec]

Actually, after looking closely at the problem described in the previous comment
it seems that this is not just a problem of this parciular page, but a general problem
of all list that have a refresh button.

I'll be opening a separate issue for this.

[mzazrivec]

I reported the bug described in last comments as a separate issue #1182

[mzazrivec]

I believe this issue should be now closed, the problem described here has been
addressed with pull request #1231