From 8fd209c1e70d75ce9092d385ac61521a24a9e511 Mon Sep 17 00:00:00 2001
From: Michael Abashian <mabashian@ansible.com>
Date: Wed, 29 Jun 2016 15:11:26 -0400
Subject: [PATCH 1/2] Fixed bug where breadcrumb was not being shown when
 refreshing edit group/edit host pages

---
 .../inventories/manage/breadcrumbs/breadcrumbs.partial.html  | 1 -
 .../src/inventories/manage/groups/groups-list.controller.js  | 5 +++++
 .../src/inventories/manage/hosts/hosts-list.controller.js    | 5 +++++
 3 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.partial.html b/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.partial.html
index d40d9f6aba..aeebf7a8fa 100644
--- a/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.partial.html
+++ b/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.partial.html
@@ -17,7 +17,6 @@
 			<a href ng-click="goToGroup($index+1)">{{group.name}}</a>
 		</li>
 		<li class="BreadCrumb-item" ng-if="currentState !== 'inventoryManage'"></li>
-		</span>
         <div class="InventoryManageBreadCrumb-ncy" ng-if="!licenseMissing" ncy-breadcrumb></div>
 	</ol>
     <div class="BreadCrumb-menuLink"
diff --git a/awx/ui/client/src/inventories/manage/groups/groups-list.controller.js b/awx/ui/client/src/inventories/manage/groups/groups-list.controller.js
index 3b5bd9c318..9fe6d9809a 100644
--- a/awx/ui/client/src/inventories/manage/groups/groups-list.controller.js
+++ b/awx/ui/client/src/inventories/manage/groups/groups-list.controller.js
@@ -12,6 +12,11 @@
             view = generateList,
             pageSize = 20;
         $scope.inventory_id = $stateParams.inventory_id;
+        // The ncy breadcrumb directive will look at this attribute when attempting to bind to the correct scope.
+        // In this case, we don't want to incidentally bind to this scope when editing a host or a group.  See:
+        // https://github.com/ncuillery/angular-breadcrumb/issues/42 for a little more information on the
+        // problem that this solves.
+        $scope.ncyBreadcrumbIgnore = true;
         if($state.current.name === "inventoryManage.editGroup") {
             $scope.rowBeingEdited = $state.params.group_id;
             $scope.listBeingEdited = "groups";
diff --git a/awx/ui/client/src/inventories/manage/hosts/hosts-list.controller.js b/awx/ui/client/src/inventories/manage/hosts/hosts-list.controller.js
index 1e294a96bb..b88801bcd7 100644
--- a/awx/ui/client/src/inventories/manage/hosts/hosts-list.controller.js
+++ b/awx/ui/client/src/inventories/manage/hosts/hosts-list.controller.js
@@ -11,6 +11,11 @@
         var list = InventoryHosts,
             view = generateList,
             pageSize = 20;
+        // The ncy breadcrumb directive will look at this attribute when attempting to bind to the correct scope.
+        // In this case, we don't want to incidentally bind to this scope when editing a host or a group.  See:
+        // https://github.com/ncuillery/angular-breadcrumb/issues/42 for a little more information on the
+        // problem that this solves.
+        $scope.ncyBreadcrumbIgnore = true;
         if($state.current.name === "inventoryManage.editHost") {
             $scope.rowBeingEdited = $state.params.host_id;
             $scope.listBeingEdited = "hosts";

From 1d5cceda084c406a860a54287b52d5442f4022d2 Mon Sep 17 00:00:00 2001
From: Michael Abashian <mabashian@ansible.com>
Date: Thu, 30 Jun 2016 10:07:35 -0400
Subject: [PATCH 2/2] Tell the breadcrumb directive to ignore this scope as
 well

---
 .../inventories/manage/breadcrumbs/breadcrumbs.controller.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.controller.js b/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.controller.js
index fb6b2f321a..0ace8e9dec 100644
--- a/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.controller.js
+++ b/awx/ui/client/src/inventories/manage/breadcrumbs/breadcrumbs.controller.js
@@ -13,6 +13,11 @@ export default
         });
         $scope.inventory = inventoryData;
         $scope.currentState = $state.current.name;
+        // The ncy breadcrumb directive will look at this attribute when attempting to bind to the correct scope.
+        // In this case, we don't want to incidentally bind to this scope when editing a host or a group.  See:
+        // https://github.com/ncuillery/angular-breadcrumb/issues/42 for a little more information on the
+        // problem that this solves.
+        $scope.ncyBreadcrumbIgnore = true;
         // slices the group stack at $index to supply new group params to $state.go()
         $scope.goToGroup = function(index){
             var group = $stateParams.group.slice(0, index);