Issue #3755
allow_blank is not set to true for optional fields
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix
Sprint:
Quarter:
Description
By default allow_blank is set to False:
http://www.django-rest-framework.org/api-guide/fields/#charfield
Optional fields that do not have this set to True have a minlength field in the schema:
description: {
title: "Description",
description: "An optional description.",
type: "string",
minLength: 1
},
So when the field is not being sent in a request, the request fails validation.
Every CharField that does not have required=True should have allow_blank.
Related issues
Associated revisions
Revision bbd50620
View on GitHub
allow_blank for optional fields
Added allow_blank=True to all CharField serializers that are not required.
History
#1
Updated by CodeHeeler over 3 years ago
- Triaged changed from No to Yes
#2
Updated by amacdona@redhat.com over 3 years ago
During triage, I brought up the allow_null option, but we should use allow_blank, as it is preferred by DRF in the docs linked in the description.
#3
Updated by CodeHeeler over 3 years ago
- Status changed from NEW to POST
- Assignee set to CodeHeeler
#4
Updated by CodeHeeler over 3 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|bbd50620678a9f07e9a31e287661489baac8e78e.
#5
Updated by daviddavis over 3 years ago
- Related to Issue #3964: OpenAPI schema requires Progress Report suffix to be 1 char or longer added
#6
Updated by daviddavis almost 3 years ago
- Sprint/Milestone set to 3.0.0
#7
Updated by bmbouter almost 3 years ago
- Tags deleted (
Pulp 3)
#8
Updated by bmbouter about 2 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
allow_blank for optional fields
Added allow_blank=True to all CharField serializers that are not required.
fixes #3755 https://pulp.plan.io/issues/3755