Bug 1397392 - WebUI:Spinner spins infinitely with "uncaught exception: [object Object]" in developer console while accessing "All Saved Reports" for "EvmGroup-approver" user
Summary: WebUI:Spinner spins infinitely with "uncaught exception: [object Object]" in ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.8.0
Assignee: Martin Hradil
QA Contact: Niyaz Akhtar Ansari
URL:
Whiteboard: ui:report
Depends On:
Blocks: 1399217
TreeView+ depends on / blocked
 
Reported: 2016-11-22 12:40 UTC by Ramesh A
Modified: 2017-06-12 16:15 UTC (History)
6 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1399217 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:15:20 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:
rananda: automate_bug-


Attachments (Terms of Use)
Spinner issue in Reports page (110.58 KB, image/png)
2016-11-22 12:40 UTC, Ramesh A
no flags Details

Description Ramesh A 2016-11-22 12:40:08 UTC
Created attachment 1222716 [details]
Spinner issue in Reports page

Description of problem:
spinner spins infinitely with "uncaught exception: [object Object]" in developer console while accessing "All Saved Reports" for "EvmGroup-approver" user

Version-Release number of selected component (if applicable):
5.7.0.11-rc1.20161115160629_46cf4f1

How reproducible:
Very

Steps to Reproduce:
1. Manage a Infra Provider
2. Create a new user and associate him with "EvmGroup-approver"
3. With new user, navigate to Could Intel==> Reports

Actual results:
1. Spinner spins infinitely
2. Browser web console throws "uncaught exception: [object Object]" error
Refer the screenshot for further information

Expected results:
Spinner should stop after loading the page and Should be able to see the "All Saved Reports" page

Additional info:

Comment 3 CFME Bot 2016-11-25 13:51:14 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/100c32a9c725468989c2616afe380c9760bd18d7

commit 100c32a9c725468989c2616afe380c9760bd18d7
Author:     Martin Hradil <mhradil>
AuthorDate: Thu Nov 24 15:55:58 2016 +0000
Commit:     Martin Hradil <mhradil>
CommitDate: Thu Nov 24 15:55:58 2016 +0000

    ManageIQ.explorer.lock_tree - don't crash when the tree doesn't exist
    
    `$(selector).treeview(true)` normally returns a tree instance, but if the tree doesn't exist, it returns the same as `$(selector)`.
    
    Adding a warn & return when attempting to lock/unlock a tree which doesn't exist.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1397392

 app/assets/javascripts/miq_explorer.js | 7 +++++++
 1 file changed, 7 insertions(+)

Comment 4 CFME Bot 2016-11-25 13:51:19 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/2d3bd31a4d166f520e8b24a3792f342ef98e044b

commit 2d3bd31a4d166f520e8b24a3792f342ef98e044b
Author:     Martin Hradil <mhradil>
AuthorDate: Thu Nov 24 15:58:35 2016 +0000
Commit:     Martin Hradil <himdel.com>
CommitDate: Thu Nov 24 21:58:57 2016 +0100

    ReportController - don't attempt to unlock uninitialized trees
    
    when logged in as non-admin, not all accordions and trees are available.
    
    However, loading the CI > Reports page causes all the trees to be unlocked, even if they aren't there.
    
    Adding a check that the tree has actually been instantiated before trying to unlock.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1397392

 app/controllers/application_controller/tree_support.rb | 4 ++++
 app/controllers/report_controller.rb                   | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

Comment 5 CFME Bot 2016-11-28 14:30:59 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/b87dc834b48f09620bbb529a16453efb3056bfb4

commit b87dc834b48f09620bbb529a16453efb3056bfb4
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Fri Nov 25 14:48:34 2016 +0100
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Mon Nov 28 09:25:51 2016 -0500

    Merge pull request #12846 from himdel/bz1397392-report-tree-exists
    
    CI > Reports - don't crash trying to unlock a tree the user has no rights to see
    (cherry picked from commit ff551d00423830d5192fed896ab9ecb33e92bbd2)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1397392

 app/assets/javascripts/miq_explorer.js                 | 7 +++++++
 app/controllers/application_controller/tree_support.rb | 4 ++++
 app/controllers/report_controller.rb                   | 2 +-
 3 files changed, 12 insertions(+), 1 deletion(-)

Comment 7 Niyaz Akhtar Ansari 2017-03-08 11:26:22 UTC
Verified in Version 5.8.0.4


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