From: Stephen Finucane <stephen.finucane@intel.com>
Date: Tue, 26 Jan 2016 22:13:43 +0000 (+0000)
Subject: pep8: Resolve some issues with lambda functions
X-Git-Tag: v1.1.0~56
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2bce1f35799675146a3a0ec2a9e98c049bed4830;p=thirdparty%2Fpatchwork.git

pep8: Resolve some issues with lambda functions

Signed-off-by: Stephen Finucane <stephen.finucane@intel.com>
---

diff --git a/patchwork/forms.py b/patchwork/forms.py
index 808ec8a3..628761bc 100644
--- a/patchwork/forms.py
+++ b/patchwork/forms.py
@@ -20,7 +20,6 @@
 from __future__ import absolute_import
 
 from django.contrib.auth.models import User
-from django.db.models.query_utils import Q
 from django import forms
 
 from patchwork.models import Patch, State, Bundle, UserProfile
diff --git a/patchwork/tests/browser.py b/patchwork/tests/browser.py
index 4444f9dc..38413d51 100644
--- a/patchwork/tests/browser.py
+++ b/patchwork/tests/browser.py
@@ -21,11 +21,10 @@ import errno
 import os
 import time
 
-import django
-if django.VERSION < (1, 7):
-    from django.test import LiveServerTestCase as StaticLiveServerTestCase
-else:
+try:
     from django.contrib.staticfiles.testing import StaticLiveServerTestCase
+except:  # Django < 1.7
+    from django.test import LiveServerTestCase as StaticLiveServerTestCase
 from selenium.common.exceptions import (
     NoSuchElementException, StaleElementReferenceException,
     TimeoutException)
@@ -139,22 +138,24 @@ class SeleniumTestCase(StaticLiveServerTestCase):
         return self.selenium.switch_to.active_element
 
     def wait_until_present(self, name):
-        is_present = lambda driver: driver.find_element_by_name(name)
+        def is_present(driver):
+            return driver.find_element_by_name(name)
         msg = "An element named '%s' should be on the page" % name
         element = Wait(self.selenium).until(is_present, msg)
         self.screenshot()
         return element
 
     def wait_until_visible(self, selector):
-        is_visible = lambda driver: self.find(selector).is_displayed()
+        def is_visible(driver):
+            return self.find(selector).is_displayed()
         msg = "The element matching '%s' should be visible" % selector
         Wait(self.selenium).until(is_visible, msg)
         self.screenshot()
         return self.find(selector)
 
     def wait_until_focused(self, selector):
-        is_focused = lambda driver: (
-            self.find(selector) == self.focused_element())
+        def is_focused(driver):
+            return self.find(selector) == self.focused_element()
         msg = "The element matching '%s' should be focused" % selector
         Wait(self.selenium).until(is_focused, msg)
         self.screenshot()
diff --git a/patchwork/views/user.py b/patchwork/views/user.py
index d6e57974..dd5eadae 100644
--- a/patchwork/views/user.py
+++ b/patchwork/views/user.py
@@ -148,8 +148,8 @@ def link(request):
                           [form.cleaned_data['email']])
             except Exception:
                 context['confirmation'] = None
-                context['error'] = 'An error occurred during confirmation. ' + \
-                                   'Please try again later'
+                context['error'] = ('An error occurred during confirmation. '
+                                    'Please try again later')
     else:
         form = UserPersonLinkForm()
     context['linkform'] = form