Skip to content

Commit 7c0434c

Browse files
author
Youngjin Jo
authored
Merge pull request #347 from yjinjo/master
Add user_name and state when get the WorkspaceGroup
2 parents 6b586ae + bc9314d commit 7c0434c

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

src/spaceone/identity/service/workspace_group_service.py

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,11 +338,23 @@ def get(
338338
if role_type != "DOMAIN_ADMIN":
339339
raise ERROR_PERMISSION_DENIED()
340340

341+
workspace_group_id = params.workspace_group_id
342+
domain_id = params.domain_id
343+
341344
workspace_group_vo = self.workspace_group_mgr.get_workspace_group(
342-
params.workspace_group_id, params.domain_id
345+
workspace_group_id, domain_id
343346
)
344347

345-
return WorkspaceGroupResponse(**workspace_group_vo.to_dict())
348+
old_users, new_users = self.workspace_group_mgr.get_old_users_and_new_users(
349+
workspace_group_vo.users, workspace_group_id, domain_id
350+
)
351+
352+
workspace_group_user_ids: List[str] = old_users + new_users
353+
354+
workspace_group_dict = self.add_user_name_and_state_to_users(
355+
workspace_group_user_ids, workspace_group_vo, domain_id
356+
)
357+
return WorkspaceGroupResponse(**workspace_group_dict)
346358

347359
@transaction(permission="identity:WorkspaceGroup.read", role_types=["DOMAIN_ADMIN"])
348360
@append_query_filter(["workspace_group_id", "name", "domain_id"])

0 commit comments

Comments
 (0)