pk/atst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

environment role deletion method should also return a bool

Browse Source
This commit is contained in:
dandds 2018-12-12 11:57:54 -05:00
parent a75c19188f
commit 8754a74613
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
atst/domain/environment_roles.py
View File

@ -21,3 +21,6 @@ class EnvironmentRoles(object):
if existing_env_role: if existing_env_role:
db.session.delete(existing_env_role) db.session.delete(existing_env_role)
db.session.commit() db.session.commit()
return True
else:
return False

5
atst/domain/environments.py
View File

@ -72,7 +72,10 @@ class Environments(object):
environment = Environments.get(id_and_role["id"]) environment = Environments.get(id_and_role["id"])
if new_role is None: if new_role is None:
EnvironmentRoles.delete(workspace_role.user.id, environment.id) role_deleted = EnvironmentRoles.delete(
workspace_role.user.id, environment.id
)
if role_deleted:
updated = True updated = True
else: else:
env_role = EnvironmentRoles.get( env_role = EnvironmentRoles.get(