From d5f85c84c8c4e8605849de416ad0271d38518858 Mon Sep 17 00:00:00 2001
From: James Graham <j.graham@software.ac.uk>
Date: Mon, 26 Apr 2021 20:29:22 +0100
Subject: [PATCH] fix: hide relationship buttons on own profile page

---
 .../templates/people/person/detail_full.html  | 42 ++++++++++---------
 .../people/person/detail_partial.html         | 42 ++++++++++---------
 people/templates/people/person/list.html      | 22 +++++-----
 3 files changed, 56 insertions(+), 50 deletions(-)

diff --git a/people/templates/people/person/detail_full.html b/people/templates/people/person/detail_full.html
index bbc7945..ff80350 100644
--- a/people/templates/people/person/detail_full.html
+++ b/people/templates/people/person/detail_full.html
@@ -22,30 +22,32 @@
 
     <h1>{{ person.name }}</h1>
 
-    <hr>
+    {% if person.user != request.user %}
+        <hr>
+
+        <div class="row justify-content-md-center">
+            <div class="col-md-3">
+                {% if relationship %}
+                    <a class="btn btn-warning btn-block"
+                        href="{% url 'people:person.relationship.create' person_pk=person.pk %}">Update Relationship
+                        </a>
+
+                {% else %}
+                    <a class="btn btn-success btn-block"
+                        href="{% url 'people:person.relationship.create' person_pk=person.pk %}">New Relationship
+                        </a>
+                {% endif %}
+            </div>
 
-    <div class="row justify-content-md-center">
-        <div class="col-md-3">
             {% if relationship %}
-                <a class="btn btn-warning btn-block"
-                    href="{% url 'people:person.relationship.create' person_pk=person.pk %}">Update Relationship
-                    </a>
-
-            {% else %}
-                <a class="btn btn-success btn-block"
-                    href="{% url 'people:person.relationship.create' person_pk=person.pk %}">New Relationship
-                    </a>
+                <div class="col-md-3">
+                    <a class="btn btn-danger btn-block"
+                        href="{% url 'people:relationship.end' pk=relationship.pk %}">End Relationship
+                        </a>
+                </div>
             {% endif %}
         </div>
-
-        {% if relationship %}
-            <div class="col-md-3">
-                <a class="btn btn-danger btn-block"
-                    href="{% url 'people:relationship.end' pk=relationship.pk %}">End Relationship
-                    </a>
-            </div>
-        {% endif %}
-    </div>
+    {% endif %}
 
     <hr>
 
diff --git a/people/templates/people/person/detail_partial.html b/people/templates/people/person/detail_partial.html
index 3d5e146..84c6060 100644
--- a/people/templates/people/person/detail_partial.html
+++ b/people/templates/people/person/detail_partial.html
@@ -22,30 +22,32 @@
 
     <h1>{{ person.name }}</h1>
 
-    <hr>
+    {% if person.user != request.user %}
+        <hr>
+
+        <div class="row justify-content-md-center">
+            <div class="col-md-3">
+                {% if relationship %}
+                    <a class="btn btn-warning btn-block"
+                        href="{% url 'people:person.relationship.create' person_pk=person.pk %}">Update Relationship
+                        </a>
+
+                {% else %}
+                    <a class="btn btn-success btn-block"
+                        href="{% url 'people:person.relationship.create' person_pk=person.pk %}">New Relationship
+                        </a>
+                {% endif %}
+            </div>
 
-    <div class="row justify-content-md-center">
-        <div class="col-md-3">
             {% if relationship %}
-                <a class="btn btn-warning btn-block"
-                    href="{% url 'people:person.relationship.create' person_pk=person.pk %}">Update Relationship
-                    </a>
-
-            {% else %}
-                <a class="btn btn-success btn-block"
-                    href="{% url 'people:person.relationship.create' person_pk=person.pk %}">New Relationship
-                    </a>
+                <div class="col-md-3">
+                    <a class="btn btn-danger btn-block"
+                        href="{% url 'people:relationship.end' pk=relationship.pk %}">End Relationship
+                        </a>
+                </div>
             {% endif %}
         </div>
-
-        {% if relationship %}
-            <div class="col-md-3">
-                <a class="btn btn-danger btn-block"
-                    href="{% url 'people:relationship.end' pk=relationship.pk %}">End Relationship
-                    </a>
-            </div>
-        {% endif %}
-    </div>
+    {% endif %}
 
     <hr>
 
diff --git a/people/templates/people/person/list.html b/people/templates/people/person/list.html
index 77502bd..a8bf1b9 100644
--- a/people/templates/people/person/list.html
+++ b/people/templates/people/person/list.html
@@ -37,17 +37,19 @@
                         <a class="btn btn-sm btn-info"
                            href="{% url 'people:person.detail' pk=person.pk %}">Profile</a>
 
-                        {% if person.pk in existing_relationships %}
-                            <a class="btn btn-sm btn-warning"
-                                style="width: 10rem"
-                                href="{% url 'people:person.relationship.create' person_pk=person.pk %}">Update Relationship
-                                </a>
+                        {% if person.user != request.user %}
+                            {% if person.pk in existing_relationships %}
+                                <a class="btn btn-sm btn-warning"
+                                    style="width: 10rem"
+                                    href="{% url 'people:person.relationship.create' person_pk=person.pk %}">Update Relationship
+                                    </a>
 
-                        {% else %}
-                            <a class="btn btn-sm btn-success"
-                                style="width: 10rem"
-                                href="{% url 'people:person.relationship.create' person_pk=person.pk %}">New Relationship
-                                </a>
+                            {% else %}
+                                <a class="btn btn-sm btn-success"
+                                    style="width: 10rem"
+                                    href="{% url 'people:person.relationship.create' person_pk=person.pk %}">New Relationship
+                                    </a>
+                            {% endif %}
                         {% endif %}
                     </td>
                 </tr>