I have registered my signal with the callback using the @receiver decorator
@receiver(post_save, sender=User, dispatch_uid='ARandomUniqueString')
def do_callback(sender, **kwargs):
I have put the from app.signals import * code in __init__.py and I can see that it gets imported twice and I do not think there is a good way to fix it, possibly happening due to installed apps in settings.py. I cannot understand why despite using dispatch_uid and the modelInstance.save being invoked only once, it still runs do_callback twice. Any suggestions?
Ok so I moved the import to views.py (or models.py and while it was getting imported only once, it was getting called twice.
The problem was that the post_save signal was getting called when the object was created as well as saved. I have no idea why so I added a workaround which now works
created = False
#Workaround to signal being emitted twice on create and save
if 'created' in kwargs:
if kwargs['created']:
created=True
#If signal is from object creation, return
if created:
return
Edit:
post_save was getting called twice because I used .create(...) which is equivalent to __init__(...) and .save().
Conclusion
dispatch_uid does work and doing single imports is still a good practice.
I had the same problem with post_save and also post_delete signals. It seems that the session object and LogEntry object were being saved as well creating multiple signals despite setting the dispatch_uid.
What worked for me was:
from django.contrib.admin.models import LogEntry
from django.contrib.sessions.models import Session
....
if sender in [LogEntry, Session]:
return
else:
# do your thing here
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With