From e3ef4ed90c4d116aa9343e4af62f53fa6579bda8 Mon Sep 17 00:00:00 2001
From: James Graham <J.Graham@software.ac.uk>
Date: Tue, 3 Mar 2020 09:55:47 +0000
Subject: [PATCH] feat(people): Add notice to fill in user details

---
 .gitignore                                 |  1 +
 breccia_mapper/templates/base.html         | 11 +++++++++++
 people/models.py                           | 10 ++++++++++
 people/templates/people/person/detail.html |  3 +++
 4 files changed, 25 insertions(+)

diff --git a/.gitignore b/.gitignore
index 2508438..5a7cc0c 100644
--- a/.gitignore
+++ b/.gitignore
@@ -18,3 +18,4 @@ deployment-key.pub
 # Deployment
 .vagrant/
 staging.yml
+/.dbbackup/
diff --git a/breccia_mapper/templates/base.html b/breccia_mapper/templates/base.html
index 6474fca..e840c53 100644
--- a/breccia_mapper/templates/base.html
+++ b/breccia_mapper/templates/base.html
@@ -127,6 +127,17 @@
         </div>
     {% endif %}
 
+    {% if request.user.is_authenticated and not request.user.has_person %}
+        <div class="alert alert-info rounded-0" role="alert">
+            <p class="text-center mb-0">
+                Your profile is currently blank.  Please fill in your details so you can be part of the network.
+
+                <a class="btn btn-success"
+                   href="{% url 'people:person.create' %}?user">Profile</a>
+            </p>
+        </div>
+    {% endif %}
+
     {% block before_content %}{% endblock %}
 
     <main class="container">
diff --git a/people/models.py b/people/models.py
index ff68f98..9dbf32b 100644
--- a/people/models.py
+++ b/people/models.py
@@ -15,6 +15,16 @@ class User(AbstractUser):
     """
     Custom user model in case we need to make changes later.
     """
+    def has_person(self) -> bool:
+        """
+        Does this user have a linked :class:`Person` record?
+        """
+        try:
+            person = self.person
+            return True
+
+        except Person.DoesNotExist:
+            return False
 
 
 class Organisation(models.Model):
diff --git a/people/templates/people/person/detail.html b/people/templates/people/person/detail.html
index 6886a30..1d68b5e 100644
--- a/people/templates/people/person/detail.html
+++ b/people/templates/people/person/detail.html
@@ -66,6 +66,9 @@
     <a class="btn btn-success"
        href="{% url 'people:person.update' pk=person.pk %}">Update</a>
 
+    <a class="btn btn-info"
+       href="{% url 'password_change' %}?next={{ person.get_absolute_url }}">Change Password</a>
+
     <hr>
 
     <div class="row">