refactor, add accordion

netbox/netbox/views/generic/object_views.py

@@ -321,6 +321,30 @@ class ObjectDeleteView(GetReturnURLMixin, BaseObjectView):
     def get_required_permission(self):
         return get_permission_for_model(self.queryset.model, 'delete')
 
+    def _get_dependent_objects(self, obj):
+        """
+        Returns a dict of dependent objects, model names as primary key, inside each two keys exist name and items (list)
+
+        Args:
+            obj: The object to return dependent objects for
+        """
+        using = router.db_for_write(obj._meta.model)
+        collector = Collector(using=using)
+        collector.collect([obj])
+        related_objects = {}
+        for model, instance in collector.instances_with_model():
+            if instance == obj:
+                continue
+            if model.__name__ not in related_objects:
+                related_objects[model.__name__] = {
+                    "name": model._meta.verbose_name,
+                    "items": []
+                }
+            related_objects[model.__name__]['items'].append(instance)
+            if len(related_objects[model.__name__]['items']) > 1:
+                related_objects[model.__name__]['name'] = model._meta.verbose_name_plural
+        return related_objects
+
     #
     # Request handlers
     #
@@ -335,15 +359,7 @@ class ObjectDeleteView(GetReturnURLMixin, BaseObjectView):
         obj = self.get_object(**kwargs)
         form = ConfirmationForm(initial=request.GET)
 
-        using = router.db_for_write(obj._meta.model)
+        related_objects = self._get_dependent_objects(obj)
-        collector = Collector(using=using)
-        collector.collect([obj])
-        related_objects = {}
-        for model, instance in collector.instances_with_model():
-            # we could ignore the instance == obj so that the list doesnt contain itself...
-            if model.__name__ not in related_objects:
-                related_objects[model.__name__] = []
-            related_objects[model.__name__].append(instance)
 
         # If this is an HTMX request, return only the rendered deletion form as modal content
         if is_htmx(request):

netbox/templates/htmx/delete_form.html

@@ -14,14 +14,26 @@
     </p>
     {% if related_objects %}
     <p>
-      This will cause deletion of the following items: <br>
+      This will cause deletion of the following objects: <br>
-      {% for model,objects in related_objects.items %} 
+      <div class="accordion" id="deleteAccordion">
-        - {{ model }}:
+
-        {% for object in objects %}
+      {% for model,object_dict in related_objects.items %} 
-          {{ object | linkify }}{% if not forloop.last %}, {% endif %}
+        <div class="accordion-item">
+          <h2 class="accordion-header" id="deleteheading{{ forloop.counter }}">
+            <button class="accordion-button collapsed" type="button" data-bs-toggle="collapse" data-bs-target="#collapse{{ forloop.counter }}" aria-expanded="false" aria-controls="collapse{{ forloop.counter }}">
+              {{ object_dict.items |length }} {{ object_dict.name }}
+            </button>
+          </h2>
+          <div id="collapse{{  forloop.counter }}" class="accordion-collapse collapse" aria-labelledby="deleteheading{{ forloop.counter }}" data-bs-parent="#deleteAccordion">
+            <div class="accordion-body">
+              {% for object in object_dict.items %}
+                - {{ object | linkify }}{% if not forloop.last %}<br> {% endif %}
               {% endfor %}
-        <br>
+            </div>
+          </div>
+        </div>
       {% endfor %}
+        </div>
     </p>
     {% endif %}
     {% render_form form %}