Merge pull request #2084 from eeintech/show_part_delete

Show part delete button when part still active

InvenTree/part/templates/part/part_base.html

@@ -130,7 +130,7 @@
                             {% if roles.part.change %}
                             <li><a href='#' id='part-edit'><span class='fas fa-edit icon-blue'></span> {% trans "Edit part" %}</a></li>
                             {% endif %}
-                            {% if not part.active and roles.part.delete %}
+                            {% if roles.part.delete %}
                             <li><a href='#' id='part-delete'><span class='fas fa-trash-alt icon-red'></span> {% trans "Delete part" %}</a></li>
                             {% endif %}
                         </ul>
@@ -503,12 +503,13 @@
     });
     {% endif %}
 
-    {% if not part.active and roles.part.delete %}
+    {% if roles.part.delete %}
     $("#part-delete").click(function() {
         launchModalForm(
             "{% url 'part-delete' part.id %}",
             {
-                redirect: {% if part.category %}"{% url 'category-detail' part.category.id %}"{% else %}"{% url 'part-index' %}"{% endif %}
+                redirect: {% if part.category %}"{% url 'category-detail' part.category.id %}"{% else %}"{% url 'part-index' %}"{% endif %},
+                no_post: {% if part.active %}true{% else %}false{% endif %},
             }
         );
     });

InvenTree/part/templates/part/partial_delete.html

@@ -3,6 +3,16 @@
 
 {% block pre_form_content %}
 
+{% if part.active %}
+
+<div class='alert alert-block alert-danger'>
+    {% blocktrans with full_name=part.full_name %}Part '<strong>{{full_name}}</strong>' cannot be deleted as it is still marked as <strong>active</strong>.
+    <br>Disable the "Active" part attribute and re-try.
+    {% endblocktrans %}
+</div>
+
+{% else %}
+
 <div class='alert alert-block alert-danger'>
     {% blocktrans with full_name=part.full_name %}Are you sure you want to delete part '<strong>{{full_name}}</strong>'?{% endblocktrans %}
 </div>
@@ -55,4 +65,12 @@
 <p>{% blocktrans with count=part.serials.all|length full_name=part.full_name  %}There are {{count}} unique parts tracked for '{{full_name}}'. Deleting this part will permanently remove this tracking information.{% endblocktrans %}</p>
 {% endif %}
 
+{% endif %}
+
+{% endblock %}
+
+{% block form %}
+{% if not part.active %}
+{{ block.super }}
+{% endif %}
 {% endblock %}