Skip to content
Snippets Groups Projects
Commit 3e919022 authored by Calum Mackervoy's avatar Calum Mackervoy Committed by Jean-Baptiste Pasquier
Browse files

bugfix: catching ObjectDoesNotExist in notifications listener

parent 32f4c8fa
No related branches found
No related tags found
No related merge requests found
......@@ -3,6 +3,7 @@ import logging
import requests
from django.conf import settings
from django.core.mail import send_mail
from django.core.exceptions import ObjectDoesNotExist
from django.db import models
from django.db.models.signals import post_save, post_delete
from django.dispatch import receiver
......@@ -123,11 +124,13 @@ def send_notification(sender, instance, **kwargs):
(not subscription.is_backlink or not kwargs.get("created")):
# I may have configured to send the subscription to a foreign key
if subscription.field is not None and len(subscription.field) > 1:
instance = getattr(instance, subscription.field, instance)
try:
instance = getattr(instance, subscription.field, instance)
url_resource = settings.BASE_URL + Model.resource_id(instance)
except NoReverseMatch:
continue
except ObjectDoesNotExist:
continue
process = Thread(target=send_request, args=[subscription.inbox, url_resource, instance,
kwargs.get("created", False)])
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment