diff options
| author | Alasdair Nicol <alasdair@memset.com> | 2013-10-17 23:27:45 +0100 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2013-10-17 20:20:11 -0400 |
| commit | dfb4cb9970f86487f0aaa88c5dfcfafa31e4f430 (patch) | |
| tree | 4762099924f11873e9b6608e6f056939f16410b9 /django/contrib | |
| parent | d97bec5ee3a6284d30b613c9070588a60358e7ec (diff) | |
Fixed #21285 -- Fixed E121,E122 pep8 warnings
Diffstat (limited to 'django/contrib')
| -rw-r--r-- | django/contrib/admin/filters.py | 5 | ||||
| -rw-r--r-- | django/contrib/admin/options.py | 14 | ||||
| -rw-r--r-- | django/contrib/admin/views/main.py | 6 | ||||
| -rw-r--r-- | django/contrib/auth/tests/test_forms.py | 2 | ||||
| -rw-r--r-- | django/contrib/auth/tests/test_management.py | 2 | ||||
| -rw-r--r-- | django/contrib/formtools/tests/wizard/wizardtests/urls.py | 12 | ||||
| -rw-r--r-- | django/contrib/gis/admin/widgets.py | 4 | ||||
| -rw-r--r-- | django/contrib/gis/db/models/__init__.py | 6 | ||||
| -rw-r--r-- | django/contrib/gis/forms/widgets.py | 4 | ||||
| -rw-r--r-- | django/contrib/messages/storage/base.py | 2 | ||||
| -rw-r--r-- | django/contrib/messages/tests/base.py | 4 |
11 files changed, 30 insertions, 31 deletions
diff --git a/django/contrib/admin/filters.py b/django/contrib/admin/filters.py index 9c5fbf7d8c..5f9d69dee4 100644 --- a/django/contrib/admin/filters.py +++ b/django/contrib/admin/filters.py @@ -166,9 +166,8 @@ class RelatedFieldListFilter(FieldListFilter): rel_name = other_model._meta.pk.name self.lookup_kwarg = '%s__%s__exact' % (field_path, rel_name) self.lookup_kwarg_isnull = '%s__isnull' % field_path - self.lookup_val = request.GET.get(self.lookup_kwarg, None) - self.lookup_val_isnull = request.GET.get( - self.lookup_kwarg_isnull, None) + self.lookup_val = request.GET.get(self.lookup_kwarg) + self.lookup_val_isnull = request.GET.get(self.lookup_kwarg_isnull) self.lookup_choices = field.get_choices(include_blank=False) super(RelatedFieldListFilter, self).__init__( field, request, params, model, model_admin, field_path) diff --git a/django/contrib/admin/options.py b/django/contrib/admin/options.py index e27672773c..6d3195a868 100644 --- a/django/contrib/admin/options.py +++ b/django/contrib/admin/options.py @@ -1198,9 +1198,9 @@ class ModelAdmin(BaseModelAdmin): self.message_user(request, _( 'The %(name)s "%(obj)s" was deleted successfully.') % { - 'name': force_text(opts.verbose_name), - 'obj': force_text(obj_display)}, - messages.SUCCESS) + 'name': force_text(opts.verbose_name), + 'obj': force_text(obj_display) + }, messages.SUCCESS) if self.has_change_permission(request, None): post_url = reverse('admin:%s_%s_changelist' % @@ -1221,10 +1221,10 @@ class ModelAdmin(BaseModelAdmin): return TemplateResponse(request, self.delete_confirmation_template or [ - "admin/{}/{}/delete_confirmation.html".format(app_label, opts.model_name), - "admin/{}/delete_confirmation.html".format(app_label), - "admin/delete_confirmation.html" - ], context, current_app=self.admin_site.name) + "admin/{}/{}/delete_confirmation.html".format(app_label, opts.model_name), + "admin/{}/delete_confirmation.html".format(app_label), + "admin/delete_confirmation.html" + ], context, current_app=self.admin_site.name) @csrf_protect_m @transaction.atomic diff --git a/django/contrib/admin/views/main.py b/django/contrib/admin/views/main.py index dbe59f831b..04977eeef8 100644 --- a/django/contrib/admin/views/main.py +++ b/django/contrib/admin/views/main.py @@ -49,9 +49,9 @@ def _is_changelist_popup(request): IS_LEGACY_POPUP_VAR = 'pop' if IS_LEGACY_POPUP_VAR in request.GET: warnings.warn( - "The `%s` GET parameter has been renamed to `%s`." % - (IS_LEGACY_POPUP_VAR, IS_POPUP_VAR), - DeprecationWarning, 2) + "The `%s` GET parameter has been renamed to `%s`." % + (IS_LEGACY_POPUP_VAR, IS_POPUP_VAR), + DeprecationWarning, 2) return True return False diff --git a/django/contrib/auth/tests/test_forms.py b/django/contrib/auth/tests/test_forms.py index fa21d2f917..718e8fff18 100644 --- a/django/contrib/auth/tests/test_forms.py +++ b/django/contrib/auth/tests/test_forms.py @@ -106,7 +106,7 @@ class AuthenticationFormTest(TestCase): self.assertFalse(form.is_valid()) self.assertEqual(form.non_field_errors(), [force_text(form.error_messages['invalid_login'] % { - 'username': User._meta.get_field('username').verbose_name + 'username': User._meta.get_field('username').verbose_name })]) def test_inactive_user(self): diff --git a/django/contrib/auth/tests/test_management.py b/django/contrib/auth/tests/test_management.py index edf433c835..795b72b52e 100644 --- a/django/contrib/auth/tests/test_management.py +++ b/django/contrib/auth/tests/test_management.py @@ -217,7 +217,7 @@ class PermissionTestCase(TestCase): """ # check duplicated default permission models.Permission._meta.permissions = [ - ('change_permission', 'Can edit permission (duplicate)')] + ('change_permission', 'Can edit permission (duplicate)')] six.assertRaisesRegex(self, CommandError, "The permission codename 'change_permission' clashes with a " "builtin permission for model 'auth.Permission'.", diff --git a/django/contrib/formtools/tests/wizard/wizardtests/urls.py b/django/contrib/formtools/tests/wizard/wizardtests/urls.py index dabce5306a..d2e7c575cc 100644 --- a/django/contrib/formtools/tests/wizard/wizardtests/urls.py +++ b/django/contrib/formtools/tests/wizard/wizardtests/urls.py @@ -13,10 +13,10 @@ urlpatterns = patterns('', ('form2', Page2), ('form3', Page3), ('form4', Page4)])), - url(r'^wiz_other_template/$', CookieContactWizard.as_view( - [('form1', Page1), - ('form2', Page2), - ('form3', Page3), - ('form4', Page4)], - template_name='other_wizard_form.html')), + url(r'^wiz_other_template/$', CookieContactWizard.as_view( + [('form1', Page1), + ('form2', Page2), + ('form3', Page3), + ('form4', Page4)], + template_name='other_wizard_form.html')), ) diff --git a/django/contrib/gis/admin/widgets.py b/django/contrib/gis/admin/widgets.py index 0011223617..8c90195f8e 100644 --- a/django/contrib/gis/admin/widgets.py +++ b/django/contrib/gis/admin/widgets.py @@ -36,7 +36,7 @@ class OpenLayersWidget(Textarea): except (GEOSException, ValueError) as err: logger.error( "Error creating geometry from value '%s' (%s)" % ( - value, err) + value, err) ) value = None @@ -68,7 +68,7 @@ class OpenLayersWidget(Textarea): except OGRException as err: logger.error( "Error transforming geometry from srid '%s' to srid '%s' (%s)" % ( - value.srid, srid, err) + value.srid, srid, err) ) wkt = '' else: diff --git a/django/contrib/gis/db/models/__init__.py b/django/contrib/gis/db/models/__init__.py index e36aa36fc2..19e85b1364 100644 --- a/django/contrib/gis/db/models/__init__.py +++ b/django/contrib/gis/db/models/__init__.py @@ -9,6 +9,6 @@ from django.contrib.gis.db.models.manager import GeoManager # The geographic-enabled fields. from django.contrib.gis.db.models.fields import ( - GeometryField, PointField, LineStringField, PolygonField, - MultiPointField, MultiLineStringField, MultiPolygonField, - GeometryCollectionField) + GeometryField, PointField, LineStringField, PolygonField, + MultiPointField, MultiLineStringField, MultiPolygonField, + GeometryCollectionField) diff --git a/django/contrib/gis/forms/widgets.py b/django/contrib/gis/forms/widgets.py index e8322b950c..0b08624cc9 100644 --- a/django/contrib/gis/forms/widgets.py +++ b/django/contrib/gis/forms/widgets.py @@ -43,7 +43,7 @@ class BaseGeometryWidget(Widget): except (GEOSException, ValueError) as err: logger.error( "Error creating geometry from value '%s' (%s)" % ( - value, err) + value, err) ) return None @@ -63,7 +63,7 @@ class BaseGeometryWidget(Widget): except gdal.OGRException as err: logger.error( "Error transforming geometry from srid '%s' to srid '%s' (%s)" % ( - value.srid, self.map_srid, err) + value.srid, self.map_srid, err) ) context = self.build_attrs(attrs, diff --git a/django/contrib/messages/storage/base.py b/django/contrib/messages/storage/base.py index c021fbd2ae..f682a9bbc3 100644 --- a/django/contrib/messages/storage/base.py +++ b/django/contrib/messages/storage/base.py @@ -34,7 +34,7 @@ class Message(object): def __eq__(self, other): return isinstance(other, Message) and self.level == other.level and \ - self.message == other.message + self.message == other.message def __str__(self): return force_text(self.message) diff --git a/django/contrib/messages/tests/base.py b/django/contrib/messages/tests/base.py index b17883959f..5ed12a7e46 100644 --- a/django/contrib/messages/tests/base.py +++ b/django/contrib/messages/tests/base.py @@ -219,7 +219,7 @@ class BaseTests(object): lambda m:'MessageMiddleware' not in m, settings.MIDDLEWARE_CLASSES), TEMPLATE_CONTEXT_PROCESSORS=filter( lambda p:'context_processors.messages' not in p, - settings.TEMPLATE_CONTEXT_PROCESSORS), + settings.TEMPLATE_CONTEXT_PROCESSORS), MESSAGE_LEVEL=constants.DEBUG ) def test_middleware_disabled(self): @@ -244,7 +244,7 @@ class BaseTests(object): lambda m:'MessageMiddleware' not in m, settings.MIDDLEWARE_CLASSES), TEMPLATE_CONTEXT_PROCESSORS=filter( lambda p:'context_processors.messages' not in p, - settings.TEMPLATE_CONTEXT_PROCESSORS), + settings.TEMPLATE_CONTEXT_PROCESSORS), MESSAGE_LEVEL=constants.DEBUG ) def test_middleware_disabled_fail_silently(self): |
