Enhance article templates and optimize database queries for tagged pages

library/models.py

@@ -72,22 +72,20 @@ class LibraryItem(DrupalFields, Page):  # type: ignore
 
     @classmethod
     def get_queryset(cls):
-        related_fields = [
+        related_prefetch = [
             "authors__author",
             "topics__topic",
-            "item_audience",
-            "item_genre",
-            "item_medium",
-            "item_time_period",
-            "tags__tag",
+            "tags",
         ]
         return (
-            super()
-            .get_queryset()
-            .filter(live=True)
-            .prefetch_related(
-                *related_fields,
+            cls.objects.live()
+            .select_related(
+                "item_audience",
+                "item_genre",
+                "item_medium",
+                "item_time_period",
             )
+            .prefetch_related(*related_prefetch)
         )
 
     content_panels = Page.content_panels + [

magazine/models.py

@@ -299,25 +299,13 @@ def get_context(
     ) -> dict:
         context = super().get_context(request)
 
-        context["articles"] = (
-            MagazineArticle.objects.filter(
-                department__title=self.title,
-            )
-            .live()
-            .prefetch_related(
-                "authors__author",
-                "issue",
-            )
-        )
-
         articles = (
             MagazineArticle.objects.filter(
                 department__title=self.title,
             )
             .live()
             .prefetch_related(
                 "authors__author",
-                "issue",
             )
         )
 
@@ -365,9 +353,16 @@ class MagazineArticle(DrupalFields, Page):  # type: ignore
 
     @classmethod
     def get_queryset(cls):
-        """Prefetch authors and tags for performance."""
-        related_fields = ["authors__author", "tags__tag", "department"]
-        return super().get_queryset().prefetch_related(*related_fields)
+        """Optimize related fetches for listings.
+
+        Note: Use cls.objects rather than super().get_queryset() because the
+        base class does not provide a classmethod get_queryset.
+        """
+        return (
+            cls.objects.defer_streamfields()
+            .select_related("department")
+            .prefetch_related("authors__author", "tags")
+        )
 
     class Meta:
         verbose_name = "Magazine Article"

magazine/templates/magazine/magazine_article_summary.html

@@ -1,4 +1,4 @@
-{% load wagtailcore_tags %}
+{% load i18n wagtailcore_tags %}
 
 <article class="card bg-base-100 shadow-sm hover:shadow-md transition-shadow mb-4">
     <div class="card-body">
@@ -8,29 +8,44 @@ <h3 class="card-title">
             </a>
         </h3>
 
-        {% if article.authors.count %}
-            <div class="mb-2">
-                <span class="font-medium">Author(s):</span>
-                {% for author in article.authors.all %}
-                    {% if author.author.live %}
-                        <a href="{% pageurl author.author %}">{{ author.author }}</a>{% if not forloop.last %},{% endif %}
-                    {% else %}
-                        {{ author.author }}{% if not forloop.last %},{% endif %}
-                    {% endif %}
-                {% endfor %}
-            </div>
-        {% endif %}
+        {% with author_links=article.authors.all %}
+            {% if author_links %}
+                <div class="mb-2">
+                    <span class="font-medium">{% translate "Authors" %}:</span>
+                    <ul class="inline">
+                        {% for author in author_links %}
+                            <li class="inline">
+                                {% if author.author.live %}
+                                    <a href="{% pageurl author.author %}">{{ author.author }}</a>
+                                {% else %}
+                                    {{ author.author }}
+                                {% endif %}
+                                {% if not forloop.last %}, {% endif %}
+                            </li>
+                        {% endfor %}
+                    </ul>
+                </div>
+            {% endif %}
+        {% endwith %}
 
         <div class="prose">
             {{ article.teaser|richtext }}
         </div>
 
-        <div class="card-actions">
-            <span>
-                Issue: <a href="{% pageurl article.get_parent %}" class="link">
-                    {{ article.get_parent }} ({{ article.get_parent.specific.publication_date|date:"F Y" }})
-                </a>
-            </span>
-        </div>
+        {% with issue=article.get_parent %}
+            <div class="card-actions">
+                <span>
+                    <span class="font-medium">{% translate "Issue" %}:</span>
+                    <a href="{% pageurl issue %}" class="link">
+                        {{ issue }}
+                        {% if issue.specific.publication_date %}
+                            (<time datetime="{{ issue.specific.publication_date|date:'Y-m-d' }}">
+                                {{ issue.specific.publication_date|date:"F Y" }}
+                            </time>)
+                        {% endif %}
+                    </a>
+                </span>
+            </div>
+        {% endwith %}
     </div>
 </article>

magazine/templates/search/magazine_article.html

@@ -0,0 +1 @@
+{% include "magazine/magazine_article_summary.html" with article=entity %}

magazine/templates/search/magazine_issue.html

@@ -0,0 +1,16 @@
+{% load wagtailcore_tags %}
+<article class="card bg-base-100 shadow-sm hover:shadow-md transition-shadow mb-4">
+  <div class="card-body">
+    <h3 class="card-title">
+      <a href="{% pageurl entity %}">{{ entity.title }}</a>
+    </h3>
+    {% if entity.specific.publication_date %}
+      <p class="text-sm text-base-content/70">
+        Published
+        <time datetime="{{ entity.specific.publication_date|date:'Y-m-d' }}">
+          {{ entity.specific.publication_date|date:"F Y" }}
+        </time>
+      </p>
+    {% endif %}
+  </div>
+</article>

tags/views.py

@@ -25,15 +25,42 @@ def get_queryset(self):
         # Get all the pages tagged with the given tag
         filter_condition = Q(tags__slug__in=[tag])
 
-        library_items = LibraryItem.objects.filter(filter_condition).order_by("title")
+        library_items = (
+            LibraryItem.get_queryset()
+            .filter(filter_condition)
+            .live()
+            .public()
+            .select_related("content_type")
+            .prefetch_related("authors__author")
+            # DB-level ordering unnecessary; combined list is sorted below
+        )
 
-        magazine_articles = MagazineArticle.objects.filter(filter_condition).order_by(
-            "title",
+        magazine_articles = (
+            MagazineArticle.get_queryset()
+            .filter(filter_condition)
+            .live()
+            .public()
+            .select_related("content_type")
+            .prefetch_related("authors__author")
+            # DB-level ordering unnecessary; combined list is sorted below
         )
 
-        news_items = NewsItem.objects.filter(filter_condition).order_by("title")
+        news_items = (
+            NewsItem.objects.filter(filter_condition)
+            .live()
+            .public()
+            .select_related("content_type")
+            # DB-level ordering unnecessary; combined list is sorted below
+        )
 
-        pages = WfPage.objects.filter(filter_condition).order_by("title")
+        pages = (
+            WfPage.get_queryset()
+            .filter(filter_condition)
+            .live()
+            .public()
+            .select_related("content_type")
+            # DB-level ordering unnecessary; combined list is sorted below
+        )
 
         combined_queryset_list = list(
             chain(

wf_pages/models.py

@@ -87,7 +87,7 @@ class WfPage(DrupalFields, Page):
     @classmethod
     def get_queryset(cls):
         """Prefetch tags for performance."""
-        return super().get_queryset().prefetch_related("tags__tag")
+        return cls.objects.prefetch_related("tags")
 
     content_panels = Page.content_panels + [
         FieldPanel("body"),