flake8 fixes

Author: vdboor

polymorphic_tree/admin/parentadmin.py

@@ -118,15 +118,17 @@ def get_action_icons(self, node):
  actions = []
  if node.can_have_children:
  actions.append(
- u'<a href="add/?{parent_attr}={id}" title="{title}" class="add-child-object"><img src="{static}polymorphic_tree/icons/page_new.gif" width="16" height="16" alt="{title}" /></a>'.format(
+ u'<a href="add/?{parent_attr}={id}" title="{title}" class="add-child-object">'
+ u'<img src="{static}polymorphic_tree/icons/page_new.gif" width="16" height="16" alt="{title}" /></a>'.format(
  parent_attr=self.model._mptt_meta.parent_attr, id=node.pk, title=_('Add sub node'), static=settings.STATIC_URL)
  )
  else:
  actions.append(self.EMPTY_ACTION_ICON.format(STATIC_URL=settings.STATIC_URL, css_class='add-child-object'))
 
  if self.can_preview_object(node):
  actions.append(
- u'<a href="{url}" title="{title}" target="_blank"><img src="{static}polymorphic_tree/icons/world.gif" width="16" height="16" alt="{title}" /></a>'.format(
+ u'<a href="{url}" title="{title}" target="_blank">'
+ u'<img src="{static}polymorphic_tree/icons/world.gif" width="16" height="16" alt="{title}" /></a>'.format(
  url=node.get_absolute_url(), title=_('View on site'), static=settings.STATIC_URL)
  )
 

polymorphic_tree/models.py

@@ -35,7 +35,6 @@ class PolymorphicMPTTModelBase(MPTTModelBase, PolymorphicModelBase):
  Metaclass for all polymorphic models.
  Needed to support both MPTT and Polymorphic metaclasses.
  """
- pass
 
 
 class PolymorphicTreeForeignKey(TreeForeignKey):
@@ -134,7 +133,8 @@ class Meta:
  ordering = ('tree_id', 'lft',)
 
  # Define:
- # parent = PolymorphicTreeForeignKey('self', blank=True, null=True, related_name='children', verbose_name=_('parent'), help_text=_('You can also change the parent by dragging the item in the list.'))
+ # parent = PolymorphicTreeForeignKey('self', blank=True, null=True, related_name='children', verbose_name=_('parent'),
+ # help_text=_('You can also change the parent by dragging the item in the list.'))
  # class MPTTMeta:
  # order_insertion_by = 'title'
 
@@ -203,7 +203,6 @@ def validate_move_to(self, new_parent):
  To deny move, this method have to be raised ``ValidationError`` or
  ``InvalidMove`` from ``mptt.exceptions``
  """
- pass
 
 
 def _get_new_parent(moved, target, position='first-child'):

polymorphic_tree/templatetags/stylable_admin_list.py

@@ -200,7 +200,7 @@ def _get_mptt_indent_field(cl, result):
  mptt_indent_field = None
  for field_name in cl.list_display:
  try:
- f = cl.lookup_opts.get_field(field_name)
+ cl.lookup_opts.get_field(field_name)
  except models.FieldDoesNotExist:
  if mptt_indent_field is None:
  attr = getattr(result, field_name, None)
@@ -244,7 +244,7 @@ def _get_non_field_repr(cl, result, field_name):
  if callable(field_name):
  attr = field_name
  value = attr(result)
- elif hasattr(cl.model_admin, field_name) and not field_name in ('__str__', '__unicode__'):
+ elif hasattr(cl.model_admin, field_name) and field_name not in ('__str__', '__unicode__'):
  attr = getattr(cl.model_admin, field_name)
  value = attr(result)
  else:

polymorphic_tree/tests/__init__.py

@@ -2,4 +2,4 @@
 
 # allow test runners in Django < 1.7 to find the tests
 if django.VERSION[:2] < (1, 7):
- from .test_models import *
+ from .test_models import * # noqa

setup.cfg

@@ -1,3 +1,7 @@
 [wheel]
 # create "py2.py3-none-any.whl" package
 universal = 1
+
+[flake8]
+exclude=polymorphic_tree/migrations/*.py, polymorphic_tree/tests/test_models.py
+max-line-length=160