From 711dbcaa7f4c4c31091f4f0c40bd210c8827b340 Mon Sep 17 00:00:00 2001
From: tomtom5152 <tom@codedinternet.com>
Date: Wed, 5 Nov 2014 16:57:31 +0000
Subject: [PATCH] Refactored some views to make it more consistent with Django
 internal namings

---
 RIGS/templates/RIGS/person_list.html | 9 ++++++---
 RIGS/urls.py                         | 8 ++++----
 RIGS/views.py                        | 6 ++++--
 3 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/RIGS/templates/RIGS/person_list.html b/RIGS/templates/RIGS/person_list.html
index c7da422f..0d7b0cb1 100644
--- a/RIGS/templates/RIGS/person_list.html
+++ b/RIGS/templates/RIGS/person_list.html
@@ -13,12 +13,14 @@
                 <h3>People</h3>
             </div>
             <div class="col-sm-2">
-                <a href="{% url 'person_add' %}" class="btn btn-default pull-right">New <span class="glyphicon glyphicon-plus"></span></a>
+                <a href="{% url 'person_create' %}" class="btn btn-default pull-right">New <span
+                        class="glyphicon glyphicon-plus"></span></a>
             </div>
             <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" />
+                        <input type="search" name="q" placeholder="Search" value="{{ request.GET.q }}"
+                               class="form-control"/>
                     </div>
                 </form>
             </div>
@@ -48,7 +50,8 @@
                             <a href="{% url 'person_detail' person.pk %}" class="btn btn-default modal-href">
                                 <span class="glyphicon glyphicon-eye-open"></span>
                             </a>
-                            <a href="{% url 'person_update' person.pk %}" class="btn btn-default"><span class="glyphicon glyphicon-pencil"></span></a>
+                            <a href="{% url 'person_update' person.pk %}" class="btn btn-default"><span
+                                    class="glyphicon glyphicon-pencil"></span></a>
                         </td>
                     </tr>
                 {% empty %}
diff --git a/RIGS/urls.py b/RIGS/urls.py
index eecc025d..65bd7bc7 100644
--- a/RIGS/urls.py
+++ b/RIGS/urls.py
@@ -13,11 +13,11 @@ urlpatterns = patterns('',
                        url('^user/login/$', 'RIGS.views.login', name='login'),
 
                        # People
-                       url(r'^people/$', permission_required_with_403('RIGS.view_person')(views.PersonIndex.as_view()),
+                       url(r'^people/$', permission_required_with_403('RIGS.view_person')(views.PersonList.as_view()),
                            name='person_list'),
                        url(r'^people/add/$',
                            permission_required_with_403('RIGS.add_person')(views.PersonCreate.as_view()),
-                           name='person_add'),
+                           name='person_create'),
                        url(r'^people/(?P<pk>\d+)/$',
                            permission_required_with_403('RIGS.view_person')(views.PersonDetail.as_view()),
                            name='person_detail'),
@@ -27,8 +27,8 @@ urlpatterns = patterns('',
 
                        # Organisations
                        url(r'^organisations/$',
-                           permission_required_with_403('RIGS.view_organisation')(views.OrganisationIndex.as_view()),
-                           name='organisation_index'),
+                           permission_required_with_403('RIGS.view_organisation')(views.OrganisationList.as_view()),
+                           name='organisation_list'),
                        url(r'^organisations/add/$',
                            permission_required_with_403('RIGS.add_organisation')(views.OrganisationCreate.as_view()),
                            name='organisation_create'),
diff --git a/RIGS/views.py b/RIGS/views.py
index 9d061625..8484754f 100644
--- a/RIGS/views.py
+++ b/RIGS/views.py
@@ -26,7 +26,8 @@ class CloseModal(generic.TemplateView):
         from django.contrib import messages
         return {'messages', messages.get_messages(self.request)}
 
-class PersonIndex(generic.ListView):
+
+class PersonList(generic.ListView):
     model = models.Person
     paginate_by = 20
 
@@ -60,7 +61,8 @@ class PersonUpdate(generic.UpdateView):
             'pk': self.object.pk,
         })
 
-class OrganisationIndex(generic.ListView):
+
+class OrganisationList(generic.ListView):
     model = models.Organisation
     paginate_by = 20