Skip to content

Commit 7580916

Browse files
committed
Use defined rows instead of hybrid property for query filter
1 parent d33bb91 commit 7580916

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

brood/resources/actions.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ def acl_auth(
4141
.filter(models.ResourceHolderPermission.resource_id == resource_id)
4242
.filter(
4343
or_(
44-
models.ResourceHolderPermission.holder_id == user_id,
45-
models.ResourceHolderPermission.holder_id.in_(user_group_id_list),
44+
models.ResourceHolderPermission.user_id == user_id,
45+
models.ResourceHolderPermission.group_id.in_(user_group_id_list),
4646
)
4747
)
4848
.all()

brood/resources/api.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ async def update_resource_handler(
214214
- **drop_keys** (list): List of keys to drop
215215
"""
216216
ensure_resource_permission(
217-
db_session, current_user.id, resource_id, {data.ResourcePermissions.ADMIN},
217+
db_session, current_user.id, resource_id, {data.ResourcePermissions.UPDATE},
218218
)
219219
try:
220220
updated_resource = actions.update_resource_data(
@@ -247,7 +247,7 @@ async def delete_resource_handler(
247247
- **resource_id** (uuid): Resource ID
248248
"""
249249
ensure_resource_permission(
250-
db_session, current_user.id, resource_id, {data.ResourcePermissions.ADMIN},
250+
db_session, current_user.id, resource_id, {data.ResourcePermissions.DELETE},
251251
)
252252
try:
253253
resource = actions.delete_resource(db_session, resource_id)

0 commit comments

Comments
 (0)