Bug 1380485

Summary:	UI: Explorer trees issue with clicking on nodes that are not yet present in tree.
Product:	Red Hat CloudForms Management Engine	Reporter:	Harpreet Kataria <hkataria>
Component:	UI - OPS	Assignee:	Dávid Halász <dhalasz>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Niyaz Akhtar Ansari <nansari>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.7.0	CC:	dhalasz, hkataria, jhardy, mpovolny, nansari, obarenbo, simaishi
Target Milestone:	GA	Keywords:	TestOnly
Target Release:	5.8.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	5.8.0.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1383347 (view as bug list)		Environment:
Last Closed:	2017-06-12 16:09:17 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Bug Depends On:
Bug Blocks:	1383347

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 4 CFME Bot 2016-10-03 08:21:19 UTC

https://github.com/ManageIQ/manageiq/pull/11649

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