Remove models from ancestors context

Before this, we generated the list of ancestors based on a list of model
instances.

We now create a simple python object, and pass that into the template
context instead.

Author: meshy

cbv/templates/cbv/klass_detail.html

@@ -73,7 +73,7 @@ <h2>Ancestors (<abbr title="Method Resolution Order">MRO</abbr>)</h2>
                         <li><strong>{{ klass.name }}</strong></li>
                         {% for ancestor in all_ancestors %}
                             <li>
-                                <a href="{{ ancestor.get_absolute_url }}" class="{% if ancestor in direct_ancestors %}direct{% endif %}">
+                                <a href="{{ ancestor.url }}" class="{% if ancestor.is_direct %}direct{% endif %}">
                                     {{ ancestor.name }}
                                 </a>
                             </li>

cbv/views.py

@@ -21,6 +21,12 @@ def get_redirect_url(self, *, url_name: str, **kwargs):
 class KlassDetailView(TemplateView):
     template_name = "cbv/klass_detail.html"
 
+    @attrs.frozen
+    class Ancestor:
+        name: str
+        url: str
+        is_direct: bool
+
     def get_context_data(self, **kwargs):
         qs = Klass.objects.filter(
             name__iexact=self.kwargs["klass"],
@@ -46,8 +52,16 @@ def get_context_data(self, **kwargs):
             klass.module.project_version, klass.module, klass
         )
         direct_ancestors = list(klass.get_ancestors())
+        ancestors = [
+            self.Ancestor(
+                name=ancestor.name,
+                url=ancestor.get_absolute_url(),
+                is_direct=ancestor in direct_ancestors,
+            )
+            for ancestor in klass.get_all_ancestors()
+        ]
         return {
-            "all_ancestors": list(klass.get_all_ancestors()),
+            "all_ancestors": ancestors,
             "all_children": list(klass.get_all_children()),
             "attributes": klass.get_prepared_attributes(),
             "canonical_url": self.request.build_absolute_uri(canonical_url_path),