Skip to content

Commit ab5a10d

Browse files
authored
16627 - fix check for team membership deactivation (bcgov#2652)
1 parent f7ee5cf commit ab5a10d

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

auth-api/src/auth_api/services/membership.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,12 +270,13 @@ def deactivate_membership(self, **kwargs):
270270
"""Mark this membership as inactive."""
271271
current_app.logger.debug('<deactivate_membership')
272272
user_from_context: UserContext = kwargs['user_context']
273+
273274
# if this is a member removing another member, check that they admin or owner
274275
if self._model.user.username != user_from_context.user_name:
275276
check_auth(org_id=self._model.org_id, one_of_roles=(COORDINATOR, ADMIN))
276277

277278
# check to ensure that owner isn't removed by anyone but an owner
278-
if self._model.membership_type == ADMIN:
279+
if self._model.membership_type_code == ADMIN:
279280
check_auth(org_id=self._model.org_id, one_of_roles=(ADMIN)) # pylint: disable=superfluous-parens
280281

281282
self._model.membership_status = MembershipStatusCodeModel.get_membership_status_by_code('INACTIVE')

auth-api/src/auth_api/version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,4 @@
2222
Development release segment: .devN
2323
"""
2424

25-
__version__ = '2.9.2' # pylint: disable=invalid-name
25+
__version__ = '2.9.3' # pylint: disable=invalid-name

0 commit comments

Comments
 (0)