Skip to content

Commit

Permalink
fixup merge
Browse files Browse the repository at this point in the history
  • Loading branch information
Chris Dodge committed Oct 1, 2015
1 parent d9a3d4b commit f4a9e51
Showing 1 changed file with 3 additions and 19 deletions.
22 changes: 3 additions & 19 deletions edx_proctoring/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,22 +5,6 @@
from edx_proctoring.models import ProctoredExam, ProctoredExamStudentAttempt, ProctoredExamStudentAllowance


class StrictBooleanField(serializers.BooleanField):
"""
Boolean field serializer to cater for a bug in DRF BooleanField serializer
where required=True is ignored.
"""
def from_native(self, value):
"""
Convert representations of a boolean to a Python `boolean`.
"""
if value in ('true', 't', 'True', '1'):
return True
if value in ('false', 'f', 'False', '0'):
return False
return None


class ProctoredExamSerializer(serializers.ModelSerializer):
"""
Serializer for the ProctoredExam Model.
Expand All @@ -32,9 +16,9 @@ class ProctoredExamSerializer(serializers.ModelSerializer):
exam_name = serializers.CharField(required=True)
time_limit_mins = serializers.IntegerField(required=True)

is_active = StrictBooleanField(required=True)
is_practice_exam = StrictBooleanField(required=True)
is_proctored = StrictBooleanField(required=True)
is_active = serializers.BooleanField(required=True)
is_practice_exam = serializers.BooleanField(required=True)
is_proctored = serializers.BooleanField(required=True)

class Meta:
"""
Expand Down

0 comments on commit f4a9e51

Please sign in to comment.