ag-admin/OpenSlides
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #4551 from normanjaeckel/MotionReasonRequired

Browse Source 
Fixed bug that motion reason is required if respective config value i…
This commit is contained in:
Emanuel Schütze 2019-04-02 11:08:44 +02:00 committed by GitHub
commit ff41aa8061
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
openslides/motions/serializers.py
View File

@ -472,10 +472,9 @@ class MotionSerializer(ModelSerializer):
else: else:
if "text" in data and not data["text"]: if "text" in data and not data["text"]:
raise ValidationError({"detail": "The text field may not be blank."}) raise ValidationError({"detail": "The text field may not be blank."})
if ( if config["motions_reason_required"]:
"reason" in data if (self.partial and "reason" in data and not data["reason"]) or (
and not data["reason"] not self.partial and not data.get("reason")
and config["motions_reason_required"]
): ):
raise ValidationError({"detail": "The reason field may not be blank."}) raise ValidationError({"detail": "The reason field may not be blank."})