Bug 1213863 - All Catalogs spinner spins forever
Summary: All Catalogs spinner spins forever
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.4.0
Assignee: Martin Hradil
QA Contact: Shveta
URL:
Whiteboard:
: 1220872 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-04-21 13:07 UTC by Pete Savage
Modified: 2015-06-16 12:59 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-06-16 12:59:13 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:1100 0 normal SHIPPED_LIVE CFME 5.4.0 bug fixes, and enhancement update 2015-06-16 16:28:42 UTC

Description Pete Savage 2015-04-21 13:07:55 UTC
Description of problem: All Catalogs spins forever


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


How reproducible: 100%


Steps to Reproduce:
1. Services -> Catalogs
2. Catalogs (accordion)
3. Click on the Tree Item "All Catalogs"

Actual results:
Spinner spins forever

Expected results:
Spinner shouldn't spin forever

Additional info:
[----] I, [2015-04-21T09:05:18.350945 #2190:6f9e9c]  INFO -- : Processing by CatalogController#accordion_select as JS
[----] I, [2015-04-21T09:05:18.351028 #2190:6f9e9c]  INFO -- :   Parameters: {"id"=>"stcat"}
[----] I, [2015-04-21T09:05:18.408235 #2190:6f9e9c]  INFO -- :   Rendered layouts/_x_form_buttons.html.haml (0.1ms)
[----] I, [2015-04-21T09:05:18.408425 #2190:6f9e9c]  INFO -- :   Rendered layouts/_x_pagingcontrols.html.haml (0.6ms)
[----] I, [2015-04-21T09:05:18.409495 #2190:6f9e9c]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.2ms)
[----] I, [2015-04-21T09:05:18.409816 #2190:6f9e9c]  INFO -- :   Rendered layouts/_info_msg.html.haml (0.1ms)
[----] I, [2015-04-21T09:05:18.409997 #2190:6f9e9c]  INFO -- :   Rendered layouts/_x_gtl.html.haml (1.0ms)
[----] I, [2015-04-21T09:05:18.591418 #2190:6f9e9c]  INFO -- : Completed 200 OK in 240.2ms (Views: 0.2ms | ActiveRecord: 3.9ms)
[----] I, [2015-04-21T09:05:21.152641 #2190:6f9e9c]  INFO -- : Started POST "/catalog/tree_select/?id=root" for 127.0.0.1 at 2015-04-21 09:05:21 -0400
[----] I, [2015-04-21T09:05:21.156377 #2190:6f9e9c]  INFO -- : Processing by CatalogController#tree_select as JS
[----] I, [2015-04-21T09:05:21.156459 #2190:6f9e9c]  INFO -- :   Parameters: {"id"=>"root"}
[----] I, [2015-04-21T09:05:21.216822 #2190:6f9e9c]  INFO -- :   Rendered layouts/_x_form_buttons.html.haml (0.1ms)
[----] I, [2015-04-21T09:05:21.217023 #2190:6f9e9c]  INFO -- :   Rendered layouts/_x_pagingcontrols.html.haml (0.6ms)
[----] I, [2015-04-21T09:05:21.218113 #2190:6f9e9c]  INFO -- :   Rendered layouts/_flash_msg.html.haml (0.1ms)
[----] I, [2015-04-21T09:05:21.218424 #2190:6f9e9c]  INFO -- :   Rendered layouts/_info_msg.html.haml (0.1ms)
[----] I, [2015-04-21T09:05:21.218627 #2190:6f9e9c]  INFO -- :   Rendered layouts/_x_gtl.html.haml (1.0ms)
[----] I, [2015-04-21T09:05:21.226845 #2190:6f9e9c]  INFO -- : Completed 200 OK in 70.3ms (Views: 0.2ms | ActiveRecord: 3.8ms)

Comment 3 Shveta 2015-04-21 19:39:50 UTC
yes , it happens only first 1 or 2 time .

Comment 5 Martin Hradil 2015-04-29 19:03:06 UTC
https://github.com/ManageIQ/manageiq/pull/2796

Comment 7 CFME Bot 2015-04-30 23:05:59 UTC
New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/4bb4957b8e914e6b0f8b741b1e5b262f130e0448

commit 4bb4957b8e914e6b0f8b741b1e5b262f130e0448
Author:     Martin Hradil <mhradil>
AuthorDate: Wed Apr 29 18:35:39 2015 +0000
Commit:     Martin Povolny <mpovolny>
CommitDate: Thu Apr 30 17:40:12 2015 +0200

    ApplicationHelper::javascript_for_toolbar_reload cleanup, don't fail on missing global
    
    explicitly marks #{tb} as global variable, also in _dhtmlxtoolbar partial
    
    use miq_toolbars for toolbar unload (and check for its existence)
    
    however, if the corresponding div is missing, dhtmlXToolbarObject constructor will throw anyway
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1213863

 vmdb/app/helpers/application_helper.rb          | 20 +++++++++++++-------
 vmdb/app/views/layouts/_dhtmlxtoolbar.html.haml |  4 ++--
 2 files changed, 15 insertions(+), 9 deletions(-)

Comment 8 CFME Bot 2015-04-30 23:06:03 UTC
New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/d4c7b130d9dff5b910b6dad81813cc89685d0468

commit d4c7b130d9dff5b910b6dad81813cc89685d0468
Author:     Martin Hradil <mhradil>
AuthorDate: Wed Apr 29 18:42:49 2015 +0000
Commit:     Martin Povolny <mpovolny>
CommitDate: Thu Apr 30 17:40:12 2015 +0200

    _center_buttons should render even when no yaml, _dhtmlxtoolbar handles no yaml
    
    This fixes _center_buttons to always render the #center_buttons_div element, though possibly empty.
    
    Also prevents _dhtmlxtoolbar from crashing when it *is* empty, this enables us to create empty toolbars to be initialized later via the javascript_for_toolbar_reload helper.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1213863

 vmdb/app/views/layouts/_center_buttons.html.haml | 6 +++---
 vmdb/app/views/layouts/_dhtmlxtoolbar.html.haml  | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

Comment 10 Shveta 2015-05-07 03:43:40 UTC
Fixed in 5.4.0.0.25.20150429111523_0455f87

Comment 11 Harpreet Kataria 2015-05-12 17:44:32 UTC
*** Bug 1220872 has been marked as a duplicate of this bug. ***

Comment 13 errata-xmlrpc 2015-06-16 12:59:13 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-1100.html


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