From d5388a0fb48b036db37aee6729f16204c47a976c Mon Sep 17 00:00:00 2001
From: Johannes Schriewer <hallo@dunkelstern.de>
Date: Tue, 22 Dec 2020 03:47:35 +0100
Subject: [PATCH] Hide edit buttons if user is not staff

---
 inventory/templates/inventory/area_detail.html  | 17 ++++++++++++-----
 inventory/templates/inventory/cell.html         |  8 ++++++--
 .../templates/inventory/workshop_detail.html    | 17 ++++++++++++-----
 .../templates/inventory/workshop_list.html      |  8 ++++++--
 4 files changed, 36 insertions(+), 14 deletions(-)

diff --git a/inventory/templates/inventory/area_detail.html b/inventory/templates/inventory/area_detail.html
index 3ee95f5..49a3385 100644
--- a/inventory/templates/inventory/area_detail.html
+++ b/inventory/templates/inventory/area_detail.html
@@ -11,27 +11,34 @@
         {% for a in area.area_related.all %}
             <li>
                 <a href="{% url 'area-detail' a.id %}" title="{{ a.description }}">{{ a.name }}</a>
-                <a class="edit" href="{% url "admin:inventory_area_change" object_id=area.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
-
+                {% if user.is_staff %}
+                    <a class="edit" href="{% url "admin:inventory_area_change" object_id=area.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% endif %}
             </li>
         {% empty %}
             <li>No areas defined</li>
         {% endfor %}
     </ul>
 
-    <p><a href="{% url "admin:inventory_area_add" %}?container={{ area.id }}&index=0">Create new area...</a></p>
+    {% if user.is_staff %}
+        <p><a href="{% url "admin:inventory_area_add" %}?container={{ area.id }}&index=0">Create new area...</a></p>
+    {% endif %}
 
     <h3>Boxes</h3>
     <ul>
         {% for box in area.box_related.all %}
             <li>
                 <a href="{% url 'box-detail' box.id %}" title="{{ box.description }}">{{ box.name }} ({{ box.layout.name }})</a>
-                <a class="edit" href="{% url "admin:inventory_box_change" object_id=box.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% if user.is_staff %}
+                    <a class="edit" href="{% url "admin:inventory_box_change" object_id=box.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% endif %}
             </li>
         {% empty %}
             <li>No boxes defined</li>
         {% endfor %}
     </ul>
 
-    <p><a href="{% url "admin:inventory_box_add" %}?container={{ area.id }}&index=0">Create new container...</a></p>
+    {% if user.is_staff %}
+        <p><a href="{% url "admin:inventory_box_add" %}?container={{ area.id }}&index=0">Create new container...</a></p>
+    {% endif %}
 {% endblock %}
\ No newline at end of file
diff --git a/inventory/templates/inventory/cell.html b/inventory/templates/inventory/cell.html
index a51f45b..71869e6 100644
--- a/inventory/templates/inventory/cell.html
+++ b/inventory/templates/inventory/cell.html
@@ -16,7 +16,9 @@
             </div>
         {% endif %}
         <div class="cell-buttons">
-            <a class="edit" href="{% url "admin:inventory_item_change" object_id=item.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+            {% if user.is_staff %}
+                <a class="edit" href="{% url "admin:inventory_item_change" object_id=item.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+            {% endif %}
             <a class="details" href="{% url "item-detail" pk=item.pk %}"><img class="icon" src="{% static "inventory/img/detail.svg" %}"></a>
         </div>
 
@@ -28,7 +30,9 @@
         {% endif %}
     {% else %}
         <div class="cell-buttons">
-            <a class="title missing" href="{% url "admin:inventory_item_add" %}?index={{ item.index }}&container={{ item.container_id }}"><br><img class="icon" src="{% static "inventory/img/add.svg" %}"></a>
+            {% if user.is_staff %}
+                <a class="title missing" href="{% url "admin:inventory_item_add" %}?index={{ item.index }}&container={{ item.container_id }}"><br><img class="icon" src="{% static "inventory/img/add.svg" %}"></a>
+            {% endif %}
         </div>
     {% endif %}
 </div>
\ No newline at end of file
diff --git a/inventory/templates/inventory/workshop_detail.html b/inventory/templates/inventory/workshop_detail.html
index cd63715..9c161d1 100644
--- a/inventory/templates/inventory/workshop_detail.html
+++ b/inventory/templates/inventory/workshop_detail.html
@@ -12,14 +12,18 @@
             <li>
                 <a href="{% url 'area-detail' area.id %}" title="{{ area.description }}">{{ area.name }}</a>
                 ({{ area.description }})
-                <a class="edit" href="{% url "admin:inventory_area_change" object_id=area.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% if user.is_staff %}
+                    <a class="edit" href="{% url "admin:inventory_area_change" object_id=area.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% endif %}
             </li>
         {% empty %}
             <li>No areas defined</li>
         {% endfor %}
     </ul>
 
-    <p><a href="{% url "admin:inventory_area_add" %}?container={{ workshop.id }}&index=0">Create new area...</a></p>
+    {% if user.is_staff %}
+        <p><a href="{% url "admin:inventory_area_add" %}?container={{ workshop.id }}&index=0">Create new area...</a></p>
+    {% endif %}
 
     <h3>Boxes</h3>
     <ul>
@@ -27,13 +31,16 @@
             <li>
                 <a href="{% url 'box-detail' box.id %}" title="{{ box.description }}">{{ box.name }} ({{ box.layout.name }})</a>
                 ({{ box.description }})
-                <a class="edit" href="{% url "admin:inventory_box_change" object_id=box.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% if user.is_staff %}
+                    <a class="edit" href="{% url "admin:inventory_box_change" object_id=box.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% endif %}
             </li>
         {% empty %}
             <li>No boxes defined</li>
         {% endfor %}
     </ul>
 
-    <p><a href="{% url "admin:inventory_box_add" %}?container={{ workshop.id }}&index=0">Create new box...</a></p>
-
+    {% if user.is_staff %}
+        <p><a href="{% url "admin:inventory_box_add" %}?container={{ workshop.id }}&index=0">Create new box...</a></p>
+    {% endif %}
 {% endblock %}
\ No newline at end of file
diff --git a/inventory/templates/inventory/workshop_list.html b/inventory/templates/inventory/workshop_list.html
index 85bfa0b..5b394e6 100644
--- a/inventory/templates/inventory/workshop_list.html
+++ b/inventory/templates/inventory/workshop_list.html
@@ -9,11 +9,15 @@
         {% for workshop in object_list %}
             <li>
                 <a href="{% url 'workshop-detail' workshop.id %}" title="{{ workshop.description }}">{{ workshop.name }}</a>
-                <a class="edit" href="{% url "admin:inventory_workshop_change" object_id=workshop.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% if user.is_staff %}
+                    <a class="edit" href="{% url "admin:inventory_workshop_change" object_id=workshop.pk %}"><img class="icon" src="{% static "inventory/img/edit.svg" %}"></a>
+                {% endif %}
             </li>
         {% endfor %}
     </ul>
 
-    <p><a href="{% url "admin:inventory_workshop_add" %}?layout=1">Create new workshop</a></p>
+    {% if user.is_staff %}
+        <p><a href="{% url "admin:inventory_workshop_add" %}?layout=1">Create new workshop...</a></p>
+    {% endif %}
 
 {% endblock %}
\ No newline at end of file