[requires.io] dependency update

2026-03-06 03:58:23 +00:00 · 2020-02-29 11:34:58 +00:00
73 changed files with 458 additions and 1571 deletions
--- a/.travis.yml
+++ b/.travis.yml
@@ -12,14 +12,14 @@ install:
  - export PATH=$PATH:$(pwd)
  - chmod +x chromedriver
  - pip install -r requirements.txt
-  - pip install coveralls codeclimate-test-reporter pycodestyle
+  - pip install coveralls codeclimate-test-reporter pep8

 before_script:
  - export PATH=$PATH:/usr/lib/chromium-browser/
  - python manage.py collectstatic --noinput

 script:
-  - pycodestyle . --exclude=migrations,importer*
+  - pep8 . --exclude=migrations,importer*
  - python manage.py check
  - python manage.py makemigrations --check --dry-run
  - coverage run manage.py test --verbosity=2
--- a/PyRIGS/settings.py
+++ b/PyRIGS/settings.py
@@ -50,6 +50,7 @@ if DEBUG:
    ADMINS.append(('Testing Superuser', 'superuser@example.com'))

 # Application definition
+
 INSTALLED_APPS = (
    'django.contrib.admin',
    'django.contrib.auth',
@@ -168,8 +169,6 @@ RECAPTCHA_PUBLIC_KEY = os.environ.get('RECAPTCHA_PUBLIC_KEY', "6LeIxAcTAAAAAJcZV
 RECAPTCHA_PRIVATE_KEY = os.environ.get('RECAPTCHA_PRIVATE_KEY', "6LeIxAcTAAAAAGG-vFI1TnRWxMZNFuojJ4WifJWe")  # If not set, use development key
 NOCAPTCHA = True

-SILENCED_SYSTEM_CHECKS = ['captcha.recaptcha_test_key_error']
-
 # Email
 EMAILER_TEST = False
 if not DEBUG or EMAILER_TEST:
--- a/PyRIGS/tests/base.py
+++ b/PyRIGS/tests/base.py
@@ -11,7 +11,7 @@ def create_browser():
    if os.environ.get('CI', False):
        options.add_argument("--headless")
        options.add_argument("--no-sandbox")
-    driver = webdriver.Chrome(options=options)
+    driver = webdriver.Chrome(chrome_options=options)
    return driver


--- a/PyRIGS/tests/pages.py
+++ b/PyRIGS/tests/pages.py
@@ -2,7 +2,6 @@ from pypom import Page, Region
 from selenium.webdriver.common.by import By
 from selenium.webdriver import Chrome
 from selenium.common.exceptions import NoSuchElementException
-from PyRIGS.tests import regions


 class BasePage(Page):
@@ -35,19 +34,37 @@ class FormPage(BasePage):
        self.driver.execute_script("Array.from(document.getElementsByTagName(\"input\")).forEach(function (el, ind, arr) { el.removeAttribute(\"required\")});")
        self.driver.execute_script("Array.from(document.getElementsByTagName(\"select\")).forEach(function (el, ind, arr) { el.removeAttribute(\"required\")});")

-    def submit(self):
-        previous_errors = self.errors
-        self.find_element(*self._submit_locator).click()
-        self.wait.until(lambda x: self.errors != previous_errors or self.success)
-
    @property
    def errors(self):
        try:
-            error_page = regions.ErrorPage(self, self.find_element(*self._errors_selector))
+            error_page = self.ErrorPage(self, self.find_element(*self._errors_selector))
            return error_page.errors
        except NoSuchElementException:
            return None

+    class ErrorPage(Region):
+        _error_item_selector = (By.CSS_SELECTOR, "dl>span")
+
+        class ErrorItem(Region):
+            _field_selector = (By.CSS_SELECTOR, "dt")
+            _error_selector = (By.CSS_SELECTOR, "dd>ul>li")
+
+            @property
+            def field_name(self):
+                return self.find_element(*self._field_selector).text
+
+            @property
+            def errors(self):
+                return [x.text for x in self.find_elements(*self._error_selector)]
+
+        @property
+        def errors(self):
+            error_items = [self.ErrorItem(self, x) for x in self.find_elements(*self._error_item_selector)]
+            errors = {}
+            for error in error_items:
+                errors[error.field_name] = error.errors
+            return errors
+

 class LoginPage(BasePage):
    URL_TEMPLATE = '/user/login'
--- a/PyRIGS/tests/regions.py
+++ b/PyRIGS/tests/regions.py
@@ -131,27 +131,3 @@ class SingleSelectPicker(Region):
    def set_value(self, value):
        picker = Select(self.root)
        picker.select_by_visible_text(value)
-
-
-class ErrorPage(Region):
-    _error_item_selector = (By.CSS_SELECTOR, "dl>span")
-
-    class ErrorItem(Region):
-        _field_selector = (By.CSS_SELECTOR, "dt")
-        _error_selector = (By.CSS_SELECTOR, "dd>ul>li")
-
-        @property
-        def field_name(self):
-            return self.find_element(*self._field_selector).text
-
-        @property
-        def errors(self):
-            return [x.text for x in self.find_elements(*self._error_selector)]
-
-    @property
-    def errors(self):
-        error_items = [self.ErrorItem(self, x) for x in self.find_elements(*self._error_item_selector)]
-        errors = {}
-        for error in error_items:
-            errors[error.field_name] = error.errors
-        return errors
--- a/PyRIGS/urls.py
+++ b/PyRIGS/urls.py
@@ -1,4 +1,3 @@
-from django.urls import path
 from django.conf.urls import include, url
 from django.contrib import admin
 from django.contrib.staticfiles.urls import staticfiles_urlpatterns
@@ -16,8 +15,8 @@ urlpatterns = [
    url('^assets/', include('assets.urls')),
    url('^user/register/$', RegistrationView.as_view(form_class=RIGS.forms.ProfileRegistrationFormUniqueEmail),
        name="registration_register"),
-    path('user/', include('django.contrib.auth.urls')),
-    path('user/', include('registration.backends.default.urls')),
+    url('^user/', include('django.contrib.auth.urls')),
+    url('^user/', include('registration.backends.default.urls')),

    url(r'^admin/', admin.site.urls),
 ]
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
 # TEC PA & Lighting - PyRIGS #
-[![Build Status](https://travis-ci.org/nottinghamtec/PyRIGS.svg)](https://travis-ci.org/nottinghamtec/PyRIGS)
-[![Coverage Status](https://coveralls.io/repos/github/nottinghamtec/PyRIGS/badge.svg)](https://coveralls.io/github/nottinghamtec/PyRIGS)
+[![Build Status](https://travis-ci.org/nottinghamtec/PyRIGS.svg?branch=develop)](https://travis-ci.org/nottinghamtec/PyRIGS)
+[![Coverage Status](https://coveralls.io/repos/github/nottinghamtec/PyRIGS/badge.svg?branch=develop)](https://coveralls.io/github/nottinghamtec/PyRIGS)

 Welcome to TEC PA & Lightings PyRIGS program. This is a reimplementation of the existing Rig Information Gathering System (RIGS) that was developed using Ruby on Rails.

--- a/RIGS/admin.py
+++ b/RIGS/admin.py
@@ -1,7 +1,7 @@
 from django.contrib import admin
 from RIGS import models, forms
 from django.contrib.auth.admin import UserAdmin
-from django.utils.translation import gettext_lazy as _
+from django.utils.translation import ugettext_lazy as _
 from reversion.admin import VersionAdmin

 from django.contrib.admin import helpers
--- a/RIGS/finance.py
+++ b/RIGS/finance.py
@@ -11,7 +11,6 @@ from django.template.loader import get_template
 from django.views import generic
 from django.db.models import Q
 from z3c.rml import rml2pdf
-from django.db.models import Q

 from RIGS import models

@@ -77,7 +76,7 @@ class InvoicePrint(generic.View):

        pdfData = buffer.read()

-        escapedEventName = re.sub(r'[^a-zA-Z0-9 \n\.]', '', object.name)
+        escapedEventName = re.sub('[^a-zA-Z0-9 \n\.]', '', object.name)

        response = HttpResponse(content_type='application/pdf')
        response['Content-Disposition'] = "filename=Invoice %05d - N%05d | %s.pdf" % (invoice.pk, invoice.event.pk, escapedEventName)
@@ -123,34 +122,6 @@ class InvoiceArchive(generic.ListView):
    template_name = 'RIGS/invoice_list_archive.html'
    paginate_by = 25

-    def get_queryset(self):
-        q = self.request.GET.get('q', "")
-
-        filter = Q(event__name__icontains=q)
-
-        # try and parse an int
-        try:
-            val = int(q)
-            filter = filter | Q(pk=val)
-            filter = filter | Q(event__pk=val)
-        except:  # noqa
-            # not an integer
-            pass
-
-        try:
-            if q[0] == "N":
-                val = int(q[1:])
-                filter = Q(event__pk=val)  # If string is Nxxxxx then filter by event number
-            elif q[0] == "#":
-                val = int(q[1:])
-                filter = Q(pk=val)  # If string is #xxxxx then filter by invoice number
-        except:  # noqa
-            pass
-
-        object_list = self.model.objects.filter(filter).order_by('-invoice_date')
-
-        return object_list
-

 class InvoiceWaiting(generic.ListView):
    model = models.Event
--- a/RIGS/migrations/0038_auto_20200306_2000.py
+++ b/RIGS/migrations/0038_auto_20200306_2000.py
@@ -1,37 +0,0 @@
-# Generated by Django 2.0.13 on 2020-03-06 20:00
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('RIGS', '0037_approve_legacy'),
-    ]
-
-    operations = [
-        migrations.AlterModelOptions(
-            name='event',
-            options={},
-        ),
-        migrations.AlterModelOptions(
-            name='invoice',
-            options={'ordering': ['-invoice_date']},
-        ),
-        migrations.AlterModelOptions(
-            name='organisation',
-            options={},
-        ),
-        migrations.AlterModelOptions(
-            name='person',
-            options={},
-        ),
-        migrations.AlterModelOptions(
-            name='profile',
-            options={'verbose_name': 'user', 'verbose_name_plural': 'users'},
-        ),
-        migrations.AlterModelOptions(
-            name='venue',
-            options={},
-        ),
-    ]
--- a/RIGS/models.py
+++ b/RIGS/models.py
@@ -8,6 +8,7 @@ from django.contrib.auth.models import AbstractUser
 from django.conf import settings
 from django.utils import timezone
 from django.utils.functional import cached_property
+from django.utils.encoding import python_2_unicode_compatible
 from reversion import revisions as reversion
 from reversion.models import Version
 import string
@@ -21,6 +22,7 @@ from django.urls import reverse_lazy


 # Create your models here.
+@python_2_unicode_compatible
 class Profile(AbstractUser):
    initials = models.CharField(max_length=5, unique=True, null=True, blank=False)
    phone = models.CharField(max_length=13, null=True, blank=True)
@@ -64,6 +66,11 @@ class Profile(AbstractUser):
    def __str__(self):
        return self.name

+    class Meta:
+        permissions = (
+            ('view_profile', 'Can view Profile'),
+        )
+

 class RevisionMixin(object):
    @property
@@ -94,6 +101,7 @@ class RevisionMixin(object):


@reversion.register
+@python_2_unicode_compatible
 class Person(models.Model, RevisionMixin):
    name = models.CharField(max_length=50)
    phone = models.CharField(max_length=15, blank=True, null=True)
@@ -129,8 +137,14 @@ class Person(models.Model, RevisionMixin):
    def get_absolute_url(self):
        return reverse_lazy('person_detail', kwargs={'pk': self.pk})

+    class Meta:
+        permissions = (
+            ('view_person', 'Can view Persons'),
+        )
+

@reversion.register
+@python_2_unicode_compatible
 class Organisation(models.Model, RevisionMixin):
    name = models.CharField(max_length=50)
    phone = models.CharField(max_length=15, blank=True, null=True)
@@ -167,6 +181,11 @@ class Organisation(models.Model, RevisionMixin):
    def get_absolute_url(self):
        return reverse_lazy('organisation_detail', kwargs={'pk': self.pk})

+    class Meta:
+        permissions = (
+            ('view_organisation', 'Can view Organisations'),
+        )
+

 class VatManager(models.Manager):
    def current_rate(self):
@@ -183,6 +202,7 @@ class VatManager(models.Manager):


@reversion.register
+@python_2_unicode_compatible
 class VatRate(models.Model, RevisionMixin):
    start_at = models.DateField()
    rate = models.DecimalField(max_digits=6, decimal_places=6)
@@ -203,6 +223,7 @@ class VatRate(models.Model, RevisionMixin):


@reversion.register
+@python_2_unicode_compatible
 class Venue(models.Model, RevisionMixin):
    name = models.CharField(max_length=255)
    phone = models.CharField(max_length=15, blank=True, null=True)
@@ -225,6 +246,11 @@ class Venue(models.Model, RevisionMixin):
    def get_absolute_url(self):
        return reverse_lazy('venue_detail', kwargs={'pk': self.pk})

+    class Meta:
+        permissions = (
+            ('view_venue', 'Can view Venues'),
+        )
+

 class EventManager(models.Manager):
    def current_events(self):
@@ -271,6 +297,7 @@ class EventManager(models.Manager):


@reversion.register(follow=['items'])
+@python_2_unicode_compatible
 class Event(models.Model, RevisionMixin):
    # Done to make it much nicer on the database
    PROVISIONAL = 0
@@ -464,6 +491,11 @@ class Event(models.Model, RevisionMixin):
        self.full_clean()
        super(Event, self).save(*args, **kwargs)

+    class Meta:
+        permissions = (
+            ('view_event', 'Can view Events'),
+        )
+

 class EventItem(models.Model):
    event = models.ForeignKey('Event', related_name='items', blank=True, on_delete=models.CASCADE)
@@ -501,7 +533,7 @@ class EventAuthorisation(models.Model, RevisionMixin):
    uni_id = models.CharField(max_length=10, blank=True, null=True, verbose_name="University ID")
    account_code = models.CharField(max_length=50, blank=True, null=True)
    amount = models.DecimalField(max_digits=10, decimal_places=2, verbose_name="authorisation amount")
-    sent_by = models.ForeignKey('Profile', on_delete=models.CASCADE)
+    sent_by = models.ForeignKey('RIGS.Profile', on_delete=models.CASCADE)

    def get_absolute_url(self):
        return reverse_lazy('event_detail', kwargs={'pk': self.event.pk})
@@ -511,6 +543,7 @@ class EventAuthorisation(models.Model, RevisionMixin):
        return str("N%05d" % self.event.pk + ' (requested by ' + self.sent_by.initials + ')')


+@python_2_unicode_compatible
 class Invoice(models.Model):
    event = models.OneToOneField('Event', on_delete=models.CASCADE)
    invoice_date = models.DateField(auto_now_add=True)
@@ -543,9 +576,13 @@ class Invoice(models.Model):
        return "%i: %s (%.2f)" % (self.pk, self.event, self.balance)

    class Meta:
+        permissions = (
+            ('view_invoice', 'Can view Invoices'),
+        )
        ordering = ['-invoice_date']


+@python_2_unicode_compatible
 class Payment(models.Model):
    CASH = 'C'
    INTERNAL = 'I'
--- a/RIGS/rigboard.py
+++ b/RIGS/rigboard.py
@@ -110,7 +110,7 @@ class EventCreate(generic.CreateView):
        context['currentVAT'] = models.VatRate.objects.current_rate()

        form = context['form']
-        if re.search(r'"-\d+"', form['items_json'].value()):
+        if re.search('"-\d+"', form['items_json'].value()):
            messages.info(self.request, "Your item changes have been saved. Please fix the errors and save the event.")

        # Get some other objects to include in the form. Used when there are errors but also nice and quick.
@@ -206,6 +206,7 @@ class EventPrint(generic.View):
        }

        rml = template.render(context)
+
        buffer = rml2pdf.parseString(rml)
        merger.append(PdfFileReader(buffer))
        buffer.close()
@@ -218,25 +219,17 @@ class EventPrint(generic.View):

        response = HttpResponse(content_type='application/pdf')

-        escapedEventName = re.sub(r'[^a-zA-Z0-9 \n\.]', '', object.name)
+        escapedEventName = re.sub('[^a-zA-Z0-9 \n\.]', '', object.name)

        response['Content-Disposition'] = "filename=N%05d | %s.pdf" % (object.pk, escapedEventName)
        response.write(merged.getvalue())
        return response


-class EventArchive(generic.ListView):
+class EventArchive(generic.ArchiveIndexView):
    model = models.Event
+    date_field = "start_date"
    paginate_by = 25
-    template_name = "RIGS/event_archive.html"
-
-    def get_context_data(self, **kwargs):
-        # get super context
-        context = super(EventArchive, self).get_context_data(**kwargs)
-
-        context['start'] = self.request.GET.get('start', None)
-        context['end'] = self.request.GET.get('end', datetime.date.today().strftime('%Y-%m-%d'))
-        return context

    def get_queryset(self):
        start = self.request.GET.get('start', None)
@@ -248,34 +241,19 @@ class EventArchive(generic.ListView):
                                 "Muppet! Check the dates, it has been fixed for you.")
            start, end = end, start  # Stop the impending fail

-        filter = Q()
+        filter = False
        if end != "":
-            filter &= Q(start_date__lte=end)
+            filter = Q(start_date__lte=end)
        if start:
-            filter &= Q(start_date__gte=start)
+            if filter:
+                filter = filter & Q(start_date__gte=start)
+            else:
+                filter = Q(start_date__gte=start)

-        q = self.request.GET.get('q', "")
-
-        if q is not "":
-            qfilter = Q(name__icontains=q) | Q(description__icontains=q) | Q(notes__icontains=q)
-
-            # try and parse an int
-            try:
-                val = int(q)
-                qfilter = qfilter | Q(pk=val)
-            except:  # noqa not an integer
-                pass
-
-            try:
-                if q[0] == "N":
-                    val = int(q[1:])
-                    qfilter = Q(pk=val)  # If string is N###### then do a simple PK filter
-            except:  # noqa
-                pass
-
-            filter &= qfilter
-
-        qs = self.model.objects.filter(filter).order_by('-start_date')
+        if filter:
+            qs = self.model.objects.filter(filter).order_by('-start_date')
+        else:
+            qs = self.model.objects.all().order_by('-start_date')

        # Preselect related for efficiency
        qs.select_related('person', 'organisation', 'venue', 'mic')
--- a/RIGS/signals.py
+++ b/RIGS/signals.py
@@ -73,7 +73,7 @@ def send_eventauthorisation_success_email(instance):
                     external_styles=css).transform()
    client_email.attach_alternative(html, 'text/html')

-    escapedEventName = re.sub(r'[^a-zA-Z0-9 \n\.]', '', instance.event.name)
+    escapedEventName = re.sub('[^a-zA-Z0-9 \n\.]', '', instance.event.name)

    client_email.attach('N%05d - %s - CONFIRMATION.pdf' % (instance.event.pk, escapedEventName),
                        merged.getvalue(),
--- a/RIGS/templates/RIGS/event_archive.html
+++ b/RIGS/templates/RIGS/event_archive.html
@@ -5,49 +5,34 @@

 {% block content %}
    <div class="row">
-        <div class="col-sm-12">
-            <h2>Event Archive</h2>
-        </div>
-        <div class="col-sm-12">
+        <h2>Event Archive</h2>
+
+        <div class="col-sm-12 col-md-6 pagination">
            <form class="form-inline">
-                    
-                <div class="input-group">
-                      <div class="input-group-addon">Start</div>
-                    <input type="date" name="start" id="start" value="{{ start|default_if_none:"" }}" placeholder="Start" class="form-control" />
+                <div class="form-group">
+                    <label for="start">Start</label>
+                    <input type="date" name="start" id="start" value="{{ request.GET.start }}" placeholder="Start" class="form-control" />
                </div>
-            
-                <div class="input-group">
-                      <div class="input-group-addon">End</div>
-                    <input type="date" name="end" id="end" value="{{ end|default_if_none:"" }}" placeholder="End" class="form-control" />
+                <div class="form-group">
+                    <label for="end">End</label>
+                    <input type="date" name="end" id="end" value="{% if request.GET.end %}{{ request.GET.end }}{% else %}{% now "Y-m-d" %}{% endif %}" placeholder="End" class="form-control" />
                </div>
-            
-                <div class="input-group">
-                    <div class="input-group-addon">Keyword</div>
-                    <input type="search" name="q" placeholder="Keyword" value="{{ request.GET.q }}"
-                           class="form-control"/>
+                <div class="form-group">
+                    <input type="submit" class="btn btn-primary" />
                </div>
-                        
-                <div class="input-group">
-                        <input type="submit" class="btn btn-primary" value="Search"/>
-                </div>
-                    
            </form>
        </div>

-    <div class="col-sm-12">
-    {% if is_paginated %}
-        <div class="pull-right">
-            {% paginator %}
-        </div>
-    {% endif %}  
-        
+        {% if is_paginated %}
+            <div class="col-md-6 text-right">
+                {% paginator %}
+            </div>
+        {% endif %}
    </div>
    <div class="row">
-        <div class="col-sm-12">
-            {% with object_list as events %}
-                {% include 'RIGS/event_table.html' %}
-            {% endwith %}
-        </div>
+        {% with latest as events %}
+            {% include 'RIGS/event_table.html' %}
+        {% endwith %}
    </div>

    {% if is_paginated %}
--- a/RIGS/templates/RIGS/event_embed.html
+++ b/RIGS/templates/RIGS/event_embed.html
@@ -1,7 +1,8 @@
 {% extends 'base_embed.html' %}
-{% load static %}
+{% load static from staticfiles %}

 {% block content %}
+
 <div class="row">
    <div class="col-sm-12">
        <a href="/">
@@ -71,6 +72,7 @@
                </p>
            </div>
            <div class="col-xs-6">
+
                {% if object.meet_at %}
                    <p>
                        <strong>Crew meet:</strong>
@@ -95,7 +97,10 @@
            {{ object.description|linebreaksbr }}
            </p>
        {% endif %}
+
        </table>
    </div>
 </div>
+
+
 {% endblock %}
--- a/RIGS/templates/RIGS/event_print_page.xml
+++ b/RIGS/templates/RIGS/event_print_page.xml
@@ -1,6 +1,7 @@
-{% load filters %}
 <setNextFrame name="main"/>
 <nextFrame/>
+
+
 <blockTable style="headLayout" colWidths="330,165">
 <tr>
    <td>
@@ -12,7 +13,7 @@

        <keepInFrame>
            <para style="style.event_description">
-            {{ object.description|default_if_none:""|linebreaksxml }}
+            {{ object.description|default_if_none:""|linebreaksbr }}
                </para>
        </keepInFrame>
    </td>
@@ -74,9 +75,9 @@
                {% if invoice %}
                    <keepInFrame>
                        {% if object.organisation.address %}
-                            <para style="specific_description">{{ object.organisation.address|default_if_none:""|linebreaksxml }}</para>
+                            <para style="specific_description">{{ object.organisation.address|default_if_none:""|linebreaksbr }}</para>
                        {% elif object.person.address %}
-                            <para style="specific_description">{{ object.person.address|default_if_none:""|linebreaksxml }}</para>
+                            <para style="specific_description">{{ object.person.address|default_if_none:""|linebreaksbr }}</para>
                        {% endif %}
                    </keepInFrame>
                {% endif %}
@@ -108,12 +109,12 @@
            <h3>{{ object.venue.name }}</h3>
            {% if not invoice %}
            <keepInFrame>
-            <para style="specific_description">{{ object.venue.address|default_if_none:""|linebreaksxml }}</para>
+            <para style="specific_description">{{ object.venue.address|default_if_none:""|linebreaksbr }}</para>
            </keepInFrame>
            {% endif %}
        </td>
        <td rightPadding="0">
-
+            
                <h2>Timings</h2>
                <blockTable style="eventDetails" colWidths="55,75">
                <tr>
@@ -184,7 +185,7 @@
                {% if item.description %}
                    </para>
                    <para style="item_description">
-                    <em>{{ item.description|linebreaksxml }}</em>
+                    <em>{{ item.description|linebreaksbr }}</em>
                    </para>
                    <para>
                {% endif %}
--- a/RIGS/templates/RIGS/index.html
+++ b/RIGS/templates/RIGS/index.html
@@ -1,14 +1,6 @@
 {% extends 'base_rigs.html' %}
 {% block title %}RIGS{% endblock %}

-{% block js %}
-<script>
-    $(function () {
-          $('[data-toggle="tooltip"]').tooltip();
-        })
-</script>
-{% endblock %}
-
 {% block content %}
    <div class="col-sm-12">
        <h1>R<small>ig</small> I<small>nformation</small> G<small>athering</small> S<small>ystem</small></h1>
@@ -44,41 +36,35 @@

        <div class="panel panel-default">
            <div class="panel-heading">
-              <h4 class="panel-title">Search Rigboard
-                <a href="{% url 'search_help' %}" class="pull-right modal-href"><span class="glyphicon glyphicon-question-sign"</span></a></h4>
+              <h4 class="panel-title">Search Rigboard</h4>
            </div>
-            <script>
-                $(document).ready(function(){
-                    $('#search-options li a').click(function(){
-                        $('#searchForm').attr('action', $(this).data('action')).submit();
-                    });
-                    $('#id_search_input').keypress(function (e) {
-                      if (e.which == 13) {
-                        $('#searchForm').attr('action', $('#search-options li a').first().data('action')).submit();
-                        return false;
-                      }
-                    });
-
-                });
-            </script>
            <div class="list-group">
                <div class="list-group-item">
-                    <form id="searchForm" class="form" role="form" method="GET">
-                        <div class="input-group" data-toggle="tooltip" title="Use the dropdown button to select what to search. The default is Event Archive.">
-                            <input id="id_search_input" type="search" name="q" class="form-control" placeholder="Search..." />
+                    <form class="form" role="form" action="{% url 'person_list' %}" method="GET">
+                        <div class="input-group">
+                            <input type="search" name="q" class="form-control" placeholder="Search People" />
                            <span class="input-group-btn">
-                                <div class="btn-group" role="group">
-                                    <button class="btn btn-default dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false"><span class="glyphicon glyphicon-search"></span> Search <span class="caret"></span></button>
-                                     <ul id="search-options" class="dropdown-menu">
-                                      <li><a data-action="{% url 'event_archive' %}" href="#">Events</a></li>
-                                      <li><a data-action="{% url 'person_list' %}" href="#">People</a></li>
-                                      <li><a data-action="{% url 'organisation_list' %}" href="#">Organisations</a></li>
-                                      <li><a data-action="{% url 'venue_list' %}" href="#">Venues</a></li>
-                                      {% if perms.RIGS.view_invoice %}
-                                      <li><a data-action="{% url 'invoice_archive' %}" href="#">Invoices</a></li>
-                                      {% endif %}
-                                    </ul>
-                                </div>
+                                <button type="submit" class="btn btn-default"><span class="glyphicon glyphicon-search"></span></button>
+                            </span>
+                        </div>
+                    </form>
+                </div>
+                <div class="list-group-item">
+                    <form class="form" role="form" action="{% url 'organisation_list' %}" method="GET">
+                        <div class="input-group">
+                            <input type="search" name="q" class="form-control" placeholder="Search Organisations" />
+                            <span class="input-group-btn">
+                                <button type="submit" class="btn btn-default"><span class="glyphicon glyphicon-search"></span></button>
+                            </span>
+                        </div>
+                    </form>
+                </div>
+                <div class="list-group-item">
+                    <form class="form" role="form" action="{% url 'venue_list' %}" method="GET">
+                        <div class="input-group">
+                            <input type="search" name="q" class="form-control" placeholder="Search Venues" />
+                            <span class="input-group-btn">
+                                <button type="submit" class="btn btn-default"><span class="glyphicon glyphicon-search"></span></button>
                            </span>
                        </div>
                    </form>
--- a/RIGS/templates/RIGS/invoice_detail.html
+++ b/RIGS/templates/RIGS/invoice_detail.html
@@ -111,7 +111,7 @@
                                {% endif %}
                            </dd>

-                            <dt>Authorisation request sent by</dt>
+                            <dt>Authorsation request sent by</dt>
                            <dd>{{ object.authorisation.sent_by }}</dd>
                        </dl>
                    </div>
--- a/RIGS/templates/RIGS/invoice_list.html
+++ b/RIGS/templates/RIGS/invoice_list.html
@@ -12,7 +12,6 @@
                {% paginator %}
            </div>
        {% endif %}
-        {% block search %}{% endblock %}
        <div class="table-responsive col-sm-12">
            <table class="table table-hover">
                <thead>
--- a/RIGS/templates/RIGS/invoice_list_archive.html
+++ b/RIGS/templates/RIGS/invoice_list_archive.html
@@ -10,15 +10,4 @@ All Invoices

 {% block description %}
 <p>This page displays all invoices: outstanding, paid, and void</p>
-{% endblock %}
-
-{% block search %}
-	<div class="col-sm-3 col-sm-offset-9">
-		<form class="form form-horizontal col-sm-12">
-		    <div class="form-group">
-		        <input type="search" name="q" placeholder="Search" value="{{ request.GET.q }}"
-		               class="form-control"/>
-		    </div>
-		</form>
-	</div>
 {% endblock %}
--- a/RIGS/templates/RIGS/item_modal.html
+++ b/RIGS/templates/RIGS/item_modal.html
@@ -1,4 +1,4 @@
-<div class="modal fade" id="itemModal" role="dialog" aria-labelledby="itemModal" aria-hidden="true">
+<div class="modal fade" id="itemModal" role="dialog" aria-labelledby="itemModal" aria-hidded="true">
    <div class="modal-dialog modal-lg">
        <div class="modal-content">
            <div class="modal-header">
@@ -33,6 +33,7 @@
                    </div>

                    <div class="row">
+
                        <div class="col-sm-6">
                            <div class="form-group">
                                <label for="item_quantity" class="col-sm-4 control-label">Quantity</label>
@@ -70,4 +71,4 @@
            </form>
        </div>
    </div>
-</div>
+</div>
--- a/RIGS/templates/RIGS/item_table.html
+++ b/RIGS/templates/RIGS/item_table.html
@@ -13,7 +13,7 @@
 				{% if edit %}
 				<td class="text-right">
 					<button type="button" class="btn btn-default btn-xs item-add"
-						data-toggle="modal"
+						data-url="{#% url eventitem_add object.pk %#}" data-toggle="modal"
 						data-target="#itemModal">
 						<span class="glyphicon glyphicon-plus"></span>
 					</button>
--- a/RIGS/templates/RIGS/password_reset_disable.html
+++ b/RIGS/templates/RIGS/password_reset_disable.html
@@ -0,0 +1,9 @@
+{% extends 'base_rigs.html' %}
+
+{% block title %}Password Reset Disabled{% endblock %}
+
+{% block content %}
+<h1>Password reset is disabled</h1>
+<p> We are very sorry for the inconvenience, but due to a security vulnerability, password reset is currently disabled until the vulnerability can be patched.</p>
+<p> If you are locked out of your account, please contact an administrator and we can manually perform a reset</p>
+{% endblock %}
--- a/RIGS/templates/RIGS/search_help.html
+++ b/RIGS/templates/RIGS/search_help.html
@@ -1,70 +0,0 @@
-{% extends request.is_ajax|yesno:"base_ajax.html,base.html" %}
-
-{% block title %}Search Help{% endblock %}
-
-{% block content %}
-    <div class="row">
-        {% if not request.is_ajax %}
-            <div class="col-sm-12">
-                <h1>Search Help</h1>
-            </div>
-        {% endif %}
-        <div class="col-sm-12">
-            <div class="panel panel-default">
-              <div class="panel-heading">
-                <h3 class="panel-title">Searching Events</h3>
-              </div>
-              <div class="panel-body">
-                <p>
-                Searches for entire query in:
-                    <button type="button" class="btn btn-default btn-xs">name</button>
-                    <button type="button" class="btn btn-default btn-xs">description</button> and 
-                    <button type="button" class="btn btn-default btn-xs">notes</button>
-                </p>
-                <p>You can search for an event by <button type="button" class="btn btn-default btn-xs">event_id</button> by entering an integer, or using the format <code>N01234</code></p>
-                <p>On the search results page you can also specify the date range for the <button type="button" class="btn btn-default btn-xs">start_date</button> of the event</p>
-                <p>Events are sorted in reverse <button type="button" class="btn btn-default btn-xs">start_date</button> order (most recent events at the top)</p>
-              </div>
-            </div>
-
-            <div class="panel panel-default">
-              <div class="panel-heading">
-                <h3 class="panel-title">Searching People/Organisations/Venues</h3>
-              </div>
-              <div class="panel-body">
-                <p>
-                Searches for entire search phrase in:
-                    <button type="button" class="btn btn-default btn-xs">name</button>
-                    <button type="button" class="btn btn-default btn-xs">email</button>
-                    <button type="button" class="btn btn-default btn-xs">address</button>
-                    <button type="button" class="btn btn-default btn-xs">notes</button> and 
-                    <button type="button" class="btn btn-default btn-xs">phone</button>
-                </p>
-                <p>You can search for an entry by <button type="button" class="btn btn-default btn-xs">id</button> by entering an integer</p>
-                <p>Entries are sorted in alphabetical order by <button type="button" class="btn btn-default btn-xs">name</button></p>
-              </div>
-            </div>
-
-            {% if perms.RIGS.view_invoice %}
-                <div class="panel panel-default">
-                  <div class="panel-heading">
-                    <h3 class="panel-title">Searching Invoices</h3>
-                  </div>
-                  <div class="panel-body">
-                    <p>
-                    Searches for entire search phrase in:
-                        <button type="button" class="btn btn-default btn-xs">event__name</button>
-                    </p>
-                    <p>You can search for an event's invoice by entering the <button type="button" class="btn btn-default btn-xs">event_id</button> using the format <code>N01234</code></p>
-                    <p>You can search for an invoice by <button type="button" class="btn btn-default btn-xs">invoice_id</button> using the format <code>#01234</code></p>
-                    <p>Entering a raw integer will search by both <button type="button" class="btn btn-default btn-xs">invoice_id</button> and <button type="button" class="btn btn-default btn-xs">event_id</button></p>
-                    <p>Entries are sorted in reverse <button type="button" class="btn btn-default btn-xs">invoice_date</button> order</p>
-                  </div>
-                </div>
-            {% endif %}
-
-            
-
-        </div>
-    </div>
-{% endblock %}
--- a/RIGS/templatetags/filters.py
+++ b/RIGS/templatetags/filters.py
@@ -2,24 +2,10 @@ from django import template
 from django import forms
 from django.forms.forms import NON_FIELD_ERRORS
 from django.forms.utils import ErrorDict
-from django.utils.text import normalize_newlines
-from django.template.defaultfilters import stringfilter
-from django.utils.safestring import SafeData, mark_safe
-from django.utils.html import escape

 register = template.Library()


-@register.filter(is_safe=True, needs_autoescape=True)
-@stringfilter
-def linebreaksxml(value, autoescape=True):
-    autoescape = autoescape and not isinstance(value, SafeData)
-    value = normalize_newlines(value)
-    if autoescape:
-        value = escape(value)
-    return mark_safe(value.replace('\n', '<br />'))
-
-
@register.filter
 def multiply(value, arg):
    return value * arg
--- a/RIGS/test_functional.py
+++ b/RIGS/test_functional.py
@@ -94,8 +94,7 @@ class UserRegistrationTest(LiveServerTestCase):
        # Read what the error is
        alert = self.browser.find_element_by_css_selector(
            'div.alert-danger').text
-        # TODO Use regex matching to handle smart/unsmart quotes...
-        self.assertIn("password fields didn", alert)
+        self.assertIn("password fields didn't match", alert)

        # Passwords should be empty
        self.assertEqual(password1.get_attribute('value'), '')
@@ -122,7 +121,7 @@ class UserRegistrationTest(LiveServerTestCase):
        email = mail.outbox[0]
        self.assertIn('John Smith "JS" activation required', email.subject)
        urls = re.findall(
-            r'http[s]?://(?:[a-zA-Z]|[0-9]|[$-_@.&+]|[!*\(\),]|(?:%[0-9a-fA-F][0-9a-fA-F]))+', email.body)
+            'http[s]?://(?:[a-zA-Z]|[0-9]|[$-_@.&+]|[!*\(\),]|(?:%[0-9a-fA-F][0-9a-fA-F]))+', email.body)
        self.assertEqual(len(urls), 1)

        mail.outbox = []  # empty this for later
@@ -255,7 +254,7 @@ class EventTest(LiveServerTestCase):

        # Slider expands and save button visible
        self.assertTrue(save.is_displayed())
-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')

        # For now, just check that HTML5 Client validation is in place TODO Test needs rewriting to properly test all levels of validation.
        self.assertTrue(self.browser.find_element_by_id('id_name').get_attribute('required') is not None)
@@ -493,7 +492,7 @@ class EventTest(LiveServerTestCase):

        save = self.browser.find_element_by_xpath(
            '(//button[@type="submit"])[3]')
-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')

        # Check the items are visible
        table = self.browser.find_element_by_id('item-table')  # ID number is known, see above
@@ -505,10 +504,8 @@ class EventTest(LiveServerTestCase):

        # Add item
        form.find_element_by_xpath('//button[contains(@class, "item-add")]').click()
-        modal = self.browser.find_element_by_id("itemModal")
        wait.until(animation_is_finished())
-        # See modal has opened
-        self.assertTrue(modal.is_displayed())
+        modal = self.browser.find_element_by_id("itemModal")
        modal.find_element_by_id("item_name").send_keys("Test Item 3")
        modal.find_element_by_id("item_description").send_keys(
            "This is an item description\nthat for reasons unknown spans two lines")
@@ -577,7 +574,7 @@ class EventTest(LiveServerTestCase):
        # Click Rig button
        self.browser.find_element_by_xpath('//button[.="Rig"]').click()

-        form = self.browser.find_element_by_xpath('//*[@id="content"]/form')
+        form = self.browser.find_element_by_tag_name('form')
        save = self.browser.find_element_by_xpath('(//button[@type="submit"])[3]')

        # Set title
@@ -607,7 +604,7 @@ class EventTest(LiveServerTestCase):
        self.assertIn("can't finish before it has started", error.find_element_by_xpath('//dd[1]/ul/li').text)

        # Same date, end time before start time
-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')
        save = self.browser.find_element_by_xpath('(//button[@type="submit"])[3]')

        self.browser.execute_script("document.getElementById('id_start_date').value='3015-04-24'")
@@ -626,7 +623,7 @@ class EventTest(LiveServerTestCase):
        self.assertIn("can't finish before it has started", error.find_element_by_xpath('//dd[1]/ul/li').text)

        # Same date, end time before start time
-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')
        save = self.browser.find_element_by_xpath('(//button[@type="submit"])[3]')

        self.browser.execute_script("document.getElementById('id_start_date').value='3015-04-24'")
@@ -639,7 +636,7 @@ class EventTest(LiveServerTestCase):
        form.find_element_by_id('id_end_time').send_keys('06:00')

        # No end date, end time before start time
-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')
        save = self.browser.find_element_by_xpath('(//button[@type="submit"])[3]')

        self.browser.execute_script("document.getElementById('id_start_date').value='3015-04-24'")
@@ -658,7 +655,7 @@ class EventTest(LiveServerTestCase):
        self.assertIn("can't finish before it has started", error.find_element_by_xpath('//dd[1]/ul/li').text)

        # 2 dates, end after start
-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')
        save = self.browser.find_element_by_xpath('(//button[@type="submit"])[3]')
        self.browser.execute_script("document.getElementById('id_start_date').value='3015-04-24'")
        self.browser.execute_script("document.getElementById('id_end_date').value='3015-04-26'")
@@ -691,7 +688,7 @@ class EventTest(LiveServerTestCase):
        # Click Rig button
        self.browser.find_element_by_xpath('//button[.="Rig"]').click()

-        form = self.browser.find_element_by_xpath('/html/body/div[2]/div[1]/form')
+        form = self.browser.find_element_by_tag_name('form')
        save = self.browser.find_element_by_xpath('(//button[@type="submit"])[3]')

        # Set title
@@ -1225,47 +1222,3 @@ class TECEventAuthorisationTest(TestCase):
        self.assertEqual(self.event.auth_request_by, self.profile)
        self.assertEqual(self.event.auth_request_to, 'client@functional.test')
        self.assertIsNotNone(self.event.auth_request_at)
-
-
-class SearchTest(LiveServerTestCase):
-    def setUp(self):
-        self.profile = models.Profile(
-            username="SearchTest", first_name="Search", last_name="Test", initials="STU", is_superuser=True)
-        self.profile.set_password("SearchTestPassword")
-        self.profile.save()
-
-        self.vatrate = models.VatRate.objects.create(start_at='2014-03-05', rate=0.20, comment='test1')
-
-        self.browser = create_browser()
-        self.browser.implicitly_wait(10)  # Set implicit wait session wide
-
-        os.environ['RECAPTCHA_TESTING'] = 'True'
-
-        models.Event.objects.create(name="Right Event", status=models.Event.PROVISIONAL,
-                                    start_date=date.today(), description="This event is searched for endlessly over and over")
-        models.Event.objects.create(name="Wrong Event", status=models.Event.PROVISIONAL, start_date=date.today(),
-                                    description="This one should never be found.")
-
-    def tearDown(self):
-        self.browser.quit()
-        os.environ['RECAPTCHA_TESTING'] = 'False'
-
-    def test_search(self):
-        self.browser.get(self.live_server_url)
-        username = self.browser.find_element_by_id('id_username')
-        password = self.browser.find_element_by_id('id_password')
-        submit = self.browser.find_element_by_css_selector(
-            'input[type=submit]')
-
-        username.send_keys("SearchTest")
-        password.send_keys("SearchTestPassword")
-        submit.click()
-
-        form = self.browser.find_element_by_id('searchForm')
-        search_box = form.find_element_by_id('id_search_input')
-        search_box.send_keys('Right')
-        search_box.send_keys(Keys.ENTER)
-
-        event_name = self.browser.find_element_by_xpath('//*[@id="content"]/div[1]/div[4]/div/div/table/tbody/tr[1]/td[3]/h4').text
-        self.assertIn('Right', event_name)
-        self.assertNotIn('Wrong', event_name)
--- a/RIGS/test_models.py
+++ b/RIGS/test_models.py
@@ -424,7 +424,7 @@ class RIGSVersionTestCase(TestCase):

    def test_find_parent_version(self):
        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')
        self.assertEqual(currentVersion._object_version.object.notes, "A new note on the event")

        # Check the prev version is loaded correctly
@@ -436,7 +436,7 @@ class RIGSVersionTestCase(TestCase):

    def test_changes_since(self):
        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')

        changes = currentVersion.changes
        self.assertEqual(len(changes.field_changes), 1)
@@ -453,7 +453,7 @@ class RIGSVersionTestCase(TestCase):
            self.event.save()

        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')
        diff = currentVersion.changes

        # There are two changes
@@ -475,7 +475,7 @@ class RIGSVersionTestCase(TestCase):
            self.person.save()

        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.person).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.person).latest(field_name='revision__date_created')
        diff = currentVersion.changes

        # Should be declared as long
@@ -488,7 +488,7 @@ class RIGSVersionTestCase(TestCase):
            self.event.save()

        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')

        # Check the diff is correct
        self.assertEqual(currentVersion.changes.field_changes[0].diff,
@@ -504,12 +504,12 @@ class RIGSVersionTestCase(TestCase):
            self.event.status = models.Event.CONFIRMED
            self.event.save()

-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')
        self.assertEqual(currentVersion.changes.field_changes[0].old, 'Provisional')
        self.assertEqual(currentVersion.changes.field_changes[0].new, 'Confirmed')

    def test_creation_behaviour(self):
-        firstVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created').parent
+        firstVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created').parent
        diff = firstVersion.changes

        # Mainly to check for exceptions:
@@ -522,7 +522,7 @@ class RIGSVersionTestCase(TestCase):
            self.event.save()

        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')

        diffs = currentVersion.changes.item_changes

@@ -541,7 +541,7 @@ class RIGSVersionTestCase(TestCase):
            item1.save()
            self.event.save()

-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')

        diffs = currentVersion.changes.item_changes

@@ -563,7 +563,7 @@ class RIGSVersionTestCase(TestCase):
            self.event.save()

        # Find the most recent version
-        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest('revision__date_created')
+        currentVersion = versioning.RIGSVersion.objects.get_for_object(self.event).latest(field_name='revision__date_created')

        diffs = currentVersion.changes.item_changes

--- a/RIGS/test_unit.py
+++ b/RIGS/test_unit.py
@@ -226,7 +226,7 @@ class TestPrintPaperwork(TestCase):
        cls.vatrate = models.VatRate.objects.create(start_at='2014-03-05', rate=0.20, comment='test1')

        cls.events = {
-            1: models.Event.objects.create(name="TE E1", start_date=date.today(), description="This is an event description\nthat for a very specific reason spans two lines."),
+            1: models.Event.objects.create(name="TE E1", start_date=date.today()),
        }

        cls.invoices = {
@@ -423,107 +423,3 @@ class TestSampleDataGenerator(TestCase):
        from django.core.management.base import CommandError

        self.assertRaisesRegex(CommandError, ".*production", call_command, 'generateSampleRIGSData')
-
-
-class TestSearchLogic(TestCase):
-    @classmethod
-    def setUpTestData(cls):
-        cls.profile = models.Profile.objects.create(username="testuser1", email="1@test.com", is_superuser=True,
-                                                    is_active=True, is_staff=True)
-
-        cls.persons = {
-            1: models.Person.objects.create(name="Right Person", phone="1234"),
-            2: models.Person.objects.create(name="Wrong Person", phone="5678"),
-        }
-
-        cls.organisations = {
-            1: models.Organisation.objects.create(name="Right Organisation", email="test@example.com"),
-            2: models.Organisation.objects.create(name="Wrong Organisation", email="check@fake.co.uk"),
-        }
-
-        cls.venues = {
-            1: models.Venue.objects.create(name="Right Venue", address="1 Test Street, EX1"),
-            2: models.Venue.objects.create(name="Wrong Venue", address="2 Check Way, TS2"),
-        }
-
-        cls.events = {
-            1: models.Event.objects.create(name="Right Event", start_date=date.today(), person=cls.persons[1],
-                                           organisation=cls.organisations[1], venue=cls.venues[1]),
-            2: models.Event.objects.create(name="Wrong Event", start_date=date.today(), person=cls.persons[2],
-                                           organisation=cls.organisations[2], venue=cls.venues[2]),
-        }
-
-    def setUp(self):
-        self.profile.set_password('testuser')
-        self.profile.save()
-        self.assertTrue(self.client.login(username=self.profile.username, password='testuser'))
-
-    def test_event_search(self):
-        # Test search by name
-        request_url = "%s?q=%s" % (reverse('event_archive'), self.events[1].name)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.events[1].name)
-        self.assertNotContains(response, self.events[2].name)
-
-        # Test search by ID
-        request_url = "%s?q=%s" % (reverse('event_archive'), self.events[1].pk)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.events[1].name)
-        self.assertNotContains(response, self.events[2].name)
-
-    def test_people_search(self):
-        # Test search by name
-        request_url = "%s?q=%s" % (reverse('person_list'), self.persons[1].name)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.persons[1].name)
-        self.assertNotContains(response, self.persons[2].name)
-
-        # Test search by ID
-        request_url = "%s?q=%s" % (reverse('person_list'), self.persons[1].pk)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.persons[1].name)
-        self.assertNotContains(response, self.persons[2].name)
-
-        # Test search by phone
-        request_url = "%s?q=%s" % (reverse('person_list'), self.persons[1].phone)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.persons[1].name)
-        self.assertNotContains(response, self.persons[2].name)
-
-    def test_organisation_search(self):
-        # Test search by name
-        request_url = "%s?q=%s" % (reverse('organisation_list'), self.organisations[1].name)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.organisations[1].name)
-        self.assertNotContains(response, self.organisations[2].name)
-
-        # Test search by ID
-        request_url = "%s?q=%s" % (reverse('organisation_list'), self.organisations[1].pk)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.organisations[1].name)
-        self.assertNotContains(response, self.organisations[2].name)
-
-        # Test search by email
-        request_url = "%s?q=%s" % (reverse('organisation_list'), self.organisations[1].email)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.organisations[1].email)
-        self.assertNotContains(response, self.organisations[2].email)
-
-    def test_venue_search(self):
-        # Test search by name
-        request_url = "%s?q=%s" % (reverse('venue_list'), self.venues[1].name)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.venues[1].name)
-        self.assertNotContains(response, self.venues[2].name)
-
-        # Test search by ID
-        request_url = "%s?q=%s" % (reverse('venue_list'), self.venues[1].pk)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.venues[1].name)
-        self.assertNotContains(response, self.venues[2].name)
-
-        # Test search by address
-        request_url = "%s?q=%s" % (reverse('venue_list'), self.venues[1].address)
-        response = self.client.get(request_url, follow=True)
-        self.assertContains(response, self.venues[1].address)
-        self.assertNotContains(response, self.venues[2].address)
--- a/RIGS/urls.py
+++ b/RIGS/urls.py
@@ -1,9 +1,7 @@
-from django.urls import path
 from django.conf.urls import url
-from django.contrib.auth.views import PasswordResetView
+from django.contrib.auth.views import password_reset

 from django.contrib.auth.decorators import login_required
-from django.contrib.auth.views import LoginView
 from RIGS import models, views, rigboard, finance, ical, versioning, forms
 from django.views.generic import RedirectView
 from django.views.decorators.clickjacking import xframe_options_exempt
@@ -18,10 +16,10 @@ urlpatterns = [
    url('^$', login_required(views.Index.as_view()), name='index'),
    url(r'^closemodal/$', views.CloseModal.as_view(), name='closemodal'),

-    path('user/login/', LoginView.as_view(authentication_form=forms.CheckApprovedForm), name='login'),
-    path('user/login/embed/', xframe_options_exempt(views.LoginEmbed.as_view()), name='login_embed'),
+    url('^user/login/$', views.login, name='login'),
+    url('^user/login/embed/$', xframe_options_exempt(views.login_embed), name='login_embed'),

-    url(r'^search_help/$', views.SearchHelp.as_view(), name='search_help'),
+    url(r'^user/password_reset/$', views.PasswordResetDisabled.as_view()),

    # People
    url(r'^people/$', permission_required_with_403('RIGS.view_person')(views.PersonList.as_view()),
--- a/RIGS/versioning.py
+++ b/RIGS/versioning.py
@@ -25,7 +25,7 @@ class FieldComparison(object):
        self._new = new

    def display_value(self, value):
-        if isinstance(self.field, IntegerField) and self.field.choices is not None and len(self.field.choices) > 0:
+        if isinstance(self.field, IntegerField) and len(self.field.choices) > 0:
            return [x[1] for x in self.field.choices if x[0] == value][0]
        if self.field.name == "risk_assessment_edit_url":
            return "completed" if value else ""
@@ -184,7 +184,8 @@ class RIGSVersion(Version):
        versions = RIGSVersion.objects.get_for_object_reference(self.content_type.model_class(), thisId).select_related("revision", "revision__user").all()

        try:
-            previousVersion = versions.filter(revision_id__lt=self.revision_id).latest('revision__date_created')
+            previousVersion = versions.filter(revision_id__lt=self.revision_id).latest(
+                field_name='revision__date_created')
        except ObjectDoesNotExist:
            return False

--- a/RIGS/views.py
+++ b/RIGS/views.py
@@ -3,7 +3,6 @@ from django.http.response import HttpResponseRedirect
 from django.http import HttpResponse
 from django.urls import reverse_lazy, reverse, NoReverseMatch
 from django.views import generic
-from django.contrib.auth.views import LoginView
 from django.db.models import Q
 from django.shortcuts import get_object_or_404
 from django.core import serializers
@@ -35,19 +34,28 @@ class Index(generic.TemplateView):
        return context


-class SearchHelp(generic.TemplateView):
-    template_name = 'RIGS/search_help.html'
+def login(request, **kwargs):
+    if request.user.is_authenticated:
+        next = request.GET.get('next', '/')
+        return HttpResponseRedirect(next)
+    else:
+        from django.contrib.auth.views import login
+
+        return login(request, authentication_form=forms.CheckApprovedForm)


 # This view should be exempt from requiring CSRF token.
 # Then we can check for it and show a nice error
 # Don't worry, django.contrib.auth.views.login will
 # check for it before logging  the user in
-class LoginEmbed(LoginView):
-    template_name = 'registration/login_embed.html'
+@csrf_exempt
+def login_embed(request, **kwargs):
+    if request.user.is_authenticated:
+        next = request.GET.get('next', '/')
+        return HttpResponseRedirect(next)
+    else:
+        from django.contrib.auth.views import login

-    @csrf_exempt
-    def dispatch(self, request, *args, **kwargs):
        if request.method == "POST":
            csrf_cookie = request.COOKIES.get('csrftoken', None)

@@ -55,7 +63,7 @@ class LoginEmbed(LoginView):
                messages.warning(request, 'Cookies do not seem to be enabled. Try logging in using a new tab.')
                request.method = 'GET'  # Render the page without trying to login

-        return super().dispatch(request, *args, **kwargs)
+        return login(request, template_name="registration/login_embed.html", authentication_form=forms.EmbeddedAuthenticationForm)


 """
@@ -78,20 +86,11 @@ class PersonList(generic.ListView):

    def get_queryset(self):
        q = self.request.GET.get('q', "")
-
-        filter = Q(name__icontains=q) | Q(email__icontains=q) | Q(address__icontains=q) | Q(notes__icontains=q) | Q(phone__startswith=q) | Q(phone__endswith=q)
-
-        # try and parse an int
-        try:
-            val = int(q)
-            filter = filter | Q(pk=val)
-        except:  # noqa
-            # not an integer
-            pass
-
-        object_list = self.model.objects.filter(filter)
-
-        orderBy = self.request.GET.get('orderBy', 'name')
+        if len(q) >= 3:
+            object_list = self.model.objects.filter(Q(name__icontains=q) | Q(email__icontains=q))
+        else:
+            object_list = self.model.objects.all()
+        orderBy = self.request.GET.get('orderBy', None)
        if orderBy is not None:
            object_list = object_list.order_by(orderBy)
        return object_list
@@ -141,20 +140,11 @@ class OrganisationList(generic.ListView):

    def get_queryset(self):
        q = self.request.GET.get('q', "")
-
-        filter = Q(name__icontains=q) | Q(email__icontains=q) | Q(address__icontains=q) | Q(notes__icontains=q) | Q(phone__startswith=q) | Q(phone__endswith=q)
-
-        # try and parse an int
-        try:
-            val = int(q)
-            filter = filter | Q(pk=val)
-        except:  # noqa
-            # not an integer
-            pass
-
-        object_list = self.model.objects.filter(filter)
-
-        orderBy = self.request.GET.get('orderBy', "name")
+        if len(q) >= 3:
+            object_list = self.model.objects.filter(Q(name__icontains=q) | Q(address__icontains=q))
+        else:
+            object_list = self.model.objects.all()
+        orderBy = self.request.GET.get('orderBy', "")
        if orderBy is not "":
            object_list = object_list.order_by(orderBy)
        return object_list
@@ -204,20 +194,11 @@ class VenueList(generic.ListView):

    def get_queryset(self):
        q = self.request.GET.get('q', "")
-
-        filter = Q(name__icontains=q) | Q(email__icontains=q) | Q(address__icontains=q) | Q(notes__icontains=q) | Q(phone__startswith=q) | Q(phone__endswith=q)
-
-        # try and parse an int
-        try:
-            val = int(q)
-            filter = filter | Q(pk=val)
-        except:  # noqa
-            # not an integer
-            pass
-
-        object_list = self.model.objects.filter(filter)
-
-        orderBy = self.request.GET.get('orderBy', "name")
+        if len(q) >= 3:
+            object_list = self.model.objects.filter(Q(name__icontains=q) | Q(address__icontains=q))
+        else:
+            object_list = self.model.objects.all()
+        orderBy = self.request.GET.get('orderBy', "")
        if orderBy is not "":
            object_list = object_list.order_by(orderBy)
        return object_list
@@ -411,3 +392,7 @@ class ResetApiKey(generic.RedirectView):
        self.request.user.save()

        return reverse_lazy('profile_detail')
+
+
+class PasswordResetDisabled(generic.TemplateView):
+    template_name = "RIGS/password_reset_disable.html"
--- a/assets/admin.py
+++ b/assets/admin.py
@@ -23,15 +23,10 @@ class SupplierAdmin(admin.ModelAdmin):
@admin.register(assets.Asset)
 class AssetAdmin(admin.ModelAdmin):
    list_display = ['id', 'asset_id', 'description', 'category', 'status']
-    list_filter = ['is_cable', 'category', 'status']
+    list_filter = ['is_cable', 'category']
    search_fields = ['id', 'asset_id', 'description']


-@admin.register(assets.CableType)
-class CableTypeAdmin(admin.ModelAdmin):
-    list_display = ['id', '__str__', 'plug', 'socket', 'cores', 'circuits']
-
-
@admin.register(assets.Connector)
 class ConnectorAdmin(admin.ModelAdmin):
    list_display = ['id', '__str__', 'current_rating', 'voltage_rating', 'num_pins']
--- a/assets/forms.py
+++ b/assets/forms.py
@@ -1,7 +1,6 @@
 from django import forms

 from assets import models
-from django.db.models import Q


 class AssetForm(forms.ModelForm):
@@ -13,7 +12,7 @@ class AssetForm(forms.ModelForm):
    class Meta:
        model = models.Asset
        fields = '__all__'
-        exclude = ['asset_id_prefix', 'asset_id_number', 'last_audited_at', 'last_audited_by']
+        exclude = ['asset_id_prefix', 'asset_id_number']

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
@@ -21,13 +20,6 @@ class AssetForm(forms.ModelForm):
        self.fields['date_acquired'].widget.format = '%Y-%m-%d'


-class AssetAuditForm(AssetForm):
-    class Meta(AssetForm.Meta):
-        # Prevents assets losing existing data that isn't included in the audit form
-        exclude = ['asset_id_prefix', 'asset_id_number', 'last_audited_at', 'last_audited_by',
-                   'parent', 'purchased_from', 'purchase_price', 'comments']
-
-
 class AssetSearchForm(forms.Form):
    query = forms.CharField(required=False)
    category = forms.ModelMultipleChoiceField(models.AssetCategory.objects.all(), required=False)
@@ -42,17 +34,3 @@ class SupplierForm(forms.ModelForm):

 class SupplierSearchForm(forms.Form):
    query = forms.CharField(required=False)
-
-
-class CableTypeForm(forms.ModelForm):
-    class Meta:
-        model = models.CableType
-        fields = '__all__'
-
-    def clean(self):
-        form_data = self.cleaned_data
-        queryset = models.CableType.objects.filter(Q(plug=form_data['plug']) & Q(socket=form_data['socket']) & Q(circuits=form_data['circuits']) & Q(cores=form_data['cores']))
-        # Being identical to itself shouldn't count...
-        if queryset.exists() and self.instance.pk != queryset[0].pk:
-            raise forms.ValidationError("A cable type that exactly matches this one already exists, please use that instead.", code="notunique")
-        return form_data
--- a/assets/management/commands/generateSampleAssetsData.py
+++ b/assets/management/commands/generateSampleAssetsData.py
@@ -1,9 +1,7 @@
 import random
 from django.core.management.base import BaseCommand, CommandError
 from django.utils import timezone
-from reversion import revisions as reversion
 from assets import models
-from RIGS import models as rigsmodels


 class Command(BaseCommand):
@@ -17,7 +15,6 @@ class Command(BaseCommand):

        random.seed('Some object to see the random number generator')

-        self.create_profile()
        self.create_categories()
        self.create_statuses()
        self.create_suppliers()
@@ -25,13 +22,6 @@ class Command(BaseCommand):
        self.create_connectors()
        self.create_cables()

-    # Make sure that there's at least one profile if this command is run standalone
-    def create_profile(self):
-        name = "Fred Johnson"
-        models.Profile.objects.create(username=name.replace(" ", ""), first_name=name.split(" ")[0], last_name=name.split(" ")[-1],
-                                      email=name.replace(" ", "") + "@example.com",
-                                      initials="".join([j[0].upper() for j in name.split()]))
-
    def create_categories(self):
        categories = ['Case', 'Video', 'General', 'Sound', 'Lighting', 'Rigging']

@@ -39,19 +29,17 @@ class Command(BaseCommand):
            models.AssetCategory.objects.create(name=cat)

    def create_statuses(self):
-        statuses = [('In Service', True, 'success'), ('Lost', False, 'warning'), ('Binned', False, 'danger'), ('Sold', False, 'danger'), ('Broken', False, 'warning')]
+        statuses = [('In Service', True), ('Lost', False), ('Binned', False), ('Sold', False), ('Broken', False)]

        for stat in statuses:
-            models.AssetStatus.objects.create(name=stat[0], should_show=stat[1], display_class=stat[2])
+            models.AssetStatus.objects.create(name=stat[0], should_show=stat[1])

    def create_suppliers(self):
        suppliers = ["Acme, inc.", "Widget Corp", "123 Warehousing", "Demo Company", "Smith and Co.", "Foo Bars", "ABC Telecom", "Fake Brothers", "QWERTY Logistics", "Demo, inc.", "Sample Company", "Sample, inc", "Acme Corp", "Allied Biscuit", "Ankh-Sto Associates", "Extensive Enterprise", "Galaxy Corp", "Globo-Chem", "Mr. Sparkle", "Globex Corporation", "LexCorp", "LuthorCorp", "North Central Positronics", "Omni Consimer Products", "Praxis Corporation", "Sombra Corporation", "Sto Plains Holdings", "Tessier-Ashpool", "Wayne Enterprises", "Wentworth Industries", "ZiffCorp", "Bluth Company", "Strickland Propane", "Thatherton Fuels", "Three Waters", "Water and Power", "Western Gas & Electric", "Mammoth Pictures", "Mooby Corp", "Gringotts", "Thrift Bank", "Flowers By Irene", "The Legitimate Businessmens Club", "Osato Chemicals", "Transworld Consortium", "Universal Export", "United Fried Chicken", "Virtucon", "Kumatsu Motors", "Keedsler Motors", "Powell Motors", "Industrial Automation", "Sirius Cybernetics Corporation", "U.S. Robotics and Mechanical Men", "Colonial Movers", "Corellian Engineering Corporation", "Incom Corporation", "General Products", "Leeding Engines Ltd.", "Blammo",  # noqa
                     "Input, Inc.", "Mainway Toys", "Videlectrix", "Zevo Toys", "Ajax", "Axis Chemical Co.", "Barrytron", "Carrys Candles", "Cogswell Cogs", "Spacely Sprockets", "General Forge and Foundry", "Duff Brewing Company", "Dunder Mifflin", "General Services Corporation", "Monarch Playing Card Co.", "Krustyco", "Initech", "Roboto Industries", "Primatech", "Sonky Rubber Goods", "St. Anky Beer", "Stay Puft Corporation", "Vandelay Industries", "Wernham Hogg", "Gadgetron", "Burleigh and Stronginthearm", "BLAND Corporation", "Nordyne Defense Dynamics", "Petrox Oil Company", "Roxxon", "McMahon and Tate", "Sixty Second Avenue", "Charles Townsend Agency", "Spade and Archer", "Megadodo Publications", "Rouster and Sideways", "C.H. Lavatory and Sons", "Globo Gym American Corp", "The New Firm", "SpringShield", "Compuglobalhypermeganet", "Data Systems", "Gizmonic Institute", "Initrode", "Taggart Transcontinental", "Atlantic Northern", "Niagular", "Plow King", "Big Kahuna Burger", "Big T Burgers and Fries", "Chez Quis", "Chotchkies", "The Frying Dutchman", "Klimpys", "The Krusty Krab", "Monks Diner", "Milliways", "Minuteman Cafe", "Taco Grande", "Tip Top Cafe", "Moes Tavern", "Central Perk", "Chasers"]  # noqa

-        with reversion.create_revision():
-            for supplier in suppliers:
-                reversion.set_user(random.choice(rigsmodels.Profile.objects.all()))
-                models.Supplier.objects.create(name=supplier)
+        for supplier in suppliers:
+            models.Supplier.objects.create(name=supplier)

    def create_assets(self):
        asset_description = ['Large cable', 'Shiny thing', 'New lights', 'Really expensive microphone', 'Box of fuse flaps', 'Expensive tool we didn\'t agree to buy', 'Cable drums', 'Boring amount of tape', 'Video stuff no one knows how to use', 'More amplifiers', 'Heatshrink']
@@ -60,24 +48,22 @@ class Command(BaseCommand):
        statuses = models.AssetStatus.objects.all()
        suppliers = models.Supplier.objects.all()

-        with reversion.create_revision():
-            for i in range(100):
-                reversion.set_user(random.choice(rigsmodels.Profile.objects.all()))
-                asset = models.Asset(
-                    asset_id='{}'.format(models.Asset.get_available_asset_id()),
-                    description=random.choice(asset_description),
-                    category=random.choice(categories),
-                    status=random.choice(statuses),
-                    date_acquired=timezone.now().date()
-                )
+        for i in range(100):
+            asset = models.Asset(
+                asset_id='{}'.format(models.Asset.get_available_asset_id()),
+                description=random.choice(asset_description),
+                category=random.choice(categories),
+                status=random.choice(statuses),
+                date_acquired=timezone.now().date()
+            )

-                if i % 4 == 0:
-                    asset.parent = models.Asset.objects.order_by('?').first()
+            if i % 4 == 0:
+                asset.parent = models.Asset.objects.order_by('?').first()

-                if i % 3 == 0:
-                    asset.purchased_from = random.choice(suppliers)
-                asset.clean()
-                asset.save()
+            if i % 3 == 0:
+                asset.purchased_from = random.choice(suppliers)
+            asset.clean()
+            asset.save()

    def create_cables(self):
        asset_description = ['The worm', 'Harting without a cap', 'Heavy cable', 'Extension lead', 'IEC cable that we should remember to prep']
@@ -92,9 +78,6 @@ class Command(BaseCommand):
        suppliers = models.Supplier.objects.all()
        connectors = models.Connector.objects.all()

-        for i in range(len(connectors)):
-            models.CableType.objects.create(plug=random.choice(connectors), socket=random.choice(connectors), circuits=random.choice(circuits), cores=random.choice(cores))
-
        for i in range(100):
            asset = models.Asset(
                asset_id='{}'.format(models.Asset.get_available_asset_id()),
@@ -104,9 +87,12 @@ class Command(BaseCommand):
                date_acquired=timezone.now().date(),

                is_cable=True,
-                cable_type=random.choice(models.CableType.objects.all()),
+                plug=random.choice(connectors),
+                socket=random.choice(connectors),
                csa=random.choice(csas),
                length=random.choice(lengths),
+                circuits=random.choice(circuits),
+                cores=random.choice(circuits)
            )

            if i % 5 == 0:
--- a/assets/migrations/0006_auto_20180728_1451_squashed_0021_auto_20190105_1156.py
+++ b/assets/migrations/0006_auto_20180728_1451_squashed_0021_auto_20190105_1156.py
@@ -57,7 +57,7 @@ class Migration(migrations.Migration):
        migrations.AddField(
            model_name='asset',
            name='parent',
-            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='asset_parent', to='assets.Asset'),
+            field=models.ForeignKey(blank=True, null=True, on_delete=None, related_name='asset_parent', to='assets.Asset'),
        ),
        migrations.RemoveField(
            model_name='asset',
@@ -85,7 +85,7 @@ class Migration(migrations.Migration):
                ('circuits', models.IntegerField(blank=True, null=True)),
                ('cores', models.IntegerField(blank=True, null=True)),
                ('category', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='assets.AssetCategory')),
-                ('parent', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='asset_parent', to='assets.Cable')),
+                ('parent', models.ForeignKey(blank=True, null=True, on_delete=None, related_name='asset_parent', to='assets.Cable')),
            ],
            options={
                'abstract': False,
--- a/assets/migrations/0010_auto_20200207_1737.py
+++ b/assets/migrations/0010_auto_20200207_1737.py
@@ -0,0 +1,17 @@
+# Generated by Django 2.0.13 on 2020-02-07 17:37
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('assets', '0009_auto_20200103_2215'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='supplier',
+            options={'ordering': ['name'], 'permissions': (('view_supplier', 'Can view a supplier'),)},
+        ),
+    ]
--- a/assets/migrations/0010_auto_20200219_1444.py
+++ b/assets/migrations/0010_auto_20200219_1444.py
@@ -1,21 +0,0 @@
-# Generated by Django 3.0.3 on 2020-02-19 14:44
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0009_auto_20200103_2215'),
-    ]
-
-    operations = [
-        migrations.AlterModelOptions(
-            name='asset',
-            options={'ordering': ['asset_id_prefix', 'asset_id_number'], 'permissions': [('asset_finance', 'Can see financial data for assets')]},
-        ),
-        migrations.AlterModelOptions(
-            name='supplier',
-            options={'ordering': ['name']},
-        ),
-    ]
--- a/assets/migrations/0011_auto_20200218_1617.py
+++ b/assets/migrations/0011_auto_20200218_1617.py
@@ -1,29 +0,0 @@
-# Generated by Django 2.0.13 on 2020-02-18 16:17
-
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0010_auto_20200219_1444'),
-    ]
-
-    operations = [
-        migrations.CreateModel(
-            name='CableType',
-            fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
-                ('circuits', models.IntegerField(blank=True, null=True)),
-                ('cores', models.IntegerField(blank=True, null=True)),
-                ('plug', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='plug', to='assets.Connector')),
-                ('socket', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='socket', to='assets.Connector')),
-            ],
-        ),
-        migrations.AddField(
-            model_name='asset',
-            name='cable_type',
-            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, to='assets.CableType'),
-        ),
-    ]
--- a/assets/migrations/0012_auto_20200218_1627.py
+++ b/assets/migrations/0012_auto_20200218_1627.py
@@ -1,26 +0,0 @@
-# Generated by Django 2.0.13 on 2020-02-18 16:27
-
-from django.db import migrations
-from django.db.models import Q
-
-
-def move_cable_type_data(apps, schema_editor):
-    Asset = apps.get_model('assets', 'Asset')
-    CableType = apps.get_model('assets', 'CableType')
-    for asset in Asset.objects.filter(is_cable=True):
-        # Only create one type per...well...type
-        if(not CableType.objects.filter(Q(plug=asset.plug) & Q(socket=asset.socket))):
-            cabletype = CableType.objects.create(plug=asset.plug, socket=asset.socket, circuits=asset.circuits, cores=asset.cores)
-            asset.save()
-            cabletype.save()
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0011_auto_20200218_1617'),
-    ]
-
-    operations = [
-        migrations.RunPython(move_cable_type_data)
-    ]
--- a/assets/migrations/0013_auto_20200218_1639.py
+++ b/assets/migrations/0013_auto_20200218_1639.py
@@ -1,29 +0,0 @@
-# Generated by Django 2.0.13 on 2020-02-18 16:39
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0012_auto_20200218_1627'),
-    ]
-
-    operations = [
-        migrations.RemoveField(
-            model_name='asset',
-            name='circuits',
-        ),
-        migrations.RemoveField(
-            model_name='asset',
-            name='cores',
-        ),
-        migrations.RemoveField(
-            model_name='asset',
-            name='plug',
-        ),
-        migrations.RemoveField(
-            model_name='asset',
-            name='socket',
-        ),
-    ]
--- a/assets/migrations/0014_auto_20200218_1840.py
+++ b/assets/migrations/0014_auto_20200218_1840.py
@@ -1,17 +0,0 @@
-# Generated by Django 2.0.13 on 2020-02-18 18:40
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0013_auto_20200218_1639'),
-    ]
-
-    operations = [
-        migrations.AlterModelOptions(
-            name='cabletype',
-            options={'ordering': ['plug', 'socket', '-circuits']},
-        ),
-    ]
--- a/assets/migrations/0015_remove_asset_next_sched_maint.py
+++ b/assets/migrations/0015_remove_asset_next_sched_maint.py
@@ -1,17 +0,0 @@
-# Generated by Django 3.0.3 on 2020-04-13 15:13
-
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0014_auto_20200218_1840'),
-    ]
-
-    operations = [
-        migrations.RemoveField(
-            model_name='asset',
-            name='next_sched_maint',
-        ),
-    ]
--- a/assets/migrations/0016_auto_20200413_1632.py
+++ b/assets/migrations/0016_auto_20200413_1632.py
@@ -1,34 +0,0 @@
-# Generated by Django 3.0.3 on 2020-04-13 15:32
-
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('assets', '0015_remove_asset_next_sched_maint'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='cabletype',
-            name='circuits',
-            field=models.IntegerField(default=1),
-        ),
-        migrations.AlterField(
-            model_name='cabletype',
-            name='cores',
-            field=models.IntegerField(default=3),
-        ),
-        migrations.AlterField(
-            model_name='cabletype',
-            name='plug',
-            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, related_name='plug', to='assets.Connector'),
-        ),
-        migrations.AlterField(
-            model_name='cabletype',
-            name='socket',
-            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, related_name='socket', to='assets.Connector'),
-        ),
-    ]
--- a/assets/migrations/0017_add_audit.py
+++ b/assets/migrations/0017_add_audit.py
@@ -1,31 +0,0 @@
-# Generated by Django 3.0.3 on 2020-04-13 00:06
-
-from django.conf import settings
-from django.db import migrations, models
-import django.db.models.deletion
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-        ('assets', '0016_auto_20200413_1632'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='asset',
-            name='last_audited_at',
-            field=models.DateTimeField(blank=True, null=True),
-        ),
-        migrations.AddField(
-            model_name='asset',
-            name='last_audited_by',
-            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='audited_by', to=settings.AUTH_USER_MODEL),
-        ),
-        migrations.AlterField(
-            model_name='asset',
-            name='csa',
-            field=models.DecimalField(blank=True, decimal_places=2, help_text='mm²', max_digits=10, null=True),
-        ),
-    ]
--- a/assets/models.py
+++ b/assets/models.py
@@ -9,7 +9,7 @@ from django.dispatch.dispatcher import receiver
 from reversion import revisions as reversion
 from reversion.models import Version

-from RIGS.models import RevisionMixin, Profile
+from RIGS.models import RevisionMixin


 class AssetCategory(models.Model):
@@ -41,10 +41,13 @@ class AssetStatus(models.Model):

@reversion.register
 class Supplier(models.Model, RevisionMixin):
+    name = models.CharField(max_length=80)
+
    class Meta:
        ordering = ['name']
-
-    name = models.CharField(max_length=80)
+        permissions = (
+            ('view_supplier', 'Can view a supplier'),
+        )

    def get_absolute_url(self):
        return reverse('supplier_list')
@@ -63,32 +66,14 @@ class Connector(models.Model):
        return self.description


-# Things are nullable that shouldn't be because I didn't properly fix the data structure when moving this to its own model...
-class CableType(models.Model):
-    class Meta:
-        ordering = ['plug', 'socket', '-circuits']
-
-    circuits = models.IntegerField(default=1)
-    cores = models.IntegerField(default=3)
-    plug = models.ForeignKey(Connector, on_delete=models.CASCADE,
-                             related_name='plug', null=True)
-    socket = models.ForeignKey(Connector, on_delete=models.CASCADE,
-                               related_name='socket', null=True)
-
-    def __str__(self):
-        if self.plug and self.socket:
-            return "%s → %s" % (self.plug.description, self.socket.description)
-        else:
-            return "Unknown"
-
-
@reversion.register
 class Asset(models.Model, RevisionMixin):
    class Meta:
        ordering = ['asset_id_prefix', 'asset_id_number']
-        permissions = [
-            ('asset_finance', 'Can see financial data for assets')
-        ]
+        permissions = (
+            ('asset_finance', 'Can see financial data for assets'),
+            ('view_asset', 'Can view an asset')
+        )

    parent = models.ForeignKey(to='self', related_name='asset_parent',
                               blank=True, null=True, on_delete=models.SET_NULL)
@@ -103,18 +88,20 @@ class Asset(models.Model, RevisionMixin):
    purchase_price = models.DecimalField(blank=True, null=True, decimal_places=2, max_digits=10)
    salvage_value = models.DecimalField(blank=True, null=True, decimal_places=2, max_digits=10)
    comments = models.TextField(blank=True)
-
-    # Audit
-    last_audited_at = models.DateTimeField(blank=True, null=True)
-    last_audited_by = models.ForeignKey(Profile, on_delete=models.SET_NULL, related_name='audited_by', blank=True, null=True)
+    next_sched_maint = models.DateField(blank=True, null=True)

    # Cable assets
    is_cable = models.BooleanField(default=False)
-    cable_type = models.ForeignKey(to=CableType, blank=True, null=True, on_delete=models.SET_NULL)
+    plug = models.ForeignKey(Connector, on_delete=models.SET_NULL,
+                             related_name='plug', blank=True, null=True)
+    socket = models.ForeignKey(Connector, on_delete=models.SET_NULL,
+                               related_name='socket', blank=True, null=True)
    length = models.DecimalField(decimal_places=1, max_digits=10,
                                 blank=True, null=True, help_text='m')
    csa = models.DecimalField(decimal_places=2, max_digits=10,
-                              blank=True, null=True, help_text='mm²')
+                              blank=True, null=True, help_text='mm^2')
+    circuits = models.IntegerField(blank=True, null=True)
+    cores = models.IntegerField(blank=True, null=True)

    # Hidden asset_id components
    # For example, if asset_id was "C1001" then asset_id_prefix would be "C" and number "1001"
@@ -144,7 +131,7 @@ class Asset(models.Model, RevisionMixin):
    def __str__(self):
        out = str(self.asset_id) + ' - ' + self.description
        if self.is_cable:
-            out += '{} - {}m - {}'.format(self.cable_type.plug, self.length, self.cable_type.socket)
+            out += '{} - {}m - {}'.format(self.plug, self.length, self.socket)
        return out

    def clean(self):
@@ -169,16 +156,14 @@ class Asset(models.Model, RevisionMixin):
                errdict["length"] = ["The length of a cable must be more than 0"]
            if not self.csa or self.csa <= 0:
                errdict["csa"] = ["The CSA of a cable must be more than 0"]
-            if not self.cable_type:
-                errdict["cable_type"] = ["A cable must have a type"]
-            # if not self.circuits or self.circuits <= 0:
-            #     errdict["circuits"] = ["There must be at least one circuit in a cable"]
-            # if not self.cores or self.cores <= 0:
-            #    errdict["cores"] = ["There must be at least one core in a cable"]
-            # if self.socket is None:
-            #     errdict["socket"] = ["A cable must have a socket"]
-            # if self.plug is None:
-            #    errdict["plug"] = ["A cable must have a plug"]
+            if not self.circuits or self.circuits <= 0:
+                errdict["circuits"] = ["There must be at least one circuit in a cable"]
+            if not self.cores or self.cores <= 0:
+                errdict["cores"] = ["There must be at least one core in a cable"]
+            if self.socket is None:
+                errdict["socket"] = ["A cable must have a socket"]
+            if self.plug is None:
+                errdict["plug"] = ["A cable must have a plug"]

        if errdict != {}:  # If there was an error when validation
            raise ValidationError(errdict)
--- a/assets/templates/asset_audit.html
+++ b/assets/templates/asset_audit.html
@@ -1,142 +0,0 @@
-{% extends request.is_ajax|yesno:'base_ajax.html,base_assets.html' %}
-{% load widget_tweaks %}
-{% block title %}Audit Asset {{ object.asset_id }}{% endblock %}
-
-{% block content %}
-<script>
-  function setAcquired(today) {
-        var date = new Date(1970, 0, 1);
-        if(today) {
-            date = new Date();
-        }
-        $('#id_date_acquired').val([date.getFullYear(), ('0' + (date.getMonth()+1)).slice(-2), ('0' + date.getDate()).slice(-2)].join('-'));
-  }
-  function setLength(length) {
-      $('#id_length').val(length);
-  }
-  function setCSA(CSA) {
-      $('#id_csa').val(CSA);
-  }
-  function checkIfCableHidden() {
-    if (document.getElementById("id_is_cable").checked) {
-      document.getElementById("cable-table").hidden = false;
-    } else {
-      document.getElementById("cable-table").hidden = true;
-    }
-  }
-  checkIfCableHidden();
-</script>
-<form class="form-horizontal" method="POST" id="asset_audit_form" action="{{ form.action|default:request.path }}">
-  {% include 'form_errors.html' %}
-  {% csrf_token %}
-  <input type="hidden" name="id" value="{{ object.id|default:0 }}" hidden=true>
-  <div class="form-group">
-    <label for="{{ form.asset_id.id_for_label }}" class="col-sm-2 control-label">Asset ID</label>
-    <div class="col-sm-10">
-      {% render_field form.asset_id|add_class:'form-control' value=object.asset_idz %}
-    </div>
-  </div>
-  <div class="form-group">
-    <label for="{{ form.description.id_for_label }}" class="col-sm-2 control-label">Description</label>
-    <div class="col-sm-10">
-      {% render_field form.description|add_class:'form-control' value=object.description %}
-    </div>
-  </div>
-  <div class="form-group">
-    <label for="{{ form.category.id_for_label }}" class="col-sm-2 control-label">Category</label>
-    <div class="col-sm-10">
-      {% render_field form.category|add_class:'form-control'%}
-    </div>
-  </div>
-  <div class="form-group">
-    <label for="{{ form.status.id_for_label }}" class="col-sm-2 control-label">Status</label>
-    <div class="col-sm-10">
-      {% render_field form.status|add_class:'form-control'%}
-    </div>
-  </div>
-  <div class="form-group">
-    <label for="{{ form.serial_number.id_for_label }}" class="col-sm-2 control-label">Serial Number</label>
-    <div class="col-sm-10">
-      {% render_field form.serial_number|add_class:'form-control' value=object.serial_number %}
-    </div>
-  </div>
-  <div class="form-group">
-    <label for="{{ form.date_acquired.id_for_label }}" class="col-sm-2 control-label">Date Acquired</label>
-    <div class="col-sm-6">
-      {% render_field form.date_acquired|add_class:'form-control' value=object.date_acquired %}
-    </div>
-    <div class="col-sm-4">
-        <btn class="btn btn-default" onclick="setAcquired(true);" tabindex="-1">Today</btn>
-        <btn class="btn btn-default" onclick="setAcquired(false);" tabindex="-1">Unknown</btn>
-    </div>
-  </div>
-  <div class="form-group">
-    <label for="{{ form.date_sold.id_for_label }}" class="col-sm-2 control-label">Date Sold</label>
-    <div class="col-sm-6">
-      {% render_field form.date_sold|add_class:'form-control' value=object.date_sold %}
-    </div>
-  </div>
-  <div class="form-group">
-      <label for="{{ form.salvage_value.id_for_label }}" class="col-sm-2 control-label">Salvage Value</label>
-      <div class="col-sm-10">
-          <div class="input-group">
-            <span class="input-group-addon">£</span>
-            {% render_field form.salvage_value|add_class:'form-control' value=object.salvage_value %}
-          </div>
-      </div>
-  </div>
-  <hr>
-  <div class="form-group">
-    <label for="{{ form.is_cable.id_for_label }}" class="col-sm-2 control-label">Cable?</label>
-    <div class="col-sm-10">
-      {% render_field form.is_cable|attr:'onchange=checkIfCableHidden()' %}
-    </div>
-  </div>
-  <div id="cable-table">
-    <div class="form-group">
-      <label for="{{ form.cable_type.id_for_label }}" class="col-sm-2 control-label">Cable Type</label>
-      <div class="col-sm-10">
-        {% render_field form.cable_type|add_class:'form-control' %}
-      </div>
-    </div>
-    <div class="form-group">
-      <label for="{{ form.length.id_for_label }}" class="col-sm-2 control-label">Length</label>
-      <div class="col-sm-6">
-        <div class="input-group">
-          {% render_field form.length|add_class:'form-control' %}
-          <span class="input-group-addon">{{ form.length.help_text }}</span>
-        </div>
-      </div>
-      <div class="col-sm-4">
-        <btn class="btn btn-danger" onclick="setLength('5');" tabindex="-1">5{{ form.length.help_text }}</btn>
-        <btn class="btn btn-success" onclick="setLength('10');" tabindex="-1">10{{ form.length.help_text }}</btn>
-        <btn class="btn btn-info" onclick="setLength('20');" tabindex="-1">20{{ form.length.help_text }}</btn>
-      </div>
-    </div>
-    <div class="form-group">
-      <label for="{{ form.csa.id_for_label }}" class="col-sm-2 control-label">Cross Sectional Area</label>
-      <div class="col-sm-6">
-        <div class="input-group">
-          {% render_field form.csa|add_class:'form-control' value=object.csa %}
-          <span class="input-group-addon">{{ form.csa.help_text }}</span>
-        </div>
-      </div>
-      <div class="col-sm-4">
-        <btn class="btn btn-default" onclick="setCSA('1.5');" tabindex="-1">1.5{{ form.csa.help_text }}</btn>
-        <btn class="btn btn-default" onclick="setCSA('2.5');" tabindex="-1">2.5{{ form.csa.help_text }}</btn>
-      </div>
-    </div>
-  </div>
-  {% if not request.is_ajax %}
-    <div class="form-group pull-right">
-        <button class="btn btn-success" type="submit" form="asset_audit_form" id="id_mark_audited">Mark Audited</button>
-    </div>
-  {% endif %}
-</form>
-{% endblock %}
-
-{% block footer %}
-<div class="form-group">
-  <button class="btn btn-success pull-right" type="submit" form="asset_audit_form" onclick="onAuditClick({{form.asset_id.value}});" id="id_mark_audited">Mark Audited</button>
-</div>
-{% endblock %}
--- a/assets/templates/asset_audit_list.html
+++ b/assets/templates/asset_audit_list.html
@@ -1,87 +0,0 @@
-{% extends 'base_assets.html' %}
-{% block title %}Asset Audit List{% endblock %}
-{% load static %}
-{% load paginator from filters %}
-{% load widget_tweaks %}
-
-{% block js %}
-    <script src="//code.jquery.com/ui/1.10.4/jquery-ui.js"></script>
-    <script src="{% static "js/interaction.js" %}"></script>
-    <script src="{% static "js/modal.js" %}"></script>
-    <script>
-    $('document').ready(function(){
-        $('#asset-search-form').submit(function () {
-         $('#searchButton').focus().click();
-         return false;
-        });
-        $('#searchButton').click(function (e) {
-             e.preventDefault();
-             var url = "{% url 'asset_audit' None %}";
-             var id = $("#{{form.query.id_for_label}}").val();
-             url = url.replace('None', id);
-             $.ajax({
-                url: url,
-                success: function(){
-                    $link = $(this);
-                    // Anti modal inception
-                    if ($link.parents('#modal').length == 0) {
-                        modaltarget = $link.data('target');
-                        modalobject = "";
-                        $('#modal').load(url, function (e) {
-                            $('#modal').modal();
-                        });
-                    }
-                },
-                error:function(){
-                    $("#error404").attr("hidden", false);
-                }
-             });
-        });
-    });
-    function onAuditClick(assetID) {
-         $('#' + assetID).remove();
-    }
-    </script>
-{% endblock %}
-
-{% block content %}
-<div class="page-header">
-  <h1 class="text-center">Asset Audit List</h1>
-</div>
-
-<div id="error404" class="alert alert-danger alert-dismissable" hidden=true>
-	<button type="button" class="close" data-dismiss="alert" aria-hidden="true">&times;</button>
-	<span>Asset with that ID does not exist!</span>
-</div>
-
-<h3>Audit Asset:</h3>
-<form id="asset-search-form" class="form-horizontal" method="POST">
-  <div class="input-group input-group-lg" style="width: auto;">
-    {% render_field form.query|add_class:'form-control' placeholder='Enter Asset ID' autofocus="true" %}
-    <label for="query" class="sr-only">Asset ID:</label>
-    <span class="input-group-btn"><a id="searchButton" class="btn btn-default" class="submit" type="submit">Search</a></span>
-  </div>
-</form>
-
-<h3>Assets Requiring Audit:</h3>
-<table class="table">
-  <thead>
-    <tr>
-      <th>Asset ID</th>
-      <th>Description</th>
-      <th>Category</th>
-      <th>Status</th>
-      <th class="hidden-xs"></th>
-    </tr>
-  </thead>
-  <tbody id="asset_table_body">
-    {% include 'partials/asset_list_table_body.html' with audit="true" %}
-  </tbody>
-</table>
-
-{% if is_paginated %}
-<div class="text-center">
-  {% paginator %}
-</div>
-{% endif %}
-{% endblock %}
--- a/assets/templates/asset_embed.html
+++ b/assets/templates/asset_embed.html
@@ -1,7 +1,8 @@
 {% extends 'base_embed.html' %}
-{% load static %}
+{% load static from staticfiles %}

 {% block content %}
+
 <div class="row">
    <div class="col-sm-12">
        <a href="/assets">
@@ -34,4 +35,6 @@
        </table>
    </div>
 </div>
+
+
 {% endblock %}
--- a/assets/templates/asset_update.html
+++ b/assets/templates/asset_update.html
@@ -1,4 +1,4 @@
-{% extends request.is_ajax|yesno:'base_ajax.html,base_assets.html' %}
+{% extends 'base_assets.html' %}
 {% load widget_tweaks %}
 {% block title %}Asset {{ object.asset_id }}{% endblock %}

@@ -23,23 +23,18 @@
    </div>
  </div>
  <div class="row">
-    <div class="col-md-4"
-    {% if not object.is_cable %} hidden="true" {% endif %} id="cable-table">
-      {% include 'partials/cable_form.html' %}
-    </div>
    {% if perms.assets.asset_finance %}
-    <div class="col-md-4">
+    <div class="col-md-6">
      {% include 'partials/purchasedetails_form.html' %}
    </div>
    {%endif%}
+    <div class="col-md-6"
+    {% if not object.is_cable %} hidden="true" {% endif %} id="cable-table">
+      {% include 'partials/cable_form.html' %}
+    </div>
    <div class="col-md-4">
      {% include 'partials/parent_form.html' %}
    </div>
-    {% if not edit %}
-    <div class="col-md-4">
-      {% include 'partials/audit_details.html' %}
-    </div>
-    {% endif %}
  </div>
  <div class="row">
    <div class="col-md-12">
--- a/assets/templates/cable_type_form.html
+++ b/assets/templates/cable_type_form.html
@@ -1,61 +0,0 @@
-{% extends 'base_assets.html' %}
-{% load widget_tweaks %}
-{% block title %}Cable Type{% endblock %}
-
-{% block content %}
-<div class="page-header">
-  <h1>
-      {% if create %}Create{% elif edit %}Edit{% endif %} Cable Type
-  </h1>
-</div>
-{% if create %}
-<form method="POST" action="{% url 'cable_type_create'%}">
-{% elif edit %}
-<form method="POST" action="{% url 'cable_type_update' object.id %}">
-{% endif %}
-{% include 'form_errors.html' %}
-{% csrf_token %}
-<input type="hidden" name="id" value="{{ object.id|default:0 }}" hidden=true>
-<div class="row">
-  <div class="col-sm-12">
-    {% if create or edit %}
-    <div class="form-group">
-      <label for="{{ form.plug.id_for_label }}">Plug</label>
-      {% render_field form.plug|add_class:'form-control'%}
-    </div>
-    <div class="form-group">
-      <label for="{{ form.socket.id_for_label }}">Socket</label>
-      {% render_field form.socket|add_class:'form-control'%}
-    </div>
-    <div class="form-group">
-      <label for="{{ form.circuits.id_for_label }}">Circuits</label>
-      {% render_field form.circuits|add_class:'form-control' value=object.circuits %}
-    </div>
-    <div class="form-group">
-      <label for="{{ form.cores.id_for_label }}">Cores</label>
-      {% render_field form.cores|add_class:'form-control' value=object.cores %}
-    </div>
-    <div class="pull-left">
-        <button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-floppy-disk"></i> Save</button>
-        <br>
-        <button type="reset" class="btn btn-link">Cancel</button>
-    </div>
-    {% else %}
-    <dl>
-      <dt>Socket</dt>
-      <dd>{{ object.socket|default_if_none:'-' }}</dd>
-
-      <dt>Plug</dt>
-      <dd>{{ object.plug|default_if_none:'-' }}</dd>
-
-      <dt>Circuits</dt>
-      <dd>{{ object.circuits|default_if_none:'-' }}</dd>
-
-      <dt>Cores</dt>
-      <dd>{{ object.cores|default_if_none:'-' }}</dd>
-    </dl>
-    {% endif %}
-  </div>
-</div>
-</form>
-{% endblock %}
--- a/assets/templates/cable_type_list.html
+++ b/assets/templates/cable_type_list.html
@@ -1,41 +0,0 @@
-{% extends 'base_assets.html' %}
-{% block title %}Supplier List{% endblock %}
-{% load paginator from filters %}
-{% load widget_tweaks %}
-
-{% block content %}
-<div class="page-header">
-    <h1>Cable Type List</h1>
-</div>
-
-<table class="table table-striped">
-  <thead>
-    <tr>
-      <th>Cable Type</th>
-      <th>Circuits</th>
-      <th>Cores</th>
-      <th>Quick Links</th>
-    </tr>
-  </thead>
-  <tbody>
-    {% for item in object_list %}
-    <tr>
-      <td>{{ item }}</td>
-      <td>{{ item.circuits }}</td>
-      <td>{{ item.cores }}</td>
-      <td>
-        <a href="{% url 'cable_type_detail' item.pk %}" class="btn btn-default"><i class="glyphicon glyphicon-eye-open"></i> View</a>
-        <a href="{% url 'cable_type_update' item.pk %}" class="btn btn-default"><i class="glyphicon glyphicon-edit"></i> Edit</a>
-      </td>
-    </tr>
-    {% endfor %}
-  </tbody>
-</table>
-
-{% if is_paginated %}
-<div class="text-center">
-  {% paginator %}
-</div>
-{% endif %}
-
-{% endblock %}
--- a/assets/templates/partials/asset_buttons.html
+++ b/assets/templates/partials/asset_buttons.html
@@ -1,28 +1,25 @@
-{% if perms.assets.change_asset %}
-    {% if edit and object %}
-    <!--edit-->
-    <button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-floppy-disk"></i> Save</button>
-    <a class="btn btn-default" href="{% url 'asset_duplicate' object.pk %}"><i class="glyphicon glyphicon-duplicate"></i> Duplicate</a>
-    {% elif duplicate %}
-    <!--duplicate-->
-    <button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-ok-sign"></i> Create Duplicate</button>
-    {% elif create %}
-    <!--create-->
-    <button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-floppy-disk"></i> Save</button>
-    {% else %}
-    <!--detail view-->
-    <div class="btn-group">
-        <a href="{% url 'asset_update' object.asset_id %}" class="btn btn-default"><i class="glyphicon glyphicon-edit"></i> Edit</a>
-        <a class="btn btn-default" href="{% url 'asset_duplicate' object.asset_id %}"><i class="glyphicon glyphicon-duplicate"></i> Duplicate</a>
-        <a type="button" class="btn btn-info" href="{% url 'asset_audit' object.asset_id %}"><i class="glyphicon glyphicon-object-align-left"></i> Audit</a>
-    </div>
-    {% endif %}
-    {% if create or edit or duplicate %}
-    <br>
-    <button type="reset" class="btn btn-link" onclick="
-    {%if duplicate%}
-    {% url 'asset_detail' previous_asset_id %}
-    {%else%}
-    history.back(){%endif%}">Cancel</button>
-    {% endif %}
+{% if edit and object %}
+<!--edit-->
+<button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-floppy-disk"></i> Save</button>
+<a class="btn btn-default" href="{% url 'asset_duplicate' object.pk %}"><i class="glyphicon glyphicon-duplicate"></i> Duplicate</a>
+{% elif duplicate %}
+<!--duplicate-->
+<button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-ok-sign"></i> Create Duplicate</button>
+{% elif create %}
+<!--create-->
+<button type="submit" class="btn btn-success"><i class="glyphicon glyphicon-floppy-disk"></i> Save</button>
+{% else %}
+<!--detail view-->
+<div class="btn-group">
+    <a href="{% url 'asset_update' object.asset_id %}" class="btn btn-default"><i class="glyphicon glyphicon-edit"></i> Edit</a>
+    <a class="btn btn-default" href="{% url 'asset_duplicate' object.asset_id %}"><i class="glyphicon glyphicon-duplicate"></i> Duplicate</a>
+</div>
+{% endif %}
+{% if create or edit or duplicate %}
+<br>
+<button type="reset" class="btn btn-link" onclick="
+{%if duplicate%}
+{% url 'asset_detail' previous_asset_id %}
+{%else%}
+history.back(){%endif%}">Cancel</button>
 {% endif %}
--- a/assets/templates/partials/asset_form.html
+++ b/assets/templates/partials/asset_form.html
@@ -23,6 +23,7 @@
      <label for="{{ form.category.id_for_label }}" >Category</label>
      {% render_field form.category|add_class:'form-control'%}
    </div>
+    {% render_field form.is_cable|attr:'onchange=checkIfCableHidden()' %} <label for="{{ form.is_cable.id_for_label }}">Cable?</label>
    <div class="form-group">
      <label for="{{ form.status.id_for_label }}" >Status</label>
      {% render_field form.status|add_class:'form-control'%}
@@ -31,10 +32,6 @@
      <label for="{{ form.serial_number.id_for_label }}">Serial Number</label>
      {% render_field form.serial_number|add_class:'form-control' value=object.serial_number %}
    </div>
-    <div class="form-group">
-        <label for="{{ form.is_cable.id_for_label }}">Cable?</label>
-        {% render_field form.is_cable|attr:'onchange=checkIfCableHidden()' %}
-    </div>
    <!---TODO: Lower default number of lines in comments box-->
    <div class="form-group">
      <label for="{{ form.comments.id_for_label }}">Comments</label>
--- a/assets/templates/partials/asset_list_table_body.html
+++ b/assets/templates/partials/asset_list_table_body.html
@@ -1,21 +1,19 @@
 {% for item in object_list %}
-    <tr class="{{ item.status.display_class|default:'' }} assetRow" id="{{item.asset_id}}">
+    {#                <li><a href="{% url 'asset_detail' item.pk %}">{{ item.asset_id }} - {{ item.description }}</a></li>#}
+    <!---TODO: When the ability to filter the list is added, remove the colours from the filter - specifically, stop greying out sold/binned stuff if it is being searched for-->
+    <tr class="{{ item.status.display_class|default:'' }} assetRow">
        <td style="vertical-align: middle;"><a class="assetID" href="{% url 'asset_detail' item.asset_id %}">{{ item.asset_id }}</a></td>
        <td class="assetDesc" style="vertical-align: middle; text-overflow: ellipsis; white-space: nowrap; overflow: hidden; max-width: 25vw">{{ item.description }}</td>
        <td class="assetCategory" style="vertical-align: middle;">{{ item.category }}</td>
        <td class="assetStatus" style="vertical-align: middle;">{{ item.status }}</td>
        <td class="hidden-xs">
            <div class="btn-group" role="group">
-            {% if audit %}
-                <a type="button" class="btn btn-info modal-href" href="{% url 'asset_audit' item.asset_id %}"><i class="glyphicon glyphicon-object-align-left"></i> Audit</a>
-            {% else %}
                <a type="button" class="btn btn-default btn-sm" href="{% url 'asset_detail' item.asset_id %}"><i class="glyphicon glyphicon-eye-open"></i> View</a>
                {% if perms.assets.change_asset %}
                <a type="button" class="btn btn-default btn-sm" href="{% url 'asset_update' item.asset_id %}"><i class="glyphicon glyphicon-edit"></i> Edit</a>
                <a type="button" class="btn btn-default btn-sm" href="{% url 'asset_duplicate' item.asset_id %}"><i class="glyphicon glyphicon-duplicate"></i> Duplicate</a>
                {% endif %}
            </div>
-            {% endif %}
        </td>
    </tr>
 {% endfor %}
--- a/assets/templates/partials/audit_details.html
+++ b/assets/templates/partials/audit_details.html
@@ -1,8 +0,0 @@
-<div class="panel {% if object.last_audited_at is not None %} panel-success {% else %} panel-warning {% endif %}">
-  <div class="panel-heading">
-    Audit Details
-  </div>
-  <div class="panel-body">
-        <p>Audited at <span class="label label-default">{{ object.last_audited_at|default_if_none:'-' }}</span> by <span class="label label-info">{{ object.last_audited_by|default_if_none:'-' }}</span></p>
-  </div>
-</div>
--- a/assets/templates/partials/cable_form.html
+++ b/assets/templates/partials/cable_form.html
@@ -6,10 +6,12 @@
  <div class="panel-body">
    {% if create or edit or duplicate %}
    <div class="form-group">
-      <label for="{{ form.cable_type.id_for_label }}">Cable Type</label>
-      <div class="input-group">
-        {% render_field form.cable_type|add_class:'form-control' %}
-      </div>
+      <label for="{{ form.plug.id_for_label }}">Plug</label>
+      {% render_field form.plug|add_class:'form-control'%}
+    </div>
+    <div class="form-group">
+      <label for="{{ form.socket.id_for_label }}">Socket</label>
+      {% render_field form.socket|add_class:'form-control'%}
    </div>
    <div class="form-group">
      <label for="{{ form.length.id_for_label }}">Length</label>
@@ -25,16 +27,33 @@
        <span class="input-group-addon">{{ form.csa.help_text }}</span>
      </div>
    </div>
+    <div class="form-group">
+      <label for="{{ form.circuits.id_for_label }}">Circuits</label>
+      {% render_field form.circuits|add_class:'form-control' value=object.circuits %}
+    </div>
+    <div class="form-group">
+      <label for="{{ form.cores.id_for_label }}">Cores</label>
+      {% render_field form.cores|add_class:'form-control' value=object.cores %}
+    </div>
    {% else %}
    <dl>
-      <dt>Cable Type</dt>
-      <dd>{{ object.cable_type|default_if_none:'-' }}</dd>
+      <dt>Socket</dt>
+      <dd>{{ object.socket|default_if_none:'-' }}</dd>
+
+      <dt>Plug</dt>
+      <dd>{{ object.plug|default_if_none:'-' }}</dd>

      <dt>Length</dt>
      <dd>{{ object.length|default_if_none:'-' }}m</dd>

      <dt>Cross Sectional Area</dt>
-      <dd>{{ object.csa|default_if_none:'-' }}mm²</dd>
+      <dd>{{ object.csa|default_if_none:'-' }}m^2</dd>
+
+      <dt>Circuits</dt>
+      <dd>{{ object.circuits|default_if_none:'-' }}</dd>
+
+      <dt>Cores</dt>
+      <dd>{{ object.cores|default_if_none:'-' }}</dd>
    </dl>
    {% endif %}
  </div>
--- a/assets/tests/pages.py
+++ b/assets/tests/pages.py
@@ -6,7 +6,7 @@ from selenium.webdriver import Chrome
 from django.urls import reverse
 from PyRIGS.tests import regions
 from PyRIGS.tests.pages import BasePage, FormPage
-from selenium.common.exceptions import NoSuchElementException
+import pdb


 class AssetList(BasePage):
@@ -82,9 +82,12 @@ class AssetForm(FormPage):
        'category': (regions.SingleSelectPicker, (By.ID, 'id_category')),
        'status': (regions.SingleSelectPicker, (By.ID, 'id_status')),

-        'cable_type': (regions.SingleSelectPicker, (By.ID, 'id_cable_type')),
+        'plug': (regions.SingleSelectPicker, (By.ID, 'id_plug')),
+        'socket': (regions.SingleSelectPicker, (By.ID, 'id_socket')),
        'length': (regions.TextBox, (By.ID, 'id_length')),
        'csa': (regions.TextBox, (By.ID, 'id_csa')),
+        'circuits': (regions.TextBox, (By.ID, 'id_circuits')),
+        'cores': (regions.TextBox, (By.ID, 'id_cores'))
    }

    @property
@@ -95,6 +98,11 @@ class AssetForm(FormPage):
    def parent_selector(self):
        return regions.BootstrapSelectElement(self, self.find_element(*self._parent_select_locator))

+    def submit(self):
+        previous_errors = self.errors
+        self.find_element(*self._submit_locator).click()
+        self.wait.until(lambda x: self.errors != previous_errors or self.success)
+

 class AssetEdit(AssetForm):
    URL_TEMPLATE = '/assets/asset/id/{asset_id}/edit/'
@@ -157,6 +165,11 @@ class SupplierForm(FormPage):
        'name': (regions.TextBox, (By.ID, 'id_name')),
    }

+    def submit(self):
+        previous_errors = self.errors
+        self.find_element(*self._submit_locator).click()
+        self.wait.until(lambda x: self.errors != previous_errors or self.success)
+

 class SupplierCreate(SupplierForm):
    URL_TEMPLATE = reverse('supplier_create')
@@ -173,90 +186,3 @@ class SupplierEdit(SupplierForm):
    @property
    def success(self):
        return '/edit' not in self.driver.current_url
-
-
-class AssetAuditList(AssetList):
-    URL_TEMPLATE = reverse('asset_audit_list')
-
-    _search_text_locator = (By.ID, 'id_query')
-    _go_button_locator = (By.ID, 'searchButton')
-    _modal_locator = (By.ID, 'modal')
-    _errors_selector = (By.CLASS_NAME, "alert-danger")
-
-    @property
-    def modal(self):
-        return self.AssetAuditModal(self, self.find_element(*self._modal_locator))
-
-    @property
-    def query(self):
-        return self.find_element(*self._search_text_locator).text
-
-    def set_query(self, queryString):
-        element = self.find_element(*self._search_text_locator)
-        element.clear()
-        element.send_keys(queryString)
-
-    def search(self):
-        self.find_element(*self._go_button_locator).click()
-
-    @property
-    def error(self):
-        try:
-            return self.find_element(*self._errors_selector)
-        except NoSuchElementException:
-            return None
-
-    class AssetAuditModal(Region):
-        _errors_selector = (By.CLASS_NAME, "alert-danger")
-        # Don't use the usual success selector - that tries and fails to hit the '10m long cable' helper button...
-        _submit_locator = (By.ID, "id_mark_audited")
-        form_items = {
-            'asset_id': (regions.TextBox, (By.ID, 'id_asset_id')),
-            'description': (regions.TextBox, (By.ID, 'id_description')),
-            'is_cable': (regions.CheckBox, (By.ID, 'id_is_cable')),
-            'serial_number': (regions.TextBox, (By.ID, 'id_serial_number')),
-            'salvage_value': (regions.TextBox, (By.ID, 'id_salvage_value')),
-            'date_acquired': (regions.DatePicker, (By.ID, 'id_date_acquired')),
-            'category': (regions.SingleSelectPicker, (By.ID, 'id_category')),
-            'status': (regions.SingleSelectPicker, (By.ID, 'id_status')),
-
-            'plug': (regions.SingleSelectPicker, (By.ID, 'id_plug')),
-            'socket': (regions.SingleSelectPicker, (By.ID, 'id_socket')),
-            'length': (regions.TextBox, (By.ID, 'id_length')),
-            'csa': (regions.TextBox, (By.ID, 'id_csa')),
-            'circuits': (regions.TextBox, (By.ID, 'id_circuits')),
-            'cores': (regions.TextBox, (By.ID, 'id_cores'))
-        }
-
-        @property
-        def errors(self):
-            try:
-                error_page = regions.ErrorPage(self, self.find_element(*self._errors_selector))
-                return error_page.errors
-            except NoSuchElementException:
-                return None
-
-        def submit(self):
-            previous_errors = self.errors
-            self.root.find_element(*self._submit_locator).click()
-            # self.wait.until(lambda x: not self.is_displayed) TODO
-
-        def remove_all_required(self):
-            self.driver.execute_script("Array.from(document.getElementsByTagName(\"input\")).forEach(function (el, ind, arr) { el.removeAttribute(\"required\")});")
-            self.driver.execute_script("Array.from(document.getElementsByTagName(\"select\")).forEach(function (el, ind, arr) { el.removeAttribute(\"required\")});")
-
-        def __getattr__(self, name):
-            if name in self.form_items:
-                element = self.form_items[name]
-                form_element = element[0](self, self.find_element(*element[1]))
-                return form_element.value
-            else:
-                return super().__getattribute__(name)
-
-        def __setattr__(self, name, value):
-            if name in self.form_items:
-                element = self.form_items[name]
-                form_element = element[0](self, self.find_element(*element[1]))
-                form_element.set_value(value)
-            else:
-                self.__dict__[name] = value
--- a/assets/tests/test_assets.py
+++ b/assets/tests/test_assets.py
@@ -9,13 +9,8 @@ from RIGS import models as rigsmodels
 from PyRIGS.tests.base import BaseTest, AutoLoginTest
 from assets import models, urls
 from reversion import revisions as reversion
-from selenium.webdriver.support import expected_conditions as EC
-from selenium.webdriver.common.by import By
 from selenium.webdriver.common.keys import Keys
-from selenium.webdriver.support.ui import WebDriverWait
-from RIGS.test_functional import animation_is_finished
 import datetime
-from django.utils import timezone


 class TestAssetList(AutoLoginTest):
@@ -103,7 +98,6 @@ class TestAssetForm(AutoLoginTest):
        self.supplier = models.Supplier.objects.create(name="Fullmetal Heavy Industry")
        self.parent = models.Asset.objects.create(asset_id="9000", description="Shelf", status=self.status, category=self.category, date_acquired=datetime.date(2000, 1, 1))
        self.connector = models.Connector.objects.create(description="IEC", current_rating=10, voltage_rating=240, num_pins=3)
-        self.cable_type = models.CableType.objects.create(plug=self.connector, socket=self.connector, circuits=1, cores=3)
        self.page = pages.AssetCreate(self.driver, self.live_server_url).open()

    def test_asset_create(self):
@@ -160,10 +154,12 @@ class TestAssetForm(AutoLoginTest):
        self.page.is_cable = True

        self.assertTrue(self.driver.find_element_by_id('cable-table').is_displayed())
-        self.page.cable_type = "IEC → IEC"
+        self.page.plug = "IEC"
        self.page.socket = "IEC"
        self.page.length = 10
        self.page.csa = "1.5"
+        self.page.circuits = 1
+        self.page.cores = 3

        self.page.submit()
        self.assertTrue(self.page.success)
@@ -253,83 +249,13 @@ class TestSupplierCreateAndEdit(AutoLoginTest):
    def test_supplier_edit(self):
        self.page = pages.SupplierEdit(self.driver, self.live_server_url, supplier_id=self.supplier.pk).open()

-        self.assertEqual("Fullmetal Heavy Industry", self.page.name)
+        self.assertEquals("Fullmetal Heavy Industry", self.page.name)
        new_name = "Cyberdyne Systems"
        self.page.name = new_name
        self.page.submit()
        self.assertTrue(self.page.success)


-class TestAssetAudit(AutoLoginTest):
-    def setUp(self):
-        super().setUp()
-        self.category = models.AssetCategory.objects.create(name="Haulage")
-        self.status = models.AssetStatus.objects.create(name="Probably Fine", should_show=True)
-        self.supplier = models.Supplier.objects.create(name="The Bazaar")
-        self.connector = models.Connector.objects.create(description="Trailer Socket", current_rating=1, voltage_rating=40, num_pins=13)
-        models.Asset.objects.create(asset_id="1", description="Trailer Cable", status=self.status, category=self.category, date_acquired=datetime.date(2020, 2, 1))
-        models.Asset.objects.create(asset_id="11", description="Trailerboard", status=self.status, category=self.category, date_acquired=datetime.date(2020, 2, 1))
-        models.Asset.objects.create(asset_id="111", description="Erms", status=self.status, category=self.category, date_acquired=datetime.date(2020, 2, 1))
-        models.Asset.objects.create(asset_id="1111", description="A hammer", status=self.status, category=self.category, date_acquired=datetime.date(2020, 2, 1))
-        self.page = pages.AssetAuditList(self.driver, self.live_server_url).open()
-        self.wait = WebDriverWait(self.driver, 5)
-
-    def test_audit_process(self):
-        asset_id = "1111"
-        self.page.set_query(asset_id)
-        self.page.search()
-        mdl = self.page.modal
-        self.wait.until(EC.visibility_of_element_located((By.ID, 'modal')))
-        # Do it wrong on purpose to check error display
-        mdl.remove_all_required()
-        mdl.description = ""
-        mdl.submit()
-        # self.wait.until(EC.visibility_of_element_located((By.ID, 'modal')))
-        self.wait.until(animation_is_finished())
-        # self.assertTrue(self.driver.find_element_by_id('modal').is_displayed())
-        self.assertIn("This field is required.", mdl.errors["Description"])
-        # Now do it properly
-        new_desc = "A BIG hammer"
-        mdl.description = new_desc
-        mdl.submit()
-        self.wait.until(animation_is_finished())
-        self.assertFalse(self.driver.find_element_by_id('modal').is_displayed())
-
-        # Check data is correct
-        audited = models.Asset.objects.get(asset_id="1111")
-        self.assertEqual(audited.description, new_desc)
-        # Make sure audit 'log' was filled out
-        self.assertEqual(self.profile.initials, audited.last_audited_by.initials)
-        self.assertEqual(timezone.now().date(), audited.last_audited_at.date())
-        self.assertEqual(timezone.now().hour, audited.last_audited_at.hour)
-        self.assertEqual(timezone.now().minute, audited.last_audited_at.minute)
-        # Check we've removed it from the 'needing audit' list
-        self.assertNotIn(asset_id, self.page.assets)
-
-    def test_audit_list(self):
-        self.assertEqual(len(models.Asset.objects.filter(last_audited_at=None)), len(self.page.assets))
-
-        assetRow = self.page.assets[0]
-        assetRow.find_element(By.CSS_SELECTOR, "td:nth-child(5) > div:nth-child(1) > a:nth-child(1)").click()
-        self.wait.until(EC.visibility_of_element_located((By.ID, 'modal')))
-        self.assertEqual(self.page.modal.asset_id, assetRow.id)
-
-        # First close button is for the not found error
-        self.page.find_element(By.XPATH, '(//button[@class="close"])[2]').click()
-        self.wait.until(animation_is_finished())
-        self.assertFalse(self.driver.find_element_by_id('modal').is_displayed())
-        # Make sure audit log was NOT filled out
-        audited = models.Asset.objects.get(asset_id=assetRow.id)
-        self.assertEqual(None, audited.last_audited_by)
-
-        # Check that a failed search works
-        self.page.set_query("NOTFOUND")
-        self.page.search()
-        self.wait.until(animation_is_finished())
-        self.assertFalse(self.driver.find_element_by_id('modal').is_displayed())
-        self.assertIn("Asset with that ID does not exist!", self.page.error.text)
-
-
 class TestSupplierValidation(TestCase):
    @classmethod
    def setUpTestData(cls):
@@ -449,8 +375,7 @@ class TestFormValidation(TestCase):
        cls.status = models.AssetStatus.objects.create(name="Broken", should_show=True)
        cls.asset = models.Asset.objects.create(asset_id="9999", description="The Office", status=cls.status, category=cls.category, date_acquired=datetime.date(2018, 6, 15))
        cls.connector = models.Connector.objects.create(description="16A IEC", current_rating=16, voltage_rating=240, num_pins=3)
-        cls.cable_type = models.CableType.objects.create(circuits=11, cores=3, plug=cls.connector, socket=cls.connector)
-        cls.cable_asset = models.Asset.objects.create(asset_id="666", description="125A -> Jack", comments="The cable from Hell...", status=cls.status, category=cls.category, date_acquired=datetime.date(2006, 6, 6), is_cable=True, cable_type=cls.cable_type, length=10, csa="1.5")
+        cls.cable_asset = models.Asset.objects.create(asset_id="666", description="125A -> Jack", comments="The cable from Hell...", status=cls.status, category=cls.category, date_acquired=datetime.date(2006, 6, 6), is_cable=True, plug=cls.connector, socket=cls.connector, length=10, csa="1.5", circuits=1, cores=3)

    def setUp(self):
        self.profile.set_password('testuser')
@@ -474,9 +399,12 @@ class TestFormValidation(TestCase):
        response = self.client.post(url, {'asset_id': 'X$%A', 'is_cable': True})
        self.assertFormError(response, 'form', 'asset_id', 'An Asset ID can only consist of letters and numbers, with a final number')

-        self.assertFormError(response, 'form', 'cable_type', 'A cable must have a type')
+        self.assertFormError(response, 'form', 'plug', 'A cable must have a plug')
+        self.assertFormError(response, 'form', 'socket', 'A cable must have a socket')
        self.assertFormError(response, 'form', 'length', 'The length of a cable must be more than 0')
        self.assertFormError(response, 'form', 'csa', 'The CSA of a cable must be more than 0')
+        self.assertFormError(response, 'form', 'circuits', 'There must be at least one circuit in a cable')
+        self.assertFormError(response, 'form', 'cores', 'There must be at least one core in a cable')

    # Given that validation is done at model level it *shouldn't* need retesting...gonna do it anyway!
    def test_asset_edit(self):
@@ -494,19 +422,24 @@ class TestFormValidation(TestCase):
    def test_cable_edit(self):
        url = reverse('asset_update', kwargs={'pk': self.cable_asset.asset_id})
        # TODO Why do I have to send is_cable=True here?
-        response = self.client.post(url, {'is_cable': True, 'length': -3, 'csa': -3})
+        response = self.client.post(url, {'is_cable': True, 'length': -3, 'csa': -3, 'circuits': -4, 'cores': -8})

-        # TODO Can't figure out how to select the 'none' option...
-        # self.assertFormError(response, 'form', 'cable_type', 'A cable must have a type')
+        # Can't figure out how to select the 'none' option...
+        # self.assertFormError(response, 'form', 'plug', 'A cable must have a plug')
+        # self.assertFormError(response, 'form', 'socket', 'A cable must have a socket')
        self.assertFormError(response, 'form', 'length', 'The length of a cable must be more than 0')
        self.assertFormError(response, 'form', 'csa', 'The CSA of a cable must be more than 0')
+        self.assertFormError(response, 'form', 'circuits', 'There must be at least one circuit in a cable')
+        self.assertFormError(response, 'form', 'cores', 'There must be at least one core in a cable')

    def test_asset_duplicate(self):
        url = reverse('asset_duplicate', kwargs={'pk': self.cable_asset.asset_id})
-        response = self.client.post(url, {'is_cable': True, 'length': 0, 'csa': 0})
+        response = self.client.post(url, {'is_cable': True, 'length': 0, 'csa': 0, 'circuits': 0, 'cores': 0})

        self.assertFormError(response, 'form', 'length', 'The length of a cable must be more than 0')
        self.assertFormError(response, 'form', 'csa', 'The CSA of a cable must be more than 0')
+        self.assertFormError(response, 'form', 'circuits', 'There must be at least one circuit in a cable')
+        self.assertFormError(response, 'form', 'cores', 'There must be at least one core in a cable')


 class TestSampleDataGenerator(TestCase):
--- a/assets/urls.py
+++ b/assets/urls.py
@@ -22,11 +22,6 @@ urlpatterns = [
    path('activity', permission_required_with_403('assets.view_asset')
         (views.ActivityTable.as_view()), name='asset_activity_table'),

-    path('cabletype/list/', permission_required_with_403('assets.view_cable_type')(views.CableTypeList.as_view()), name='cable_type_list'),
-    path('cabletype/create/', permission_required_with_403('assets.add_cable_type')(views.CableTypeCreate.as_view()), name='cable_type_create'),
-    path('cabletype/<int:pk>/update/', permission_required_with_403('assets.change_cable_type')(views.CableTypeUpdate.as_view()), name='cable_type_update'),
-    path('cabletype/<int:pk>/detail/', permission_required_with_403('assets.view_cable_type')(views.CableTypeDetail.as_view()), name='cable_type_detail'),
-
    path('asset/search/', views.AssetSearch.as_view(), name='asset_search_json'),
    path('asset/id/<str:pk>/embed/',
         xframe_options_exempt(
@@ -36,9 +31,6 @@ urlpatterns = [
         views.AssetOembed.as_view(),
         name='asset_oembed'),

-    path('asset/audit/', permission_required_with_403('assets.change_asset')(views.AssetAuditList.as_view()), name='asset_audit_list'),
-    path('asset/id/<str:pk>/audit/', permission_required_with_403('assets.change_asset')(views.AssetAudit.as_view()), name='asset_audit'),
-
    path('supplier/list', views.SupplierList.as_view(), name='supplier_list'),
    path('supplier/<int:pk>', views.SupplierDetail.as_view(), name='supplier_detail'),
    path('supplier/create', permission_required_with_403('assets.add_supplier')
--- a/assets/views.py
+++ b/assets/views.py
@@ -4,17 +4,13 @@ from django.http import HttpResponse, Http404
 from django.views import generic
 from django.views.decorators.csrf import csrf_exempt
 from django.utils.decorators import method_decorator
-from django.urls import reverse_lazy, reverse
+from django.urls import reverse
 from django.db.models import Q
 from django.shortcuts import get_object_or_404
-from django.core import serializers
-from django.contrib import messages
 from assets import models, forms
 from RIGS import versioning

 import simplejson
-import datetime
-from django.utils import timezone


@method_decorator(csrf_exempt, name='dispatch')
@@ -113,14 +109,7 @@ class AssetEdit(LoginRequiredMixin, AssetIDUrlMixin, generic.UpdateView):
        return context

    def get_success_url(self):
-        if self.request.is_ajax():
-            url = reverse_lazy('closemodal')
-            update_url = str(reverse_lazy('asset_update', kwargs={'pk': self.object.pk}))
-            messages.info(self.request, "modalobject=" + serializers.serialize("json", [self.object]))
-            messages.info(self.request, "modalobject[0]['update_url']='" + update_url + "'")
-        else:
-            url = reverse_lazy('asset_detail', kwargs={'pk': self.object.asset_id, })
-        return url
+        return reverse("asset_detail", kwargs={"pk": self.object.asset_id})


 class AssetCreate(LoginRequiredMixin, generic.CreateView):
@@ -184,30 +173,6 @@ class AssetEmbed(AssetDetail):
    template_name = 'asset_embed.html'


-@method_decorator(csrf_exempt, name='dispatch')
-class AssetAuditList(AssetList):
-    template_name = 'asset_audit_list.html'
-    hide_hidden_status = False
-
-    # TODO Refresh this when the modal is submitted
-    def get_queryset(self):
-        self.form = forms.AssetSearchForm(data={})
-        return self.model.objects.filter(Q(last_audited_at__isnull=True))
-
-
-class AssetAudit(AssetEdit):
-    template_name = 'asset_audit.html'
-    form_class = forms.AssetAuditForm
-
-    def get_success_url(self):
-        # TODO For some reason this doesn't stick when done in form_valid??
-        asset = self.get_object()
-        asset.last_audited_by = self.request.user
-        asset.last_audited_at = timezone.now()
-        asset.save()
-        return super().get_success_url()
-
-
 class SupplierList(generic.ListView):
    model = models.Supplier
    template_name = 'supplier_list.html'
@@ -287,45 +252,3 @@ class ActivityTable(versioning.ActivityTable):
        versions = versioning.RIGSVersion.objects.get_for_multiple_models(
            [models.Asset, models.Supplier])
        return versions
-
-
-class CableTypeList(generic.ListView):
-    model = models.CableType
-    template_name = 'cable_type_list.html'
-    paginate_by = 40
-    # ordering = ['__str__']
-
-
-class CableTypeDetail(generic.DetailView):
-    model = models.CableType
-    template_name = 'cable_type_form.html'
-
-
-class CableTypeCreate(generic.CreateView):
-    model = models.CableType
-    template_name = "cable_type_form.html"
-    form_class = forms.CableTypeForm
-
-    def get_context_data(self, **kwargs):
-        context = super(CableTypeCreate, self).get_context_data(**kwargs)
-        context["create"] = True
-
-        return context
-
-    def get_success_url(self):
-        return reverse("cable_type_detail", kwargs={"pk": self.object.pk})
-
-
-class CableTypeUpdate(generic.UpdateView):
-    model = models.CableType
-    template_name = "cable_type_form.html"
-    form_class = forms.CableTypeForm
-
-    def get_context_data(self, **kwargs):
-        context = super(CableTypeUpdate, self).get_context_data(**kwargs)
-        context["edit"] = True
-
-        return context
-
-    def get_success_url(self):
-        return reverse("cable_type_detail", kwargs={"pk": self.object.pk})
--- a/requirements.txt
+++ b/requirements.txt
@@ -1,25 +1,41 @@
+beautifulsoup4==4.8.2
+contextlib2==0.6.0.post1
 diff-match-patch==20181111
 dj-database-url==0.5.0
 dj-static==0.0.6
 Django==3.0.3
+django-filter==2.2.0
+django-widget-tweaks==1.4.5
 django-debug-toolbar==2.2
 django-ical==1.7.0
 django-recaptcha==2.0.6
 django-registration-redux==2.7
 django-reversion==3.0.7
-django-widget-tweaks==1.4.5
+django-toolbelt==0.0.1
+premailer==3.6.1
+git+git://github.com/jazzband/django-widget-tweaks.git@1.4.2
 gunicorn==20.0.4
 icalendar==4.0.4
 lxml==4.5.0
-premailer==3.6.1
+Markdown==3.2.1
+Pillow==7.0.0
 psycopg2==2.8.4
+Pygments==2.5.2
 PyPDF2==1.26.0
-PyPOM==2.2.0
+python-dateutil==2.8.1
 pytz==2019.3
 raven==6.10.0
-requests==2.23.0
+reportlab==3.5.34
 selenium==3.141.0
 simplejson==3.17.0
+six==1.14.0
+sqlparse==0.3.0
+static3==0.7.0
+svg2rlg==0.3
+yolk==0.4.3
 whitenoise==5.0.1
-reportlab==3.4.0
 z3c.rml==3.9.1
+zope.event==4.4
+zope.interface==4.7.1
+zope.schema==4.9.3
+pypom==2.2.0
--- a/templates/400.html
+++ b/templates/400.html
@@ -1,5 +1,5 @@
 {% extends 'base_rigs.html' %}
-{% load static %}
+{% load staticfiles %}
 {% block title %}Bad Request{% endblock %}

 {% block content %}
--- a/templates/401.html
+++ b/templates/401.html
@@ -1,5 +1,5 @@
 {% extends 'base_rigs.html' %}
-{% load static %}
+{% load staticfiles %}
 {% block title %}Unauthorized{% endblock %}

 {% block content %}
--- a/templates/403.html
+++ b/templates/403.html
@@ -1,5 +1,5 @@
 {% extends 'base_rigs.html' %}
-{% load static %}
+{% load staticfiles %}
 {% block title %}Forbidden{% endblock %}

 {% block content %}
--- a/templates/404.html
+++ b/templates/404.html
@@ -1,5 +1,5 @@
 {% extends 'base_rigs.html' %}
-{% load static %}
+{% load staticfiles %}
 {% block title %}Page Not Found{% endblock %}

 {% block content %}
--- a/templates/500.html
+++ b/templates/500.html
@@ -1,5 +1,5 @@
 {% extends 'base_rigs.html' %}
-{% load static %}
+{% load staticfiles %}
 {% block title %}Server error{% endblock %}

 {% block content %}
--- a/templates/base.html
+++ b/templates/base.html
@@ -1,6 +1,7 @@
-{% load static %}
+{% load static from staticfiles %}
 {% load raven %}

+
 <!DOCTYPE html>
 <html
        dir="{% if LANGUAGE_BIDI %}rtl{% else %}ltr{% endif %}"
--- a/templates/base_assets.html
+++ b/templates/base_assets.html
@@ -10,33 +10,30 @@
 {% endblock %}

 {% block titleelements %}
-    <li class="dropdown">
-        <a href="#" class="dropdown-toggle" data-toggle="dropdown">Assets<b class="caret"></b></a>
-        <ul class="dropdown-menu">
-            <li><a href="{% url 'asset_list' %}"><span class="glyphicon glyphicon-list"></span> List Assets</a></li>
-            {% if perms.assets.add_asset %}
-                <li><a href="{% url 'asset_create' %}"><span class="glyphicon glyphicon-plus"></span> Create Asset</a></li>
-            {% endif %}
-            <li role="separator" class="divider"></li>
-            <li><a href="{% url 'cable_type_list' %}"><span class="glyphicon glyphicon-list"></span>
-                List Cable Types</a></li>
-            {% if perms.assets.add_cable_type %}
-                <li><a href="{% url 'cable_type_create' %}"><span class="glyphicon glyphicon-plus"></span> Create Cable Type</a></li>
-            {% endif %}
-        </ul>
-    </li>
-    <li class="dropdown">
-        <a href="#" class="dropdown-toggle" data-toggle="dropdown"> Suppliers<b class="caret"></b></a>
-        <ul class="dropdown-menu">
-            <li><a href="{% url 'supplier_list' %}"><span class="glyphicon glyphicon-list"></span>
-                List Suppliers</a></li>
-            {% if perms.assets.add_supplier %}
-                <li><a href="{% url 'supplier_create' %}"><span class="glyphicon glyphicon-plus"></span> Create Supplier</a></li>
-            {% endif %}
-        </ul>
-    </li>
+    {# % if perms.assets.view_asset % #}
+        <li class="dropdown">
+            <a href="#" class="dropdown-toggle" data-toggle="dropdown">Assets<b class="caret"></b></a>
+            <ul class="dropdown-menu">
+                <li><a href="{% url 'asset_list' %}"><span class="glyphicon glyphicon-list"></span> List Assets</a></li>
+                {% if perms.assets.add_asset %}
+                    <li><a href="{% url 'asset_create' %}"><span class="glyphicon glyphicon-plus"></span> Create Asset</a></li>
+                {% endif %}
+            </ul>
+        </li>
+    {# % endif % #}
+    {# % if perms.assets.view_supplier % #}
+        <li class="dropdown">
+            <a href="#" class="dropdown-toggle" data-toggle="dropdown"> Suppliers<b class="caret"></b></a>
+            <ul class="dropdown-menu">
+                <li><a href="{% url 'supplier_list' %}"><span class="glyphicon glyphicon-list"></span>
+                    List Suppliers</a></li>
+                {% if perms.assets.add_supplier %}
+                    <li><a href="{% url 'supplier_create' %}"><span class="glyphicon glyphicon-plus"></span> Create Supplier</a></li>
+                {% endif %}
+            </ul>
+        </li>
+    {# % endif % #}
    {% if perms.assets.view_asset %}
        <li><a href="{% url 'asset_activity_table' %}">Recent Changes</a></li>
-        <li><a href="{% url 'asset_audit_list' %}">Audit</a></li>
    {% endif %}
 {% endblock %}
--- a/templates/base_client.html
+++ b/templates/base_client.html
@@ -1,4 +1,4 @@
-{% load static %}
+{% load static from staticfiles %}
 {% load raven %}


--- a/templates/base_client_email.html
+++ b/templates/base_client_email.html
@@ -1,14 +1,17 @@
-{% load static %}
+{% load static from staticfiles %}
 {% load raven %}

+
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml">
+
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    <meta name="viewport" content="width=device-width">
  </head>

  <body>
+
    <table class="main-table">
        <tr class="client-header">
          <td align="center">
@@ -29,7 +32,7 @@
            <!--[if mso]>
             </td></tr></table>
             </center>
-            <![endif]-->
+            <![endif]--> 
          </td>
        </tr>
        <tr>
@@ -44,9 +47,12 @@
                <!--[if mso]>
                 </td></tr></table>
                 </center>
-                <![endif]-->
+                <![endif]--> 
            </td>
        </tr>
    </table>
+
+    
+
  </body>
 </html>
--- a/templates/base_embed.html
+++ b/templates/base_embed.html
@@ -1,4 +1,4 @@
-{% load static %}
+{% load static from staticfiles %}
 {% load raven %}

 <!DOCTYPE html>
--- a/templates/login_redirect.html
+++ b/templates/login_redirect.html
@@ -1,5 +1,5 @@
 {% extends 'base_rigs.html' %}
-{% load static %}
+{% load staticfiles %}
 {% block title %}Login Required{% endblock %}

 {% block js %}
--- a/templates/registration/loginform.html
+++ b/templates/registration/loginform.html
@@ -1,6 +1,8 @@
+
 {% load widget_tweaks %}
 {% include 'form_errors.html' %}
 <div class="col-sm-6 col-sm-offset-3 col-lg-4 col-lg-offset-4">
+
    <form action="{% url 'login' %}" method="post" role="form" target="_self">{% csrf_token %}
        <div class="form-group">
            <label for="id_username">{{ form.username.label }}</label>