Bug 1740290

Summary: Breadcrumbs in Control are broken
Product: Red Hat CloudForms Management Engine Reporter: Angelina Vasileva <anikifor>
Component: UI - OPSAssignee: Richard Vsiansky <rvsiansk>
Status: CLOSED CURRENTRELEASE QA Contact: John Dupuy <jdupuy>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.11.0CC: bmidwood, hkataria, jdupuy, juwatts, lavenel, mpovolny, mshriver, obarenbo, simaishi
Target Milestone: GA   
Target Release: 5.11.0   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.11.0.21 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-12-13 14:57:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1740746    

Description Angelina Vasileva 2019-08-12 15:17:59 UTC
Description of problem:
Navigate to different submenus in Control and breadcrumb will always say the last breadcrumb from Control -> Explorer section

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

How reproducible:
Always

Steps to Reproduce:
1. Navigate to Control -> Logs
2. Navigate to Control -> Explorer -> Events -> All Events
3. Navigate again to Control -> Logs

Actual results:
1. Breadcrumb says "Control -> Explorer" on a fresh appliance
2. Breadcrumb: Control -> Explorer -> Events -> All Events
3. Breadcrumb: Control -> Explorer -> Events -> All Events

Expected results:
Breadcrumb says the actual location

Additional info:
Breadcrumb is correct only in Control -> Explorer section. 
Simulation, Import / Export and Log all are similar and just repeat the last visited location from Explorer section.

Comment 3 Richard Vsiansky 2019-08-13 14:50:21 UTC
https://github.com/ManageIQ/manageiq-ui-classic/pull/6004

Comment 4 Harpreet Kataria 2019-08-13 22:19:18 UTC
*** Bug 1740893 has been marked as a duplicate of this bug. ***

Comment 5 Harpreet Kataria 2019-08-13 22:20:04 UTC
*** Bug 1740890 has been marked as a duplicate of this bug. ***

Comment 6 Harpreet Kataria 2019-08-13 22:20:39 UTC
*** Bug 1740888 has been marked as a duplicate of this bug. ***

Comment 7 CFME Bot 2019-08-15 08:41:20 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/dee15d446ba7d3ed05e1db4ba96892fb032deadc
commit dee15d446ba7d3ed05e1db4ba96892fb032deadc
Author:     Richard Vsiansky <r.vsia>
AuthorDate: Tue Aug 13 10:36:26 2019 -0400
Commit:     Richard Vsiansky <r.vsia>
CommitDate: Tue Aug 13 10:36:26 2019 -0400

    Fix breadcrumbs in control section

    https://bugzilla.redhat.com/show_bug.cgi?id=1740290

    - also add a new option to breadcrumb mixin how to manually disable tree branch

 app/controllers/miq_policy_controller.rb | 9 +-
 app/controllers/mixins/breadcrumbs_mixin.rb | 3 +-
 spec/controllers/miq_policy_controller_spec.rb | 28 +
 spec/controllers/mixins/breadcrumbs_mixin_spec.rb | 21 +
 4 files changed, 59 insertions(+), 2 deletions(-)

Comment 8 CFME Bot 2019-08-16 20:37:14 UTC
New commit detected on ManageIQ/manageiq-ui-classic/ivanchuk:

https://github.com/ManageIQ/manageiq-ui-classic/commit/4963282b25f8a8fd5e0f015c6b1d56050533d292
commit 4963282b25f8a8fd5e0f015c6b1d56050533d292
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Aug 15 04:40:17 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Thu Aug 15 04:40:17 2019 -0400

    Merge pull request #6004 from rvsia/control-breadcrumbs

    Fix breadcrumbs in control section

    (cherry picked from commit 0d18879bf2ac90fdcbdda0155a9717f0ad7c265a)

    https://bugzilla.redhat.com/show_bug.cgi?id=1740290

 app/controllers/miq_policy_controller.rb | 9 +-
 app/controllers/mixins/breadcrumbs_mixin.rb | 3 +-
 spec/controllers/miq_policy_controller_spec.rb | 28 +
 spec/controllers/mixins/breadcrumbs_mixin_spec.rb | 21 +
 4 files changed, 59 insertions(+), 2 deletions(-)

Comment 9 John Dupuy 2019-08-26 12:53:16 UTC
Verified in CFME 5.11.0.21.20190821214526_b0060d0