diff options
author | 2020-12-15 08:28:22 +0300 | |
---|---|---|
committer | 2020-12-15 08:28:22 +0300 | |
commit | 7e98943c9d15b2f81f83fc6bfc75e9b3a57eb31c (patch) | |
tree | c7d7f209dd28abe1c2182e3fb55ea275fc628cb9 /backend/models/form.py | |
parent | fix various type annotation issues (diff) |
minor refactorings
Diffstat (limited to 'backend/models/form.py')
-rw-r--r-- | backend/models/form.py | 4 |
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 |