From fd691bc0c141a6f54ecb3fe3919bbd4645c69e1e Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Fri, 20 Sep 2019 10:50:23 -0400 Subject: [PATCH] Refactor Applications.remove_member --- atst/domain/applications.py | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/atst/domain/applications.py b/atst/domain/applications.py index 01cfd2b3..f13aefbd 100644 --- a/atst/domain/applications.py +++ b/atst/domain/applications.py @@ -2,7 +2,6 @@ from . import BaseDomainClass from flask import g from atst.database import db from atst.domain.application_roles import ApplicationRoles -from atst.domain.environment_roles import EnvironmentRoles from atst.domain.environments import Environments from atst.domain.exceptions import NotFoundError from atst.domain.invitations import ApplicationInvitations @@ -122,13 +121,4 @@ class Applications(BaseDomainClass): @classmethod def remove_member(cls, application_role): - application_role.status = ApplicationRoleStatus.DISABLED - application_role.deleted = True - - for env in application_role.application.environments: - EnvironmentRoles.delete( - application_role_id=application_role.id, environment_id=env.id - ) - - db.session.add(application_role) - db.session.commit() + ApplicationRoles.disable(application_role)