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

remove unneeded validation, django already does this for datetimerangefields

Browse source
This commit is contained in:
Thomas Steen Rasmussen 2017-01-29 10:51:31 +01:00
parent 16052601a8
commit 82e51b08e6
1 changed files with 1 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
camps/models.py
View file

@ -6,6 +6,7 @@ from django.contrib.postgres.fields import DateTimeRangeField
from psycopg2.extras import DateTimeTZRange from psycopg2.extras import DateTimeTZRange
from django.core.exceptions import ValidationError from django.core.exceptions import ValidationError
from datetime import timedelta from datetime import timedelta
from django.utils import timezone
class Camp(CreatedUpdatedModel, UUIDModel): class Camp(CreatedUpdatedModel, UUIDModel):
@ -48,18 +49,6 @@ class Camp(CreatedUpdatedModel, UUIDModel):
def clean(self): def clean(self):
''' Make sure the dates make sense - meaning no overlaps and buildup before camp before teardown ''' ''' Make sure the dates make sense - meaning no overlaps and buildup before camp before teardown '''
errors = [] errors = []
# sanity checking for buildup
if self.buildup.lower > self.buildup.upper:
errors.append(ValidationError({'buildup', 'Start of buildup must be before end of buildup'}))
# sanity checking for camp
if self.camp.lower > self.camp.upper:
errors.append(ValidationError({'camp', 'Start of camp must be before end of camp'}))
# sanity checking for teardown
if self.teardown.lower > self.teardown.upper:
errors.append(ValidationError({'teardown', 'Start of teardown must be before end of teardown'}))
# check for overlaps buildup vs. camp # check for overlaps buildup vs. camp
if self.buildup.upper > self.camp.lower: if self.buildup.upper > self.camp.lower:
msg = "End of buildup must not be after camp start" msg = "End of buildup must not be after camp start"