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

Fixed bug that motion reason is required if respective config value is set.

Browse Source
This commit is contained in:
Norman Jäckel 2019-03-30 11:08:35 +01:00
parent 8f393a1877
commit 861dc1c02e
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."})