Bug 1380485 - UI: Explorer trees issue with clicking on nodes that are not yet present in tree.
Summary: UI: Explorer trees issue with clicking on nodes that are not yet present in t...
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: Dávid Halász
QA Contact: Niyaz Akhtar Ansari
URL:
Whiteboard:
Depends On:
Blocks: 1383347
TreeView+ depends on / blocked
 
Reported: 2016-09-29 18:08 UTC by Harpreet Kataria
Modified: 2017-06-12 16:09 UTC (History)
7 users (show)

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


Attachments (Terms of Use)

Description Harpreet Kataria 2016-09-29 18:08:30 UTC
Description of problem: UI: Explorer trees issue with clicking on nodes that are not yet present in tree.



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


How reproducible:


Steps to Reproduce:
1. After signing into UI, go to Catalogs Explorer
2. In Service Catalogs accordion, click on one of the Catalog Item on the right side
3. Screen contents remain the same, on right side as well as on the left in the tree

Actual results: Screen contents remain the same, on right side as well as on the left in the tree


Expected results: It should show summary screen of selected catalog item on the right and expand the tree on the left to show selected catalog item in the tree.


Additional info: I think open_parent_nodes method is not loading/selecting correct nodes in the tree, this issue is also seen in other parts of UI. In automate explorer when trying to copy an instance, after instance is copied, tree does not get refreshed.

Comment 5 CFME Bot 2016-10-03 17:31:00 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/cf09c498ff08b297374203c7a9f456c4d6b66088

commit cf09c498ff08b297374203c7a9f456c4d6b66088
Author:     Dávid Halász <dhalasz>
AuthorDate: Mon Oct 3 10:09:44 2016 +0200
Commit:     Dávid Halász <dhalasz>
CommitDate: Mon Oct 3 16:56:24 2016 +0200

    Fixed treeview node conversion in all open_parent_nodes calls
    
    - Moved the treeview conversion to TreeSupport#tree_add_child_nodes
    - Calling add_child_nodes in all open_parent_nodes definition
    - This conversion will be removed upon the final refactoring
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1380485

 app/controllers/application_controller/tree_support.rb   | 6 +++---
 app/controllers/catalog_controller.rb                    | 6 +-----
 app/controllers/miq_ae_class_controller.rb               | 4 ++--
 app/controllers/report_controller.rb                     | 3 +--
 app/controllers/vm_common.rb                             | 2 +-
 spec/controllers/application_controller/explorer_spec.rb | 7 ++++---
 6 files changed, 12 insertions(+), 16 deletions(-)

Comment 7 Niyaz Akhtar Ansari 2017-04-11 08:47:24 UTC
Verified in Version 5.8.0.9-alpha2.20170404195944_1d7ece4


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