Well that wasn't clever of me

PyRIGS/views.py

@@ -156,6 +156,6 @@ class GenericListView(generic.ListView):
         object_list = self.model.objects.filter(filter)
 
         orderBy = self.request.GET.get('orderBy', "name")
-        if orderBy is not "":
+        if orderBy != "":
             object_list = object_list.order_by(orderBy)
         return object_list

RIGS/rigboard.py

@@ -262,7 +262,7 @@ class EventArchive(generic.ListView):
 
         q = self.request.GET.get('q', "")
 
-        if q is not "":
+        if q != "":
             qfilter = Q(name__icontains=q) | Q(description__icontains=q) | Q(notes__icontains=q)
 
             # try and parse an int

RIGS/tests/regions.py

@@ -5,7 +5,7 @@ from selenium.webdriver.remote.webelement import WebElement
 from selenium.webdriver.support.ui import WebDriverWait
 from selenium.webdriver.support.select import Select
 import datetime
-from PyRIGS.regions import Modal
+from PyRIGS.tests.regions import Modal
 
 
 class Header(Region):
@@ -40,6 +40,7 @@ class ItemRow(Region):
     def subtotal(self):
         return self.find_element(*self._subtotal_locator).text
 
+
 class ItemModal(Modal):
     _header_selector = (By.TAG_NAME, 'h4')