From 93af6116c2ffc55ea79074f4e67cc808f6602d09 Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Thu, 13 Dec 2018 15:55:57 -0500 Subject: [PATCH] Add conditional so workspace invitation is only revokable if the user is active --- atst/domain/workspaces/workspaces.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/atst/domain/workspaces/workspaces.py b/atst/domain/workspaces/workspaces.py index d8960a3d..bade9e08 100644 --- a/atst/domain/workspaces/workspaces.py +++ b/atst/domain/workspaces/workspaces.py @@ -146,7 +146,10 @@ class Workspaces(object): @classmethod def can_revoke_access_for(cls, workspace, workspace_role): - return workspace_role.user != workspace.owner + return ( + workspace_role.user != workspace.owner + and workspace_role.status == WorkspaceRoleStatus.ACTIVE + ) @classmethod def revoke_access(cls, user, workspace_id, workspace_role_id):