From e8416bc779144e09bef221f925950f7b6b770f6d Mon Sep 17 00:00:00 2001
From: James Graham <J.Graham@software.ac.uk>
Date: Tue, 18 Feb 2020 15:05:30 +0000
Subject: [PATCH] feat(people): Add people list view

---
 breccia_mapper/templates/base.html       |  2 +-
 people/templates/people/person/list.html | 37 ++++++++++++++++++++++++
 people/urls.py                           |  4 +++
 people/views.py                          | 12 ++++++--
 4 files changed, 51 insertions(+), 4 deletions(-)
 create mode 100644 people/templates/people/person/list.html

diff --git a/breccia_mapper/templates/base.html b/breccia_mapper/templates/base.html
index 60f1389..ecd2984 100644
--- a/breccia_mapper/templates/base.html
+++ b/breccia_mapper/templates/base.html
@@ -108,7 +108,7 @@
 
     {# Global banner if config.NOTICE_TEXT is set using Constance #}
     {% if config.NOTICE_TEXT %}
-        <div class="alert {{ config.NOTICE_CLASS }} rounded-0 mb-0" role="alert">
+        <div class="alert {{ config.NOTICE_CLASS }} rounded-0 mb-3" role="alert">
             <h4 class="alert-heading text-center mb-0">{{ config.NOTICE_TEXT }}</h4>
         </div>
     {% endif %}
diff --git a/people/templates/people/person/list.html b/people/templates/people/person/list.html
new file mode 100644
index 0000000..b3cf1f6
--- /dev/null
+++ b/people/templates/people/person/list.html
@@ -0,0 +1,37 @@
+{% extends 'base.html' %}
+
+{% block content %}
+    <nav aria-label="breadcrumb">
+        <ol class="breadcrumb">
+            <li class="breadcrumb-item active" aria-current="page">People</li>
+        </ol>
+    </nav>
+
+    <hr>
+
+    <table class="table table-borderless">
+        <thead>
+            <tr>
+                <th>Name</th>
+            </tr>
+        </thead>
+
+        <tbody>
+            {% for person in person_list.all %}
+                <tr>
+                    <td>{{ person }}</td>
+                    <td>
+                        <a class="btn btn-sm btn-info"
+                           href="{% url 'people:person.detail' pk=person.pk %}">Profile</a>
+                    </td>
+                </tr>
+
+            {% empty %}
+                <tr>
+                    <td>No records</td>
+                </tr>
+            {% endfor %}
+        </tbody>
+    </table>
+
+{% endblock %}
diff --git a/people/urls.py b/people/urls.py
index 82aa305..ce9dcc9 100644
--- a/people/urls.py
+++ b/people/urls.py
@@ -10,6 +10,10 @@ urlpatterns = [
          views.ProfileView.as_view(),
          name='person.profile'),
 
+    path('people',
+         views.PersonListView.as_view(),
+         name='person.list'),
+
     path('people/<int:pk>',
          views.ProfileView.as_view(),
          name='person.detail'),
diff --git a/people/views.py b/people/views.py
index a696824..4524edf 100644
--- a/people/views.py
+++ b/people/views.py
@@ -2,18 +2,25 @@
 Views for displaying or manipulating models in the 'people' app.
 """
 
-from django.views.generic import DetailView
+from django.views.generic import DetailView, ListView
 
 from . import models
 
 
+class PersonListView(ListView):
+    """
+    View displaying a list of :class:`Person` objects - searchable.
+    """
+    model = models.Person
+    template_name = 'people/person/list.html'
+
+
 class ProfileView(DetailView):
     """
     View displaying the profile of a :class:`Person` - who may be a user.
     """
     model = models.Person
     template_name = 'people/person/detail.html'
-    context_object_name = 'person'
 
 
 class RelationshipDetailView(DetailView):
@@ -22,4 +29,3 @@ class RelationshipDetailView(DetailView):
     """
     model = models.Relationship
     template_name = 'people/relationship/detail.html'
-    context_object_name = 'relationship'