From 32cfe1b95419273f6c504fff5d49c00625ad14b4 Mon Sep 17 00:00:00 2001 From: Oliver Walters Date: Tue, 30 Mar 2021 08:25:51 +1100 Subject: [PATCH] Emit warning rather than raise error - All calling functions check for None anyway --- InvenTree/users/models.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/InvenTree/users/models.py b/InvenTree/users/models.py index 5c95abfe46..42028e91f0 100644 --- a/InvenTree/users/models.py +++ b/InvenTree/users/models.py @@ -12,6 +12,11 @@ from django.utils.translation import gettext_lazy as _ from django.dispatch import receiver from django.db.models.signals import post_save, post_delete +import logging + + +logger = logging.getLogger(__name__) + class RuleSet(models.Model): """ @@ -345,7 +350,7 @@ def update_group_roles(group, debug=False): content_type = ContentType.objects.get(app_label=app, model=model) permission = Permission.objects.get(content_type=content_type, codename=perm) except ContentType.DoesNotExist: - raise ValueError(f"Error: Could not find permission matching '{permission_string}'") + logger.warning(f"Error: Could not find permission matching '{permission_string}'") permission = None return permission