fix: Regression from canView() refactoring

Either the query must be valid, and in case a key is provided, the entry must be readable.
viur-framework · Dec 31, 2024 · d0ce5f2 · d0ce5f2
1 parent e3e5a37
commit d0ce5f2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/viur/core/prototypes/list.py b/src/viur/core/prototypes/list.py
@@ -432,7 +432,7 @@ def canView(self, skel: SkeletonInstance) -> bool:
 
         query = self.listFilter(query)  # Access control
 
-        if query is None or not query.getEntry():
+        if query is None or (key and not query.getEntry()):
             return False
 
         return True

diff --git a/src/viur/core/prototypes/tree.py b/src/viur/core/prototypes/tree.py
@@ -735,7 +735,7 @@ def canView(self, skelType: SkelType, skel: SkeletonInstance) -> bool:
 
         query = self.listFilter(query)  # Access control
 
-        if query is None or not query.getEntry():
+        if query is None or (key and not query.getEntry()):
             return False
 
         return True