Bug 1175865 - UI: pressing "Reload" button in Configuration -> Tasks page messes up list navigation
Summary: UI: pressing "Reload" button in Configuration -> Tasks page messes up list na...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.3.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.3.3
Assignee: Aparna Karve
QA Contact: Nandini Chandra
URL:
Whiteboard:
: 1171151 (view as bug list)
Depends On: 1171167
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-12-18 19:17 UTC by Aparna Karve
Modified: 2015-03-23 16:21 UTC (History)
8 users (show)

Fixed In Version: 5.3.3
Doc Type: Bug Fix
Doc Text:
Clone Of: 1171167
Environment:
Last Closed: 2015-02-25 16:11:50 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2015:0261 0 normal SHIPPED_LIVE cfme (5.3.3) bug fix and enhancement update 2015-02-25 21:09:28 UTC

Description Aparna Karve 2014-12-18 19:17:59 UTC
+++ This bug was initially created as a clone of Bug #1171167 +++

Description of problem:
Configuration -> Tasks page contains a list of various tasks executed / to be
executed by the application. The list always contains navigation buttons: first,
previous, next, last.

Normally, the navigation works OK and navigating through the list
would mean doing requests like:

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

Though after pressing the "Reload" button, which is the following request:

    POST /miq_proxy/button?pressed=miq_task_reload

the received tasks list is rendered incorrectly and navigating
through the list would try to do the following request:

    GET /miq_proxy/button?page=2&pressed=miq_task_reload

which generates an ISE:

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

In another words, pressing the reload button results into
an incorrectly rendered tasks list.


Version-Release number of selected component (if applicable):
CFME 5.3

How reproducible:
Always

Steps to Reproduce:
1. Navigate to Configuration -> Tasks page
2. Navigate through the rendered tasks list (next, previous, etc.)
3. Hit reload & try to navigate throught the list again

Actual results:
Internal server error

Expected results:
The navigation works correctly.

Additional info:
https://github.com/ManageIQ/manageiq/issues/1182

--- Additional comment from CFME Bot on 2014-12-17 16:55:59 EST ---

New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/15162a22fb4acc7b18d58188967acf35f00d70eb

commit 15162a22fb4acc7b18d58188967acf35f00d70eb
Author:     Aparna Karve <akarve>
AuthorDate: Thu Dec 11 15:22:15 2014 -0800
Commit:     Aparna Karve <akarve>
CommitDate: Mon Dec 15 13:46:42 2014 -0800

    Fix for paging url path
    
    The paging url path should be constructed using action_url that is
    passed by the controllers to the paging control.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1171167
    #1182

 vmdb/app/helpers/application_helper.rb          | 11 ++++++----
 vmdb/app/views/layouts/_pagingcontrols.html.erb | 28 ++++++++++++-------------
 vmdb/spec/helpers/application_helper_spec.rb    |  3 +--
 3 files changed, 22 insertions(+), 20 deletions(-)

Comment 1 CFME Bot 2014-12-18 23:32:17 UTC
New commit detected on cfme/5.3.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=b1f48c22a422b08bb73fa1b685d9321bae9099e2

commit b1f48c22a422b08bb73fa1b685d9321bae9099e2
Author:     Aparna Karve <akarve>
AuthorDate: Thu Dec 11 15:22:15 2014 -0800
Commit:     Aparna Karve <akarve>
CommitDate: Thu Dec 18 11:21:08 2014 -0800

    Fix for paging url path
    
    The paging url path should be constructed using action_url that is
    passed by the controllers to the paging control.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1175865
    #1182

 vmdb/app/helpers/application_helper.rb          | 11 ++++++----
 vmdb/app/views/layouts/_pagingcontrols.html.erb | 28 ++++++++++++-------------
 vmdb/spec/helpers/application_helper_spec.rb    |  3 +--
 3 files changed, 22 insertions(+), 20 deletions(-)

Comment 3 Nandini Chandra 2015-01-30 21:00:56 UTC
Verified in 5.3.3

Verified that there are no routing errors while navigating through tasks list after hitting reload.

Comment 4 Milan Zázrivec 2015-02-02 10:31:39 UTC
*** Bug 1171151 has been marked as a duplicate of this bug. ***

Comment 6 errata-xmlrpc 2015-02-25 16:11:50 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-0261.html


Note You need to log in before you can comment on or make changes to this bug.