Skip to content

Commit a37807f

Browse files
committed
fix: use correct role id when deleting roles
1 parent 4fbb732 commit a37807f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

libs/api/role/data-access/src/lib/api-role-resolver.service.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -210,8 +210,8 @@ export class ApiRoleResolverService {
210210
}
211211

212212
for (const role of toRevoke) {
213-
const deleted = await this.core.data.communityMemberRole.deleteMany({
214-
where: { member: { communityId, userId }, roleId: role.roleId },
213+
const deleted = await this.core.data.communityMemberRole.delete({
214+
where: { id: role.id },
215215
})
216216
const roleName = roles.find((r) => r.id === role.roleId)?.name
217217
await this.core.logInfo(`Role revoked: ${roleName}`, {
@@ -220,7 +220,7 @@ export class ApiRoleResolverService {
220220
relatedId: role.id,
221221
relatedType: 'Role',
222222
})
223-
result.revoked += deleted.count
223+
result.revoked += deleted ? 1 : 0
224224
}
225225
for (const role of toGrant) {
226226
const created = await this.core.data.communityMemberRole.create({

0 commit comments

Comments
 (0)