#311 Admin delete InforequestEmail

chcemvediet/apps/inforequests/admin.py

@@ -267,6 +267,9 @@ def get_queryset(self, request):
         queryset = queryset.select_related(u'email')
         return queryset
 
+    def has_delete_permission(self, request, obj=None):
+        return obj.has_delete_permission if obj else True
+
 @admin.register(Branch, site=admin.site)
 class BranchAdmin(NoBulkDeleteAdminMixin, DeleteNestedInforequestEmailAdminMixin, admin.ModelAdmin):
     date_hierarchy = None

chcemvediet/apps/inforequests/models/inforequestemail.py

@@ -1,6 +1,7 @@
 # vim: expandtab
 # -*- coding: utf-8 -*-
 from django.db import models
+from django.utils.functional import cached_property
 from django.utils.translation import ugettext_lazy as _
 from aggregate_if import Count
 
@@ -72,6 +73,14 @@ class Meta:
                 [u'type', u'inforequest'],
                 ]
 
+    @cached_property
+    def has_delete_permission(self):
+        try:
+            action = self.email.action
+            return action not in Action.objects.of_inforequest(inforequest=self.inforequest)
+        except Action.DoesNotExist:
+            return True
+
     def __unicode__(self):
         return format(self.pk)
 
@@ -95,3 +104,6 @@ def datachecks(superficial, autofix):
         issues = [u'; '.join(issues)]
     for issue in issues:
         yield datacheck.Error(issue + u'.')
+
+# Must be after ``InforequestEmail`` to break cyclic dependency
+from .action import Action

chcemvediet/apps/inforequests/templates/admin/inforequests/inforequestemail/change_form.html

@@ -0,0 +1,17 @@
+{% extends "admin/change_form.html" %}
+{% load amend prepend from poleno.amend %}
+
+{% block content %}
+  {% amend %}
+    {{ block.super }}
+    {% if not original.has_delete_permission %}
+      {% prepend path='.//div[@class="submit-row"]' %}
+        <p class="deletelink-box">Email is dependent on:
+          <a href="{% url 'admin:inforequests_action_change' original.email.action.pk %}">
+          &lt;Action: {{ original.email.action.pk }}&gt;
+          </a>
+        </p>
+      {% endprepend %}
+    {% endif %}
+  {% endamend %}
+{% endblock %}