local-it/mitgliederverwaltung
2
3
Fork 0
Code Issues 72 Pull requests 3 Projects 4 Releases Packages 1 Activity Actions

Fix: Update comment in auto_filter to reflect expr(false) usage

Browse source 
Update comment from 'id IN [] = never matches' to 'expr(false) = match none'
to match the actual implementation of deny_filter().
This commit is contained in:
Moritz 2026-01-08 23:34:04 +01:00
parent dd4b88f0b7
commit a42fc8a6eb
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/mv/authorization/checks/has_permission.ex
View file

@ -131,7 +131,7 @@ defmodule Mv.Authorization.Checks.HasPermission do
cond do cond do
is_nil(actor) -> is_nil(actor) ->
# No actor - deny access (fail-closed) # No actor - deny access (fail-closed)
# Return filter that never matches (id IN [] = never matches) # Return filter that never matches (expr(false) = match none)
deny_filter() deny_filter()
is_nil(action) -> is_nil(action) ->