Use templatetag for transaction_table

nummi/category/templates/category/category_detail.html

@@ -1,5 +1,5 @@
 {% extends "main/base.html" %}
-{% load i18n main_extras history_extras %}
+{% load i18n main_extras history_extras transaction_extras %}
 {% block title %}
   {{ category }} – {{ block.super }}
 {% endblock title %}
@@ -15,7 +15,8 @@
   </p>
   <section>
     <h3>{% translate "Transactions" %}</h3>
-    {% include "transaction/transaction_table.html" %}
+    {% url "category_transactions" category.id as t_url %}
+    {% transaction_table category.transaction_set.all n_max=8 transactions_url=t_url %}
   </section>
   <section>
     <h3>{% translate "History" %}</h3>

nummi/category/views.py

@@ -1,5 +1,4 @@
 from django.shortcuts import get_object_or_404
-from django.urls import reverse_lazy
 from main.views import (
     NummiCreateView,
     NummiDeleteView,
@@ -28,19 +27,6 @@ class CategoryDetailView(NummiDetailView):
     pk_url_kwarg = "category"
     context_object_name = "category"
 
-    def get_context_data(self, **kwargs):
-        _max = 8
-        data = super().get_context_data(**kwargs)
-        category = data["object"]
-
-        data["transactions"] = category.transaction_set.all()[:_max]
-        if len(data["transactions"]) == _max:
-            data["transactions_url"] = reverse_lazy(
-                "category_transactions", args=(category.pk,)
-            )
-
-        return data
-
 
 class CategoryDeleteView(NummiDeleteView):
     model = Category

nummi/statement/templates/statement/statement_detail.html

@@ -1,15 +1,15 @@
 {% extends "main/base.html" %}
-{% load i18n main_extras statement_extras category %}
+{% load i18n main_extras statement_extras transaction_extras category %}
 {% block title %}
   {{ statement }}
   – Nummi
-{% endblock %}
+{% endblock title %}
 {% block link %}
   {{ block.super }}
   {% css "main/css/form.css" %}
   {% css "main/css/table.css" %}
   {% css "main/css/plot.css" %}
-{% endblock %}
+{% endblock link %}
 {% block body %}
   <h2>{{ statement }}</h2>
   <p>
@@ -38,10 +38,12 @@
   </div>
   <section>
     <h3>{% translate "Transactions" %}</h3>
-    {% include "transaction/transaction_table.html" %}
+    {% url "statement_transactions" statement.id as t_url %}
+    {% url "new_transaction" statement=statement.id as nt_url %}
+    {% transaction_table statement.transaction_set.all n_max=8 transactions_url=t_url new_transaction_url=nt_url %}
   </section>
   <section>
     <h3>{% translate "Categories" %}</h3>
     {% category_plot transactions budget=False statement=object %}
   </section>
-{% endblock %}
+{% endblock body %}

nummi/transaction/templates/transaction/transaction_list.html

@@ -1,12 +1,12 @@
 {% extends "main/list.html" %}
-{% load i18n %}
+{% load i18n transaction_extras %}
 {% block name %}
   {% translate "Transactions" %}
-{% endblock %}
+{% endblock name %}
 {% block h2 %}
   {% translate "Transactions" %}
-{% endblock %}
+{% endblock h2 %}
 {% block table %}
-  {% url "new_transaction" as new_transaction_url %}
+  {% url "new_transaction" as nt_url %}
-  {% include "transaction/transaction_table.html" %}
+  {% transaction_table transactions new_transaction_url=nt_url %}
-{% endblock %}
+{% endblock table %}

nummi/transaction/templatetags/transaction_extras.py

@@ -10,6 +10,18 @@ from ..utils import ac_url
 register = template.Library()
 
 
+@register.inclusion_tag("transaction/transaction_table.html")
+def transaction_table(transactions, **kwargs):
+    if (n_max := kwargs.get("n_max")) is not None:
+        if transactions.count() <= n_max:
+            del kwargs["transactions_url"]
+        transactions = transactions[:n_max]
+
+    return kwargs | {
+        "transactions": transactions,
+    }
+
+
 @register.simple_tag(takes_context=True)
 def month_url(context, month, cls="", fmt="Y-m"):
     url_name, url_params = ac_url(