diff options
| author | Pablo Recio | 2013-05-18 12:40:25 +0200 |
|---|---|---|
| committer | Pablo Recio | 2013-05-18 12:40:25 +0200 |
| commit | de5cc8de423a22009d2a643f6c268805f715b212 (patch) | |
| tree | cddd0ff96f23cf6e5bc9f9c029bee20fd38e0813 /rest_framework/serializers.py | |
| parent | 34776da9249a5d73f822b3562bc56a5674b10ac7 (diff) | |
| download | django-rest-framework-de5cc8de423a22009d2a643f6c268805f715b212.tar.bz2 | |
A model's field is required if is null or blank
Diffstat (limited to 'rest_framework/serializers.py')
| -rw-r--r-- | rest_framework/serializers.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py index 7707de7a..500bb306 100644 --- a/rest_framework/serializers.py +++ b/rest_framework/serializers.py @@ -705,15 +705,14 @@ class ModelSerializer(Serializer): Creates a default instance of a basic non-relational field. """ kwargs = {} - has_default = model_field.has_default() - if model_field.null or model_field.blank or has_default: + if model_field.null or model_field.blank: kwargs['required'] = False if isinstance(model_field, models.AutoField) or not model_field.editable: kwargs['read_only'] = True - if has_default: + if model_field.has_default(): kwargs['default'] = model_field.get_default() if issubclass(model_field.__class__, models.TextField): |
