Don't write reduntant document template permissions

qwc-services · Oct 24, 2024 · ad1a0a8 · ad1a0a8
1 parent 7cf9f05
commit ad1a0a8
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/config_generator/document_service_config.py b/src/config_generator/document_service_config.py
@@ -100,6 +100,8 @@ def permissions(self, role):
                     'document_templates': non_public_resources('document_templates', session)
                 }
 
+                is_public_role = (role == self.permissions_query.public_role())
+
                 # Collect available templates
                 available_document_templates = []
                 for root, dirs, files in os.walk(self.report_dir):
@@ -130,7 +132,10 @@ def permissions(self, role):
                     ):
                         continue
 
-                    permitted_templates.append(template)
+                    if is_public_role:
+                        permitted_templates.append(template)
+                    elif template_restricted_for_public:
+                        permitted_templates.append(template)
 
                 permissions['document_templates'] = permitted_templates