diff options
| author | Sergey Fedoseev <fedoseev.sergey@gmail.com> | 2017-07-29 19:16:43 +0500 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2017-07-29 10:16:43 -0400 |
| commit | 38988f289f7f5708f5ea85de2d5dfe0d86b23106 (patch) | |
| tree | 664f2c323b0137857e38b1fb402516487a4fbb90 /django/forms | |
| parent | b306c0c1a311ad42314423bbf70619579a94454b (diff) | |
Avoided creation of temporary sets.
Diffstat (limited to 'django/forms')
| -rw-r--r-- | django/forms/models.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/django/forms/models.py b/django/forms/models.py index 37ef43b767..ffa8982506 100644 --- a/django/forms/models.py +++ b/django/forms/models.py @@ -258,8 +258,8 @@ class ModelFormMetaclass(DeclarativeFieldsMetaclass): ) # make sure opts.fields doesn't specify an invalid field - none_model_fields = [k for k, v in fields.items() if not v] - missing_fields = set(none_model_fields) - set(new_class.declared_fields) + none_model_fields = {k for k, v in fields.items() if not v} + missing_fields = none_model_fields.difference(new_class.declared_fields) if missing_fields: message = 'Unknown field(s) (%s) specified for %s' message = message % (', '.join(missing_fields), @@ -682,8 +682,8 @@ class BaseModelFormSet(BaseFormSet): for form in valid_forms: exclude = form._get_validation_exclusions() unique_checks, date_checks = form.instance._get_unique_checks(exclude=exclude) - all_unique_checks = all_unique_checks.union(set(unique_checks)) - all_date_checks = all_date_checks.union(set(date_checks)) + all_unique_checks.update(unique_checks) + all_date_checks.update(date_checks) errors = [] # Do each of the unique checks (unique and unique_together) |