pep8

Author: Apache

aldryn_categories/admin.py

@@ -18,15 +18,15 @@ class CategoryAdmin(TranslatableAdmin, TreeAdmin):
             'fields': (
                 'name',
                 'slug',
-            )
-        }),
+                )
+            }),
         (' ', {
             'fields': (
                 '_position',
                 '_ref_node_id',
-            )
-        }),
-    )
+                )
+            }),
+        )
 
     def get_form(self, request, obj=None, **kwargs):
         FormClass = super(CategoryAdmin, self).get_form(request, obj, **kwargs)

aldryn_categories/cms_wizards.py

@@ -37,6 +37,6 @@ class Meta:
     weight=290,
     form=movenodeform_factory(Category, form=CreateCategoryForm),
     description=_('Create a new category.')
-)
+    )
 
 wizard_pool.register(aldryn_category_wizard)

aldryn_categories/fields.py

@@ -8,7 +8,7 @@
     ManyToManyField,
     OneToOneField,
     CASCADE
-)
+    )
 from django.forms.models import ModelChoiceField, ModelMultipleChoiceField
 from django.utils.html import escape
 from django.utils.safestring import mark_safe

aldryn_categories/models.py

@@ -28,7 +28,7 @@ def get_queryset(self):
         return self.queryset_class(
             self.model,
             using=self._db
-        ).order_by('tree_id', 'lft')
+            ).order_by('tree_id', 'lft')
 
 
 #
@@ -55,17 +55,17 @@ class Category(TranslatedAutoSlugifyMixin, TranslationHelperMixin,
             blank=False,
             default='',
             max_length=255,
-        ),
+            ),
         slug=models.SlugField(
             _('slug'),
             blank=True,
             default='',
             help_text=_('Provide a “slug” or leave blank for an automatically '
                         'generated one.'),
             max_length=255,
-        ),
+            ),
         meta={'unique_together': (('language_code', 'slug', ), )}
-    )
+        )
 
     class Meta:
         verbose_name = _('category')

aldryn_categories/tests/test_fields.py

@@ -14,7 +14,7 @@
     CategoryModelChoiceField,
     CategoryMultipleChoiceField,
     CategoryOneToOneField,
-)
+    )
 
 from .base import CategoryTestCaseMixin
 
@@ -34,16 +34,16 @@ def test_category_model_choice_field(self):
         self.assertEqual(
             field.label_from_instance(child2),
             "  child2",
-        )
+            )
         self.assertEqual(
             field.label_from_instance(grandchild1),
             "    grandchild1",
-        )
+            )
         self.assertEqual(
             field.label_from_instance(bad_grandchild),
             '    bad grandchild<script>alert'
             '("bad stuff");</script>',
-        )
+            )
 
         # Tests that the field correctly throws an ImproperlyConfigured
         # exception if the given object is not a Category (or something that
@@ -73,16 +73,16 @@ def test_category_multiple_choice_field(self):
         self.assertEqual(
             field.label_from_instance(child2),
             "  child2",
-        )
+            )
         self.assertEqual(
             field.label_from_instance(grandchild1),
             "    grandchild1",
-        )
+            )
         self.assertEqual(
             field.label_from_instance(bad_grandchild),
             '    bad grandchild<script>alert'
             '("bad stuff");</script>',
-        )
+            )
 
         # Tests that the field correctly throws an ImproperlyConfigured
         # exception if the given object is not a Category (or something that

aldryn_categories/tests/test_migrations.py

@@ -17,7 +17,7 @@ def test_for_missing_migrations(self):
             'dry_run': True,
             'stdout': output,
             'check_changes': True,
-        }
+            }
 
         try:
             call_command('makemigrations', **options)

aldryn_categories/tests/test_models.py

@@ -153,7 +153,7 @@ def test_add_translations(self):
             ('en', "Cheese Omelette", "cheese-omelette"),
             ('de', "Käseomelett", "kaseomelett"),
             ('fr', "Omelette au Fromage", "omelette-au-fromage"),
-        ]
+            ]
 
         node = None
 

aldryn_newsblog/admin.py

@@ -6,7 +6,7 @@
 
 from cms.admin.placeholderadmin import (
     FrontendEditableAdminMixin, PlaceholderAdminMixin,
-)
+    )
 
 from aldryn_apphooks_config.admin import BaseAppHookConfig, ModelAppHookConfig
 from aldryn_people.models import Person
@@ -68,7 +68,7 @@ class Meta:
             'slug',
             'tags',
             'title',
-        ]
+            ]
 
     def __init__(self, *args, **kwargs):
         super(ArticleAdminForm, self).__init__(*args, **kwargs)
@@ -99,23 +99,23 @@ def __init__(self, *args, **kwargs):
 
 
 class ArticleAdmin(
-    AllTranslationsMixin,
-    PlaceholderAdminMixin,
-    FrontendEditableAdminMixin,
-    ModelAppHookConfig,
-    TranslatableAdmin
-):
+        AllTranslationsMixin,
+        PlaceholderAdminMixin,
+        FrontendEditableAdminMixin,
+        ModelAppHookConfig,
+        TranslatableAdmin
+        ):
     form = ArticleAdminForm
     list_display = ('title', 'app_config', 'slug', 'is_featured',
                     'is_published')
     list_filter = [
         'app_config',
         'categories',
-    ]
+        ]
     actions = (
         make_featured, make_not_featured,
         make_published, make_unpublished,
-    )
+        )
     fieldsets = (
         (None, {
             'fields': (
@@ -126,17 +126,17 @@ class ArticleAdmin(
                 'is_featured',
                 'featured_image',
                 'lead_in',
-            )
-        }),
+                )
+            }),
         (_('Meta Options'), {
             'classes': ('collapse',),
             'fields': (
                 'slug',
                 'meta_title',
                 'meta_description',
                 'meta_keywords',
-            )
-        }),
+                )
+            }),
         (_('Advanced Settings'), {
             'classes': ('collapse',),
             'fields': (
@@ -145,15 +145,15 @@ class ArticleAdmin(
                 'related',
                 'owner',
                 'app_config',
-            )
-        }),
-    )
+                )
+            }),
+        )
     filter_horizontal = [
         'categories',
-    ]
+        ]
     app_config_values = {
         'default_published': 'is_published'
-    }
+        }
     app_config_selection_title = ''
     app_config_selection_desc = ''
 
@@ -178,18 +178,18 @@ def has_add_permission(self, request, obj=None):
 
 
 class NewsBlogConfigAdmin(
-    AllTranslationsMixin,
-    PlaceholderAdminMixin,
-    BaseAppHookConfig,
-    TranslatableAdmin
-):
+        AllTranslationsMixin,
+        PlaceholderAdminMixin,
+        BaseAppHookConfig,
+        TranslatableAdmin
+        ):
     def get_config_fields(self):
         return (
             'app_title', 'permalink_type', 'non_permalink_handling',
             'template_prefix', 'paginate_by', 'pagination_pages_start',
             'pagination_pages_visible', 'exclude_featured',
             'create_authors', 'search_indexed', 'config.default_published',
-        )
+            )
 
 
 admin.site.register(models.NewsBlogConfig, NewsBlogConfigAdmin)

aldryn_newsblog/cms_appconfig.py

@@ -22,14 +22,14 @@
     ('yms', _('1969/07/the-eagle-has-landed/', )),
     ('ymds', _('1969/07/16/the-eagle-has-landed/', )),
     ('ymdi', _('1969/07/16/11/', )),
-)
+    )
 
 NON_PERMALINK_HANDLING = (
     (200, _('Allow')),
     (302, _('Redirect to permalink (default)')),
     (301, _('Permanent redirect to permalink')),
     (404, _('Return 404: Not Found')),
-)
+    )
 
 # TODO override default if support for Django 1.6 will be dropped
 TEMPLATE_PREFIX_CHOICES = getattr(
@@ -41,7 +41,7 @@ class NewsBlogConfig(TranslatableModel, AppHookConfig):
     """Adds some translatable, per-app-instance fields."""
     translations = TranslatedFields(
         app_title=models.CharField(_('name'), max_length=234),
-    )
+        )
 
     permalink_type = models.CharField(
         _('permalink type'), max_length=8,
@@ -60,21 +60,21 @@ class NewsBlogConfig(TranslatableModel, AppHookConfig):
         blank=False,
         default=5,
         help_text=_('When paginating list views, how many articles per page?'),
-    )
+        )
     pagination_pages_start = models.PositiveIntegerField(
         _('Pagination pages start'),
         blank=False,
         default=10,
         help_text=_('When paginating list views, after how many pages '
                     'should we start grouping the page numbers.'),
-    )
+        )
     pagination_pages_visible = models.PositiveIntegerField(
         _('Pagination pages visible'),
         blank=False,
         default=4,
         help_text=_('When grouping page numbers, this determines how many '
                     'pages are visible on each side of the active page.'),
-    )
+        )
     exclude_featured = models.PositiveSmallIntegerField(
         _('Excluded featured articles count'),
         blank=True,
@@ -84,7 +84,7 @@ class NewsBlogConfig(TranslatableModel, AppHookConfig):
             'view, you may prefer to exclude featured articles from the '
             'article list itself to avoid duplicates. To do this, enter the '
             'same number here as in your Featured Articles plugin.'),
-    )
+        )
     template_prefix = models.CharField(
         max_length=20,
         null=True, blank=True,
@@ -96,49 +96,49 @@ class NewsBlogConfig(TranslatableModel, AppHookConfig):
         _('Auto-create authors?'),
         default=True,
         help_text=_('Automatically create authors from logged-in user?'),
-    )
+        )
 
     # ALDRYN_NEWSBLOG_SEARCH
     search_indexed = models.BooleanField(
         _('Include in search index?'),
         default=True,
         help_text=_('Include articles in search indexes?'),
-    )
+        )
 
     placeholder_base_top = PlaceholderField(
         'newsblog_base_top',
         related_name='aldryn_newsblog_base_top',
-    )
+        )
 
     placeholder_base_sidebar = PlaceholderField(
         'newsblog_base_sidebar',
         related_name='aldryn_newsblog_base_sidebar',
-    )
+        )
 
     placeholder_list_top = PlaceholderField(
         'newsblog_list_top',
         related_name='aldryn_newsblog_list_top',
-    )
+        )
 
     placeholder_list_footer = PlaceholderField(
         'newsblog_list_footer',
         related_name='aldryn_newsblog_list_footer',
-    )
+        )
 
     placeholder_detail_top = PlaceholderField(
         'newsblog_detail_top',
         related_name='aldryn_newsblog_detail_top',
-    )
+        )
 
     placeholder_detail_bottom = PlaceholderField(
         'newsblog_detail_bottom',
         related_name='aldryn_newsblog_detail_bottom',
-    )
+        )
 
     placeholder_detail_footer = PlaceholderField(
         'newsblog_detail_footer',
         related_name='aldryn_newsblog_detail_footer',
-    )
+        )
 
     def get_app_title(self):
         return getattr(self, 'app_title', _('untitled'))

aldryn_newsblog/cms_plugins.py

@@ -25,7 +25,7 @@ def get_render_template(self, context, instance, placeholder):
             return add_prefix_to_path(
                 self.render_template,
                 instance.app_config.template_prefix
-            )
+                )
         return self.render_template
 
 
@@ -78,7 +78,7 @@ def render(self, context, instance, placeholder):
         context['dates'] = queryset.get_months(
             request,
             namespace=instance.app_config.namespace
-        )
+            )
         return context
 
 

aldryn_newsblog/cms_toolbars.py

@@ -13,7 +13,7 @@
 from aldryn_apphooks_config.utils import get_app_instance
 from aldryn_translation_tools.utils import (
     get_admin_url, get_object_from_request,
-)
+    )
 
 from .cms_appconfig import NewsBlogConfig
 from .models import Article