Make ds.pemrission_allawed(..., default=) a keyword-only argument, refs #2262

This commit is contained in:
Simon Willison 2024-02-08 18:58:12 -08:00
commit bd9ed62e5d
2 changed files with 3 additions and 3 deletions

View file

@ -896,7 +896,7 @@ class Datasette:
await await_me_maybe(hook) await await_me_maybe(hook)
async def permission_allowed( async def permission_allowed(
self, actor, action, resource=None, default=DEFAULT_NOT_SET self, actor, action, resource=None, *, default=DEFAULT_NOT_SET
): ):
"""Check permissions using the permissions_allowed plugin hook""" """Check permissions using the permissions_allowed plugin hook"""
result = None result = None

View file

@ -444,10 +444,10 @@ class TableInsertView(BaseView):
# Must have insert-row AND upsert-row permissions # Must have insert-row AND upsert-row permissions
if not ( if not (
await self.ds.permission_allowed( await self.ds.permission_allowed(
request.actor, "insert-row", database_name, table_name request.actor, "insert-row", resource=(database_name, table_name)
) )
and await self.ds.permission_allowed( and await self.ds.permission_allowed(
request.actor, "update-row", database_name, table_name request.actor, "update-row", resource=(database_name, table_name)
) )
): ):
return _error( return _error(