mirror of
https://github.com/lemeow125/StudE-Backend.git
synced 2024-11-17 06:19:24 +08:00
Improved student status serializer
This commit is contained in:
parent
f6f87415d5
commit
e734afd631
1 changed files with 5 additions and 1 deletions
|
@ -1,4 +1,4 @@
|
||||||
from rest_framework import serializers
|
from rest_framework import serializers, exceptions
|
||||||
from .models import StudentStatus
|
from .models import StudentStatus
|
||||||
from subjects.models import Subject
|
from subjects.models import Subject
|
||||||
from django.contrib.gis.geos import Point
|
from django.contrib.gis.geos import Point
|
||||||
|
@ -37,6 +37,10 @@ class StudentStatusSerializer(serializers.ModelSerializer):
|
||||||
validated_data['subject'] = None
|
validated_data['subject'] = None
|
||||||
validated_data['landmark'] = None
|
validated_data['landmark'] = None
|
||||||
else:
|
else:
|
||||||
|
if 'subject' not in validated_data:
|
||||||
|
raise serializers.ValidationError(
|
||||||
|
{'subject': 'This field may not be empty if active is true'})
|
||||||
|
# To-do: Add geofencing to ensure locations are always within USTP
|
||||||
# Check each landmark to see if our location is within it
|
# Check each landmark to see if our location is within it
|
||||||
for landmark in Landmark.objects.all():
|
for landmark in Landmark.objects.all():
|
||||||
if landmark.location.contains(validated_data['location']):
|
if landmark.location.contains(validated_data['location']):
|
||||||
|
|
Loading…
Reference in a new issue