From d6f70a22f34c621c90a5f44ee6fd969467ef7e26 Mon Sep 17 00:00:00 2001 From: jaredevantabor Date: Wed, 8 Mar 2017 18:55:02 -0800 Subject: [PATCH 1/2] filtering out READ roles from the RBAC modal it was accidentally added in the 3.1 smart search snafoo --- .../src/access/add-rbac-resource/rbac-resource.controller.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/awx/ui/client/src/access/add-rbac-resource/rbac-resource.controller.js b/awx/ui/client/src/access/add-rbac-resource/rbac-resource.controller.js index 6e41696faa..1af53c230b 100644 --- a/awx/ui/client/src/access/add-rbac-resource/rbac-resource.controller.js +++ b/awx/ui/client/src/access/add-rbac-resource/rbac-resource.controller.js @@ -18,7 +18,9 @@ export default ['$rootScope', '$scope', 'GetBasePath', 'Rest', '$q', 'Wait', 'Pr // the object permissions are being added to scope.object = scope.resourceData.data; // array for all possible roles for the object - scope.roles = scope.object.summary_fields.object_roles; + scope.roles = _.omit(scope.object.summary_fields.object_roles, (key) => { + return key.name === 'Read'; + }); // TODO: get working with api // array w roles and descriptions for key From d8fc447309042f8f4694d6e27174ab549a67bede Mon Sep 17 00:00:00 2001 From: jaredevantabor Date: Wed, 8 Mar 2017 18:59:07 -0800 Subject: [PATCH 2/2] removing stale comment --- .../access/rbac-multiselect/rbac-multiselect-role.directive.js | 1 - 1 file changed, 1 deletion(-) diff --git a/awx/ui/client/src/access/rbac-multiselect/rbac-multiselect-role.directive.js b/awx/ui/client/src/access/rbac-multiselect/rbac-multiselect-role.directive.js index f5276dbc12..11f40109e7 100644 --- a/awx/ui/client/src/access/rbac-multiselect/rbac-multiselect-role.directive.js +++ b/awx/ui/client/src/access/rbac-multiselect/rbac-multiselect-role.directive.js @@ -16,7 +16,6 @@ export default roles: '=', model: '=' }, - // @issue why is the read-only role ommited from this selection? template: '', link: function(scope, element, attrs, ctrl) { CreateSelect2({