From 8a9ebe2086593afa41636cefefbd071f95140deb Mon Sep 17 00:00:00 2001
From: mabashian <mabashia@redhat.com>
Date: Wed, 4 Dec 2019 13:44:20 -0500
Subject: [PATCH] Fixes bug where permissions checkboxes had inverse effect
 after upgrading angular to 1.7.x.

---
 .../multi-select-list/select-list-item.directive.js   | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/awx/ui/client/src/shared/multi-select-list/select-list-item.directive.js b/awx/ui/client/src/shared/multi-select-list/select-list-item.directive.js
index 20b9c4e0a0..31d64431f0 100644
--- a/awx/ui/client/src/shared/multi-select-list/select-list-item.directive.js
+++ b/awx/ui/client/src/shared/multi-select-list/select-list-item.directive.js
@@ -31,20 +31,17 @@ export default
                 disabled: '='
             },
             require: '^multiSelectList',
-            template: '<input type="checkbox" data-multi-select-list-item ng-model="item.isSelected" ng-click="userInteractionSelect()" ng-disabled=disabled>',
+            template: '<input type="checkbox" data-multi-select-list-item ng-model="item.isSelected" ng-change="userInteractionSelect(item)" ng-disabled=disabled>',
             link: function(scope, element, attrs, multiSelectList) {
 
                 scope.decoratedItem = multiSelectList.registerItem(scope.item);
 
-                scope.$watch('item.isSelected', function(value) {
-                    if (value === true) {
+                scope.userInteractionSelect = function(item) {
+                    if (item.isSelected === true) {
                         multiSelectList.selectItem(scope.decoratedItem);
-                    } else if (value === false) {
+                    } else if (item.isSelected === false) {
                         multiSelectList.deselectItem(scope.decoratedItem);
                     }
-                });
-
-                scope.userInteractionSelect = function() {
                     scope.$emit("selectedOrDeselected", scope.decoratedItem);
                 };