From 8f779f527634d884790319fa09b7498542d7397f Mon Sep 17 00:00:00 2001 From: msramalho <19508417+msramalho@users.noreply.github.com> Date: Tue, 14 Oct 2025 12:25:32 +0100 Subject: [PATCH] adjusts access for non-grouped entries --- app/tests/web/db/test_crud.py | 5 +++-- app/web/db/crud.py | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/app/tests/web/db/test_crud.py b/app/tests/web/db/test_crud.py index 7b569b6..c3ea851 100644 --- a/app/tests/web/db/test_crud.py +++ b/app/tests/web/db/test_crud.py @@ -109,6 +109,7 @@ def test_search_archives_by_url(test_data, db_session): ) == 16 ) + # true READ will work for all groups or lack of groups assert ( len( crud.search_archives_by_url( @@ -119,7 +120,7 @@ def test_search_archives_by_url(test_data, db_session): False, ) ) - == 16 + == 33 ) assert ( len( @@ -131,7 +132,7 @@ def test_search_archives_by_url(test_data, db_session): True, ) ) - == 16 + == 33 ) # Jerry's archives are public diff --git a/app/web/db/crud.py b/app/web/db/crud.py index a6743ff..e17393e 100644 --- a/app/web/db/crud.py +++ b/app/web/db/crud.py @@ -76,7 +76,7 @@ def search_archives_by_url( if read_public: or_filters.append(models.Archive.public.is_(true())) if read_groups is True: - or_filters.append(models.Archive.group_id.isnot(None)) + or_filters.append(true()) else: or_filters.append(models.Archive.group_id.in_(read_groups)) query = query.filter(or_(*or_filters))