aboutsummaryrefslogtreecommitdiffstats
path: root/backend/models/form.py
diff options
context:
space:
mode:
authorGravatar decorator-factory <[email protected]>2020-12-15 08:28:22 +0300
committerGravatar decorator-factory <[email protected]>2020-12-15 08:28:22 +0300
commit7e98943c9d15b2f81f83fc6bfc75e9b3a57eb31c (patch)
treec7d7f209dd28abe1c2182e3fb55ea275fc628cb9 /backend/models/form.py
parentfix various type annotation issues (diff)
minor refactorings
Diffstat (limited to 'backend/models/form.py')
-rw-r--r--backend/models/form.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/backend/models/form.py b/backend/models/form.py
index 21cc549..cb58065 100644
--- a/backend/models/form.py
+++ b/backend/models/form.py
@@ -25,8 +25,8 @@ class Form(BaseModel):
"""Validates is all features in allowed list."""
# Uppercase everything to avoid mixed case in DB
value = [v.upper() for v in value]
- allowed_values = list(v.value for v in FormFeatures.__members__.values())
- if not all(v in allowed_values for v in value):
+ allowed_values = [v.value for v in FormFeatures.__members__.values()]
+ if any(v not in allowed_values for v in value):
raise ValueError("Form features list contains one or more invalid values.")
if FormFeatures.COLLECT_EMAIL in value and FormFeatures.REQUIRES_LOGIN not in value: # noqa