From 757ce1f45f147c2c5d9d07ad46e3829526339774 Mon Sep 17 00:00:00 2001
From: "Edgar P. Burkhart" <git@edgarpierre.fr>
Date: Wed, 28 Dec 2022 09:54:39 +0100
Subject: [PATCH] Update views using class based views Updated views for
 transaction Invoice form temporarily removed

---
 nummi/main/models.py                          | 12 ++--
 nummi/main/templates/main/form.html           | 10 ---
 nummi/main/templates/main/form/base.html      | 14 ++++
 nummi/main/templates/main/transaction.html    | 64 -------------------
 .../main/templates/main/transaction_form.html | 22 +++++++
 nummi/main/urls.py                            |  4 +-
 nummi/main/views.py                           | 12 +++-
 7 files changed, 57 insertions(+), 81 deletions(-)
 delete mode 100644 nummi/main/templates/main/form.html
 create mode 100644 nummi/main/templates/main/form/base.html
 delete mode 100644 nummi/main/templates/main/transaction.html
 create mode 100644 nummi/main/templates/main/transaction_form.html

diff --git a/nummi/main/models.py b/nummi/main/models.py
index b2bef5f..11e1349 100644
--- a/nummi/main/models.py
+++ b/nummi/main/models.py
@@ -6,6 +6,7 @@ from django.core.files.storage import Storage
 from django.core.validators import FileExtensionValidator
 from django.db import models
 from django.forms import ModelForm
+from django.urls import reverse
 from django.utils.translation import gettext as _
 
 
@@ -27,7 +28,7 @@ class Category(models.Model):
 
 
 class CategoryForm(ModelForm):
-    template_name = "main/form.html"
+    template_name = "main/form/base.html"
 
     class Meta:
         model = Category
@@ -62,6 +63,9 @@ class Transaction(models.Model):
     def __str__(self):
         return f"{self.date} – {self.name}"
 
+    def get_absolute_url(self):
+        return reverse("transaction", kwargs={"pk": self.pk})
+
     @property
     def invoices(self):
         return Invoice.objects.filter(transaction=self)
@@ -77,7 +81,7 @@ class Transaction(models.Model):
 
 
 class TransactionForm(ModelForm):
-    template_name = "main/form.html"
+    template_name = "main/form/base.html"
 
     class Meta:
         model = Transaction
@@ -123,7 +127,7 @@ class Invoice(models.Model):
 
 
 class InvoiceForm(ModelForm):
-    template_name = "main/form.html"
+    template_name = "main/form/base.html"
     prefix = "invoice"
 
     class Meta:
@@ -263,7 +267,7 @@ class Snapshot(models.Model):
 
 
 class SnapshotForm(ModelForm):
-    template_name = "main/form.html"
+    template_name = "main/form/base.html"
 
     class Meta:
         model = Snapshot
diff --git a/nummi/main/templates/main/form.html b/nummi/main/templates/main/form.html
deleted file mode 100644
index 692f781..0000000
--- a/nummi/main/templates/main/form.html
+++ /dev/null
@@ -1,10 +0,0 @@
-{% if form.non_field_errors %}
-    <ul class='errorlist'>
-        {% for error in form.non_field_errors %}<li>{{ error }}</li>{% endfor %}
-    </ul>
-{% endif %}
-{% for field in form %}
-    {{ field.errors }}
-    <label for="{{ field.id_for_label }}">{{ field.label }}</label>
-    <div>{{ field }}</div>
-{% endfor %}
diff --git a/nummi/main/templates/main/form/base.html b/nummi/main/templates/main/form/base.html
new file mode 100644
index 0000000..b41d1c9
--- /dev/null
+++ b/nummi/main/templates/main/form/base.html
@@ -0,0 +1,14 @@
+{% load i18n %}
+{% block fields %}
+    {% csrf_token %}
+    {% if form.non_field_errors %}
+        <ul class='errorlist'>
+            {% for error in form.non_field_errors %}<li>{{ error }}</li>{% endfor %}
+        </ul>
+    {% endif %}
+    {% for field in form %}
+        {{ field.errors }}
+        <label for="{{ field.id_for_label }}">{{ field.label }}</label>
+        <div>{{ field }}</div>
+    {% endfor %}
+{% endblock %}
diff --git a/nummi/main/templates/main/transaction.html b/nummi/main/templates/main/transaction.html
deleted file mode 100644
index 617a4fd..0000000
--- a/nummi/main/templates/main/transaction.html
+++ /dev/null
@@ -1,64 +0,0 @@
-{% extends "main/base.html" %}
-{% load static %}
-{% load main_extras %}
-{% load i18n %}
-{% block link %}
-    {{ block.super }}
-    <link rel="stylesheet"
-          href="{% static 'main/css/form.css' %}"
-          type="text/css"/>
-    <link rel="stylesheet"
-          href="{% static 'main/css/table.css' %}"
-          type="text/css"/>
-{% endblock %}
-{% block body %}
-    <h1>{{ transaction }}</h1>
-    {% spaceless %}
-        <form id="transaction"
-              action="{% url 'transaction' transaction.id %}"
-              method="post">
-            {% csrf_token %}
-            <input type="hidden" name="form" value="transaction" />
-            {{ form }}
-            {% form_buttons transaction %}
-            </form>
-        {% endspaceless %}
-        <div class="split">
-            {% if transaction.has_invoice %}
-                <div>
-                    <h2>{% translate "Invoices" %}</h2>
-                    <div id="invoices" class="table col1-1-1">
-                        <div class="header">
-                            <strong class="attach center"><i class="fa fa-file"></i></strong>
-                            <strong class="name">{% translate "Name" %}</strong>
-                            <strong class="attach right"><i class="fa fa-trash-can"></i></strong>
-                        </div>
-                        {% for inv in transaction.invoices %}
-                            <div class="invoice">
-                                <a href="{{ inv.file.url }}"><i class="fa-regular fa-file"></i></a>
-                                <a href="{{ inv.file.url }}">{{ inv.name }}</a>
-                                <a href="{% url 'del_invoice' transaction.id inv.id %}"
-                                   class="right"
-                                   onclick="return confirm('{% translate "Delete" %} <{{ inv.name }}> ?')">
-                                    {% translate "Delete" %}
-                                </a>
-                            </div>
-                        {% endfor %}
-                    </div>
-                </div>
-            {% endif %}
-            <div>
-                <h3>{% translate "Add invoice" %}</h3>
-                <form action="{% url 'transaction' transaction.id %}"
-                      method="post"
-                      enctype="multipart/form-data">
-                    {% csrf_token %}
-                    <input type="hidden" name="form" value="invoice" />
-                    {{ invoice_form }}
-                    <div class="buttons">
-                        <input class="btn" type="submit" value="{% translate "Add" %}" />
-                    </div>
-                </form>
-            </div>
-        </div>
-    {% endblock %}
diff --git a/nummi/main/templates/main/transaction_form.html b/nummi/main/templates/main/transaction_form.html
new file mode 100644
index 0000000..92a8616
--- /dev/null
+++ b/nummi/main/templates/main/transaction_form.html
@@ -0,0 +1,22 @@
+{% extends "main/base.html" %}
+{% load static %}
+{% load main_extras %}
+{% load i18n %}
+{% block link %}
+    {{ block.super }}
+    <link rel="stylesheet"
+          href="{% static 'main/css/form.css' %}"
+          type="text/css"/>
+    <link rel="stylesheet"
+          href="{% static 'main/css/table.css' %}"
+          type="text/css"/>
+{% endblock %}
+{% block body %}
+    <h1>{{ transaction }}</h1>
+    {% spaceless %}
+        <form id="transaction" action="" method="post">
+            {{ form }}
+            {% form_buttons form.instance %}
+            </form>
+        {% endspaceless %}
+    {% endblock %}
diff --git a/nummi/main/urls.py b/nummi/main/urls.py
index a72733f..bd15b5a 100644
--- a/nummi/main/urls.py
+++ b/nummi/main/urls.py
@@ -7,8 +7,8 @@ urlpatterns = [
     path("login", views.LoginView.as_view(), name="login"),
     path("logout", views.LogoutView.as_view(), name="logout"),
     path("transactions", views.TransactionListView.as_view(), name="transactions"),
-    path("transaction", views.transaction, name="transaction"),
-    path("transaction/<uuid>", views.transaction, name="transaction"),
+    path("transaction", views.TransactionCreateView.as_view(), name="transaction"),
+    path("transaction/<pk>", views.TransactionUpdateView.as_view(), name="transaction"),
     path("transaction/<uuid>/del", views.del_transaction, name="del_transaction"),
     path("del_invoice/<uuid>/<invoice_id>", views.del_invoice, name="del_invoice"),
     path("invoice/<uuid>", views.invoice, name="invoice"),
diff --git a/nummi/main/views.py b/nummi/main/views.py
index f4cc9da..1191b15 100644
--- a/nummi/main/views.py
+++ b/nummi/main/views.py
@@ -11,7 +11,7 @@ from django.core.paginator import Paginator
 from django.db import models
 from django.http import HttpResponse
 from django.shortcuts import get_object_or_404, redirect, render
-from django.views.generic import ListView
+from django.views.generic import CreateView, ListView, UpdateView
 
 from .models import (
     Category,
@@ -92,6 +92,16 @@ def transaction(request, uuid=None):
     )
 
 
+class TransactionCreateView(LoginRequiredMixin, CreateView):
+    model = Transaction
+    form_class = TransactionForm
+
+
+class TransactionUpdateView(LoginRequiredMixin, UpdateView):
+    model = Transaction
+    form_class = TransactionForm
+
+
 @login_required
 def del_transaction(request, uuid):
     _transaction = get_object_or_404(Transaction, id=uuid)