reynir/bornhack-website
1
0
Fork 0
Code Issues Pull requests 1 Releases Wiki Activity

fix bug in speaker<>camp relationship validation in signal handler

Browse source
This commit is contained in:
Thomas Steen Rasmussen 2017-07-09 16:48:02 +02:00
parent 034543595a
commit 4f0de99ea2
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/program/signal_handlers.py
View file

@ -1,13 +1,16 @@
from django.core.exceptions import ValidationError from django.core.exceptions import ValidationError
def check_speaker_event_camp_consistency(sender, instance, **kwargs):
def check_speaker_event_camp_consistency(sender, event, **kwargs):
if kwargs['action'] == 'pre_add': if kwargs['action'] == 'pre_add':
# loop over speakers being added to this event
for pk in kwargs['pk_set']: for pk in kwargs['pk_set']:
# check if this event belongs to a different event than the speaker does # check if this speaker belongs to a different event than the event does
from program.models import Event from program.models import Speaker
event = Event.objects.get(id=pk) speaker = Speaker.objects.get(id=pk)
if event.camp != instance.camp: if speaker.camp != event.camp:
raise ValidationError({'events': 'One or more events belong to a different camp (%s) than the speaker (%s) does' % (event.camp, instance.camp)}) raise ValidationError({'speakers': 'The speaker (%s) belongs to a different camp (%s) than the event does (%s)' % (speaker, speaker.camp, event.camp)})
def check_speaker_camp_change(sender, instance, **kwargs): def check_speaker_camp_change(sender, instance, **kwargs):
if instance.pk: if instance.pk: