Implemented frontend for transactions and invoices

nummi/main/templates/main/month/transaction.html

@@ -1,27 +0,0 @@
-{% extends "main/base.html" %}
-{% load static %}
-{% load main_extras %}
-{% load i18n %}
-{% block link %}
-    {{ block.super }}
-    <link rel="stylesheet"
-          href="{% static 'main/css/table.css' %}"
-          type="text/css" />
-    <link rel="stylesheet"
-          href="{% static 'main/css/plot.css' %}"
-          type="text/css" />
-{% endblock %}
-{% block body %}
-    <h2>{% translate "Transactions" %} – {{ month|date:"F Y"|capfirst }}</h2>
-    {% if account %}
-        <p>
-            <a href="{% url "account" account.pk %}">{{ account.icon|remix }}{{ account }}</a>
-        </p>
-    {% endif %}
-    {% if category %}
-        <p>
-            <a href="{% url "category" category.pk %}">{{ category.icon|remix }}{{ category }}</a>
-        </p>
-    {% endif %}
-    {% include "main/table/transaction.html" %}
-{% endblock %}

nummi/transaction/models.py

@@ -67,7 +67,7 @@ class Transaction(UserModel):
         return reverse("transaction", kwargs={"pk": self.pk})
 
     def get_delete_url(self):
-        return reverse("del_transaction", kwargs={"pk": self.pk})
+        return reverse("del_transaction", args=(self.pk,))
 
     @property
     def invoices(self):
@@ -113,14 +113,10 @@ class Invoice(UserModel):
         super().delete(*args, **kwargs)
 
     def get_absolute_url(self):
-        return reverse(
+        return reverse("invoice", args=(self.transaction.pk, self.pk))
-            "invoice", kwargs={"transaction_pk": self.transaction.pk, "pk": self.pk}
-        )
 
     def get_delete_url(self):
-        return reverse(
+        return reverse("del_invoice", args=(self.transaction.pk, self.pk))
-            "del_invoice", kwargs={"transaction_pk": self.transaction.pk, "pk": self.pk}
-        )
 
     class Meta:
         verbose_name = _("Invoice")

nummi/transaction/templates/transaction/transaction_archive_month.html

@@ -0,0 +1,2 @@
+{% extends "transaction/transaction_list.html" %}
+{% block h2 %}{{ month|date:"F Y"|capfirst }}{% endblock %}

nummi/transaction/templates/transaction/transaction_list.html

@@ -21,7 +21,11 @@
           type="text/css" />
 {% endblock %}
 {% block body %}
-    <h2>{% translate "Transactions" %}</h2>
+    <h2>
+        {% block h2 %}
+            {% translate "Transactions" %}
+        {% endblock %}
+    </h2>
     {% if account %}
         <p>
             <a href="{{ account.get_absolute_url }}">{{ account.icon|remix }}{{ account }}</a>
@@ -33,7 +37,9 @@
         </p>
     {% endif %}
     {% if search %}
+        <p>
             <a href="{% url "search" %}">{% translate "Search" %}</a>
+        </p>
     {% endif %}
     {% if transactions %}
         {% include "main/list/pagination.html" %}

nummi/transaction/urls.py

@@ -3,32 +3,32 @@ from django.urls import path
 from . import views
 
 urlpatterns = [
-    path("transactions", views.TransactionListView.as_view(), name="transactions"),
+    path("list", views.TransactionListView.as_view(), name="transactions"),
-    path("transaction", views.TransactionCreateView.as_view(), name="new_transaction"),
-    path(
-        "transaction/<transaction_pk>/invoice",
-        views.InvoiceCreateView.as_view(),
-        name="new_invoice",
-    ),
-    path("transaction/<pk>", views.TransactionUpdateView.as_view(), name="transaction"),
-    path(
-        "transaction/<transaction_pk>/invoice/<pk>",
-        views.InvoiceUpdateView.as_view(),
-        name="invoice",
-    ),
-    path(
-        "transaction/<pk>/delete",
-        views.TransactionDeleteView.as_view(),
-        name="del_transaction",
-    ),
-    path(
-        "transaction/<transaction_pk>/invoice/<pk>/delete",
-        views.InvoiceDeleteView.as_view(),
-        name="del_invoice",
-    ),
     path(
         "history/<int:year>/<int:month>",
         views.TransactionMonthView.as_view(),
         name="transaction_month",
     ),
+    path("new", views.TransactionCreateView.as_view(), name="new_transaction"),
+    path("<transaction>", views.TransactionUpdateView.as_view(), name="transaction"),
+    path(
+        "<transaction>/delete",
+        views.TransactionDeleteView.as_view(),
+        name="del_transaction",
+    ),
+    path(
+        "<transaction>/invoice/new",
+        views.InvoiceCreateView.as_view(),
+        name="new_invoice",
+    ),
+    path(
+        "<transaction>/invoice/<invoice>",
+        views.InvoiceUpdateView.as_view(),
+        name="invoice",
+    ),
+    path(
+        "<transaction>/invoice/<invoice>/delete",
+        views.InvoiceDeleteView.as_view(),
+        name="del_invoice",
+    ),
 ]

nummi/transaction/views.py

@@ -19,7 +19,6 @@ from .models import Invoice, Transaction
 class TransactionCreateView(NummiCreateView):
     model = Transaction
     form_class = TransactionForm
-    template_name = "main/form/transaction.html"
 
     def get_initial(self):
         _queryset = Statement.objects.filter(user=self.request.user)
@@ -43,32 +42,31 @@ class TransactionCreateView(NummiCreateView):
 class InvoiceCreateView(NummiCreateView):
     model = Invoice
     form_class = InvoiceForm
-    template_name = "main/form/invoice.html"
 
     def form_valid(self, form):
         form.instance.transaction = get_object_or_404(
             Transaction.objects.filter(user=self.request.user),
-            pk=self.kwargs["transaction_pk"],
+            pk=self.kwargs["transaction"],
         )
         return super().form_valid(form)
 
     def get_success_url(self):
-        return reverse_lazy("transaction", kwargs={"pk": self.object.transaction.pk})
+        return reverse_lazy("transaction", args=(self.object.transaction.pk,))
 
 
 class TransactionUpdateView(NummiUpdateView):
     model = Transaction
     form_class = TransactionForm
-    template_name = "main/form/transaction.html"
+    pk_url_kwarg = "transaction"
 
 
 class InvoiceUpdateView(NummiUpdateView):
     model = Invoice
     form_class = InvoiceForm
-    template_name = "main/form/invoice.html"
+    pk_url_kwarg = "invoice"
 
     def get_success_url(self):
-        return reverse_lazy("transaction", kwargs={"pk": self.object.transaction.pk})
+        return reverse_lazy("transaction", args=(self.object.transaction.pk,))
 
     def get_queryset(self):
         return (
@@ -76,7 +74,7 @@ class InvoiceUpdateView(NummiUpdateView):
             .get_queryset()
             .filter(
                 transaction=get_object_or_404(
-                    Transaction, pk=self.kwargs["transaction_pk"]
+                    Transaction, pk=self.kwargs["transaction"]
                 )
             )
         )
@@ -84,13 +82,15 @@ class InvoiceUpdateView(NummiUpdateView):
 
 class TransactionDeleteView(NummiDeleteView):
     model = Transaction
+    pk_url_kwarg = "transaction"
 
 
 class InvoiceDeleteView(NummiDeleteView):
     model = Invoice
+    pk_url_kwarg = "invoice"
 
     def get_success_url(self):
-        return reverse_lazy("transaction", kwargs={"pk": self.object.transaction.pk})
+        return reverse_lazy("transaction", args=(self.object.transaction.pk,))
 
     def get_queryset(self):
         return (
@@ -98,7 +98,7 @@ class InvoiceDeleteView(NummiDeleteView):
             .get_queryset()
             .filter(
                 transaction=get_object_or_404(
-                    Transaction, pk=self.kwargs["transaction_pk"]
+                    Transaction, pk=self.kwargs["transaction"]
                 )
             )
         )
@@ -106,12 +106,10 @@ class InvoiceDeleteView(NummiDeleteView):
 
 class TransactionListView(NummiListView):
     model = Transaction
-    template_name = "main/list/transaction.html"
     context_object_name = "transactions"
 
 
 class TransactionMonthView(UserMixin, MonthArchiveView):
-    template_name = "main/month/transaction.html"
     model = Transaction
     date_field = "date"
     context_object_name = "transactions"