Payments add modal form

* adds help texts to add payment form
* adds removing button for payments
* refactors user fetching into BaseForm
* adds generic RemoveModalForm which is intended to be used for every modal form which shall remove something
* adds translations
* removes unused html
* prepares payment amount field to be able to process german inputs like '1.000,50' which is not the international default

compensation/admin.py

@@ -41,7 +41,9 @@ class PaymentAdmin(admin.ModelAdmin):
     list_display = [
         "id",
         "amount",
-        "due_on"
+        "due_on",
+        "created_by",
+        "created_on",
     ]
 
 

compensation/forms.py

@@ -5,10 +5,62 @@ Contact: michel.peltriaux@sgdnord.rlp.de
 Created on: 04.12.20
 
 """
-from konova.forms import BaseForm
+from django import forms
+from django.db import transaction
+from django.utils.translation import gettext_lazy as _
+
+from compensation.models import Payment
+from konova.forms import BaseForm, BaseModalForm
 
 
 class NewCompensationForm(BaseForm):
 
     def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
+
+
+class NewPaymentForm(BaseModalForm):
+    amount = forms.DecimalField(
+        min_value=0.00,
+        decimal_places=2,
+        label=_("Amount"),
+        label_suffix=_(""),
+        help_text=_("Amount in Euro"),
+        localize=True,
+    )
+    due = forms.DateField(
+        label=_("Due on"),
+        label_suffix=_(""),
+        help_text=_("Due on which date"),
+        widget=forms.DateInput(
+            attrs={
+                "type": "date",
+                "data-provide": "datepicker",
+            },
+            format="%d.%m.%Y"
+        )
+    )
+    transfer_note = forms.CharField(
+        max_length=1000,
+        required=False,
+        label_suffix=_(""),
+        label=_("Transfer note"),
+        help_text=_("Note for money transfer")
+    )
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.intervention = self.instance
+        self.form_title = _("Payment")
+        self.form_caption = _("Add a payment for intervention '{}'").format(self.intervention.title)
+
+    def save(self):
+        with transaction.atomic():
+            pay = Payment.objects.create(
+                created_by=self.user,
+                amount=self.cleaned_data.get("amount", -1),
+                due_on=self.cleaned_data.get("due", None),
+                comment=self.cleaned_data.get("transfer_note", None),
+                intervention=self.intervention,
+            )
+            return pay

compensation/urls.py

@@ -19,10 +19,10 @@ urlpatterns = [
     path('<id>/remove', remove_view, name='remove'),
 
     # Payment
-    path('pay/new', new_view, name='pay-new'),
+    path('pay/<intervention_id>/new', new_payment_view, name='pay-new'),
     path('pay/<id>', open_view, name='pay-open'),
     path('pay/<id>/edit', edit_view, name='pay-edit'),
-    path('pay/<id>/remove', remove_view, name='pay-remove'),
+    path('pay/<id>/remove', payment_remove_view, name='pay-remove'),
 
     # Eco-account
     path("acc/", account_index_view, name="acc-index"),

compensation/views.py

@@ -1,11 +1,15 @@
 from django.contrib.auth.decorators import login_required
 from django.http import HttpRequest
-from django.shortcuts import render
+from django.shortcuts import render, get_object_or_404
+from django.utils.translation import gettext_lazy as _
 
-from compensation.models import Compensation, EcoAccount
+from compensation.forms import NewPaymentForm
+from compensation.models import Compensation, EcoAccount, Payment
 from compensation.tables import CompensationTable, EcoAccountTable
+from intervention.models import Intervention
 from konova.contexts import BaseContext
 from konova.decorators import *
+from konova.forms import RemoveModalForm
 
 
 @login_required
@@ -108,3 +112,79 @@ def account_open_view(request: HttpRequest, id: str):
 def account_remove_view(request: HttpRequest, id: str):
     # ToDo
     pass
+
+
+@login_required
+def new_payment_view(request: HttpRequest, intervention_id: str):
+    """ Renders a modal view for adding new payments
+
+    Args:
+        request (HttpRequest): The incoming request
+        intervention_id (str): The intervention's id for which a new payment shall be added
+
+    Returns:
+
+    """
+    template = "modal/modal_form.html"
+    intervention = get_object_or_404(Intervention, id=intervention_id)
+    form = NewPaymentForm(request.POST or None, instance=intervention, user=request.user)
+    if request.method == "POST":
+        if form.is_valid():
+            payment = form.save()
+            messages.success(
+                request,
+                _("Payment added")
+            )
+            return redirect(request.META.get("HTTP_REFERER", "home"))
+        else:
+            messages.info(
+                request,
+                _("There was an error on this form.")
+            )
+            return redirect(request.META.get("HTTP_REFERER", "home"))
+    elif request.method == "GET":
+        context = {
+            "form": form,
+        }
+        context = BaseContext(request, context).context
+        return render(request, template, context)
+    else:
+        raise NotImplementedError
+
+
+@login_required
+def payment_remove_view(request: HttpRequest, id: str):
+    """ Renders a modal view for adding new payments
+
+    Args:
+        request (HttpRequest): The incoming request
+        id (str): The payment's id
+
+    Returns:
+
+    """
+    template = "modal/modal_form.html"
+    payment = get_object_or_404(Payment, id=id)
+    form = RemoveModalForm(request.POST or None, instance=payment, user=request.user)
+    if request.method == "POST":
+        if form.is_valid():
+            form.save()
+            messages.success(
+                request,
+                _("Payment removed")
+            )
+            return redirect(request.META.get("HTTP_REFERER", "home"))
+        else:
+            messages.info(
+                request,
+                _("There was an error on this form.")
+            )
+            return redirect(request.META.get("HTTP_REFERER", "home"))
+    elif request.method == "GET":
+        context = {
+            "form": form,
+        }
+        context = BaseContext(request, context).context
+        return render(request, template, context)
+    else:
+        raise NotImplementedError

intervention/templates/intervention/detail-view.html

@@ -1,5 +1,5 @@
 {% extends 'base.html' %}
-{% load i18n static fontawesome_5 %}
+{% load i18n l10n static fontawesome_5 humanize %}
 
 {% block head %}
 
@@ -129,7 +129,7 @@
                     <tr>
                         <th scope="row">{% trans 'Last modified' %}</th>
                         <td class="align-middle">
-                            {{intervention.created_on|default_if_none:""}}
+                            {{intervention.created_on|default_if_none:""|naturalday}}
                             <br>
                             {% trans 'by' %}
                             {{intervention.created_by|default_if_none:""}}
@@ -171,7 +171,7 @@
                         </div>
                     </div>
                 </div>
-                <div class="card-body">
+                <div class="card-body scroll-300">
                     <table class="table table-hover">
                         <thead>
                             <tr>
@@ -211,26 +211,30 @@
                         </div>
                         <div class="col-sm-6">
                             <div class="d-flex justify-content-end">
-                                <a href="{% url 'compensation:pay-new' %}" title="{% trans 'Add new payment' %}">
+                                <button class="btn btn-outline-default btn-modal" data-form-url="{% url 'compensation:pay-new' intervention.id %}" title="{% trans 'Add new payment' %}">
-                                    <button class="btn btn-outline-default">
+                                    {% fa5_icon 'plus' %}
-                                        {% fa5_icon 'plus' %}
+                                    {% fa5_icon 'money-bill-wave' %}
-                                        {% fa5_icon 'money-bill-wave' %}
+                                </button>
-                                    </button>
-                                </a>
                             </div>
                         </div>
                     </div>
                 </div>
-                <div class="card-body">
+                <div class="card-body scroll-300">
                     <table class="table table-hover">
                         <thead>
                             <tr>
                                 <th scope="col">
                                     {% trans 'Amount' %}
                                 </th>
+                                <th scope="col">
+                                    {% trans 'Due on' %}
+                                </th>
                                 <th scope="col">
                                     {% trans 'Transfer comment' %}
                                 </th>
+                                <th scope="col">
+                                    {% trans 'Action' %}
+                                </th>
                             </tr>
                         </thead>
                         <tbody>
@@ -238,10 +242,16 @@
                             <tr>
                                 <td class="align-middle">
                                     <a href="{% url 'compensation:pay-open' pay.id %}">
-                                            {{ pay.amount }}
+                                            {{ pay.amount|floatformat:2 }} €
                                     </a>
                                 </td>
+                                <td class="align-middle">{{ pay.due_on }}</td>
                                 <td class="align-middle">{{ pay.comment }}</td>
+                                <td>
+                                    <button data-form-url="{% url 'compensation:pay-remove' pay.id %}" class="btn btn-default btn-modal" title="{% trans 'Remove payment' %}">
+                                        {% fa5_icon 'trash' %}
+                                    </button>
+                                </td>
                             </tr>
                             {% endfor %}
                         </tbody>
@@ -264,7 +274,7 @@
                         <div class="col-sm-6">
                             <div class="d-flex justify-content-end">
                                 <a href="{% url 'doc-new' %}" title="{% trans 'Add new document' %}">
-                                    <button class="btn btn-outline-default">
+                                    <button class="btn btn-outline-default ">
                                         {% fa5_icon 'plus' %}
                                         {% fa5_icon 'file' %}
                                     </button>
@@ -273,7 +283,7 @@
                         </div>
                     </div>
                 </div>
-                <div class="card-body">
+                <div class="card-body scroll-300">
                     <table class="table table-hover">
                         <thead>
                             <tr>
@@ -298,7 +308,7 @@
                                 </td>
                                 <td class="align-middle">{{ doc.comment }}</td>
                                 <td>
-                                    <button data-form-url="{% url 'doc-remove' doc.id %}" class="btn btn-default del-btn" title="{% trans 'Remove document' %}">
+                                    <button data-form-url="{% url 'doc-remove' doc.id %}" class="btn btn-default btn-modal" title="{% trans 'Remove document' %}">
                                         {% fa5_icon 'trash' %}
                                     </button>
                                 </td>
@@ -311,7 +321,7 @@
         </div>
     </div>
 
-{% with 'del-btn' as btn_class %}
+{% with 'btn-modal' as btn_class %}
     {% include 'modal/modal_form_script.html' %}
 {% endwith %}
 

intervention/views.py

@@ -4,7 +4,7 @@ from django.utils.translation import gettext_lazy as _
 from django.http import HttpRequest
 from django.shortcuts import render, get_object_or_404
 
-from intervention.forms import NewInterventionForm, EditInterventionForm, OpenInterventionForm
+from intervention.forms import NewInterventionForm, EditInterventionForm
 from intervention.models import Intervention
 from intervention.tables import InterventionTable
 from konova.contexts import BaseContext

konova/forms.py

@@ -8,12 +8,11 @@ Created on: 16.11.20
 
 from abc import abstractmethod
 
-from bootstrap_modal_forms.forms import BSModalModelForm, BSModalForm
+from bootstrap_modal_forms.forms import BSModalForm
 from django import forms
 from django.contrib.auth.models import User
 from django.contrib.gis.forms import GeometryField, OSMWidget
 from django.contrib.gis.geos import Polygon
-from django.urls import reverse
 from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 
@@ -30,6 +29,7 @@ class BaseForm(forms.Form):
 
     def __init__(self, *args, **kwargs):
         self.instance = kwargs.pop("instance", None)
+        self.user = kwargs.pop("user", None)
         super().__init__(*args, **kwargs)
 
     @abstractmethod
@@ -99,6 +99,13 @@ class RemoveForm(BaseForm):
         return self.object_to_remove
 
 
+class BaseModalForm(BaseForm, BSModalForm):
+    """ A specialzed form class for modal form handling
+
+    """
+    is_modal_form = True
+
+
 class SimpleGeomForm(BaseForm):
     """ A geometry form for rendering geometry read-only using a widget
 
@@ -131,7 +138,35 @@ class SimpleGeomForm(BaseForm):
         self.area = geom.area
 
 
-class RemoveDocumentForm(BaseForm, BSModalForm):
+class RemoveModalForm(BaseModalForm):
+    """ Generic removing modal form
+
+    Can be used for anything, where removing shall be confirmed by the user a second time.
+
+    """
+    confirm = forms.BooleanField(
+        label=_("Confirm"),
+        label_suffix=_(""),
+        widget=forms.CheckboxInput(),
+        required=True,
+    )
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.form_title = _("Remove")
+        self.form_caption = _("Are you sure?")
+
+    def save(self):
+        if hasattr(self.instance, "deleted_on"):
+            self.instance.deleted_on = timezone.now()
+            self.instance.deleted_by = self.user
+            self.instance.save()
+        else:
+            # If the class does not provide restorable delete functionality, we must delete the entry finally
+            self.instance.delete()
+
+
+class RemoveDocumentForm(BaseModalForm):
     confirm = forms.BooleanField(
         label=_("Confirm"),
         label_suffix=_(""),

konova/static/css/konova.css

@@ -168,7 +168,7 @@ a {
 }
 
 input:focus, textarea:focus, select:focus{
-    border: 1px solid var(--rlp-red) !important;
+    border-color: var(--rlp-red) !important;
     box-shadow: 0 0 3px var(--rlp-red) !important;
     -moz-box-shadow: 0 0 3px var(--rlp-red) !important;
     -webkit-box-shadow: 0 0 3px var(--rlp-red) !important;
@@ -194,4 +194,13 @@ input:focus, textarea:focus, select:focus{
 .page-item.active > .page-link{
     background-color: var(--rlp-red);
     border-color: var(--rlp-red);
+}
+
+.label-required{
+    color: var(--rlp-red);
+}
+
+.scroll-300{
+    max-height: 300px;
+    overflow: auto;
 }

konova/sub_settings/django_settings.py

@@ -53,6 +53,7 @@ INSTALLED_APPS = [
     'django.contrib.messages',
     'django.contrib.staticfiles',
     'django.contrib.gis',
+    'django.contrib.humanize',
     'simple_sso.sso_server',
     'django_tables2',
     'bootstrap_modal_forms',
@@ -143,9 +144,12 @@ AUTH_PASSWORD_VALIDATORS = [
 # Internationalization
 # https://docs.djangoproject.com/en/3.1/topics/i18n/
 
-LANGUAGE_CODE = 'en-us'
+LANGUAGE_CODE = 'de'
+
+USE_THOUSAND_SEPARATOR = True
 
 DEFAULT_DATE_TIME_FORMAT = '%d.%m.%Y %H:%M:%S'
+DATE_FORMAT = '%d.%m.%Y'
 
 TIME_ZONE = 'Europe/Berlin'
 

konova/templates/konova/form.html

@@ -3,6 +3,6 @@
 
 {% block body %}
 <div class="column">
-    {% include 'generic_table_form.html' %}
+    {% include 'table/generic_table_form.html' %}
 </div>
 {% endblock %}

locale/de/LC_MESSAGES/django.po

@@ -3,15 +3,16 @@
 # This file is distributed under the same license as the PACKAGE package.
 # FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
 #
+#: compensation/forms.py:26 compensation/forms.py:32 compensation/forms.py:45
 #: intervention/filters.py:25 intervention/filters.py:31
 #: intervention/filters.py:38 intervention/filters.py:39 konova/forms.py:73
-#: konova/forms.py:137 user/forms.py:38
+#: konova/forms.py:149 konova/forms.py:172 user/forms.py:38
 #, fuzzy
 msgid ""
 msgstr ""
 "Project-Id-Version: PACKAGE VERSION\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2021-07-23 18:26+0200\n"
+"POT-Creation-Date: 2021-07-26 10:44+0200\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"
@@ -21,6 +22,39 @@ msgstr ""
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=2; plural=(n != 1);\n"
 
+#: compensation/forms.py:25
+#: intervention/templates/intervention/detail-view.html:227
+msgid "Amount"
+msgstr "Betrag"
+
+#: compensation/forms.py:27
+msgid "Amount in Euro"
+msgstr "Betrag in Euro"
+
+#: compensation/forms.py:31
+msgid "Due on"
+msgstr "Fällig am"
+
+#: compensation/forms.py:33
+msgid "Due on which date"
+msgstr "Zahlung wird an diesem Datum erwartet"
+
+#: compensation/forms.py:46
+msgid "Transfer note"
+msgstr "Verwendungszweck"
+
+#: compensation/forms.py:47
+msgid "Note for money transfer"
+msgstr "Verwendungszweck für Überweisung"
+
+#: compensation/forms.py:55
+msgid "Payment"
+msgstr "Zahlung"
+
+#: compensation/forms.py:56
+msgid "Add a payment for intervention '{}'"
+msgstr "Neue Ersatzzahlung zu Eingriff '{}' hinzufügen"
+
 #: compensation/tables.py:18 compensation/tables.py:71 intervention/forms.py:26
 #: intervention/tables.py:23
 #: intervention/templates/intervention/detail-view.html:179
@@ -31,7 +65,7 @@ msgstr "Kennung"
 #: intervention/tables.py:28
 #: intervention/templates/intervention/detail-view.html:62
 #: intervention/templates/intervention/detail-view.html:182
-#: intervention/templates/intervention/detail-view.html:281
+#: intervention/templates/intervention/detail-view.html:287
 msgid "Title"
 msgstr "Bezeichnung"
 
@@ -72,6 +106,18 @@ msgstr "Lösche {}"
 msgid "Eco Accounts"
 msgstr "Ökokonten"
 
+#: compensation/views.py:136
+msgid "Payment added"
+msgstr "Zahlung hinzugefügt"
+
+#: compensation/views.py:142 compensation/views.py:180 konova/views.py:137
+msgid "There was an error on this form."
+msgstr "Es gab einen Fehler im Formular."
+
+#: compensation/views.py:174
+msgid "Payment removed"
+msgstr "Zahlung gelöscht"
+
 #: intervention/filters.py:24
 msgid "Show all"
 msgstr "Alle anzeigen"
@@ -291,31 +337,32 @@ msgstr "Ersatzzahlungen"
 msgid "Add new payment"
 msgstr "Neue Zahlung hinzufügen"
 
-#: intervention/templates/intervention/detail-view.html:229
+#: intervention/templates/intervention/detail-view.html:230
-msgid "Amount"
-msgstr "Betrag"
-
-#: intervention/templates/intervention/detail-view.html:232
 msgid "Transfer comment"
 msgstr "Verwendungszweck"
 
-#: intervention/templates/intervention/detail-view.html:261
+#: intervention/templates/intervention/detail-view.html:233
-msgid "Documents"
+#: intervention/templates/intervention/detail-view.html:293
-msgstr "Dokumente"
-
-#: intervention/templates/intervention/detail-view.html:266
-msgid "Add new document"
-msgstr "Neues Dokument hinzufügen"
-
-#: intervention/templates/intervention/detail-view.html:284
-msgid "Comment"
-msgstr "Kommentar"
-
-#: intervention/templates/intervention/detail-view.html:287
 msgid "Action"
 msgstr "Aktionen"
 
-#: intervention/templates/intervention/detail-view.html:301 konova/forms.py:145
+#: intervention/templates/intervention/detail-view.html:247
+msgid "Remove payment"
+msgstr "Zahlung entfernen"
+
+#: intervention/templates/intervention/detail-view.html:267
+msgid "Documents"
+msgstr "Dokumente"
+
+#: intervention/templates/intervention/detail-view.html:272
+msgid "Add new document"
+msgstr "Neues Dokument hinzufügen"
+
+#: intervention/templates/intervention/detail-view.html:290
+msgid "Comment"
+msgstr "Kommentar"
+
+#: intervention/templates/intervention/detail-view.html:307 konova/forms.py:180
 msgid "Remove document"
 msgstr "Dokument löschen"
 
@@ -357,19 +404,23 @@ msgstr "Hierfür müssen Sie einer anderen Nutzergruppe angehören!"
 msgid "Not editable"
 msgstr "Nicht editierbar"
 
-#: konova/forms.py:72 konova/forms.py:136
+#: konova/forms.py:72 konova/forms.py:148 konova/forms.py:171
 msgid "Confirm"
 msgstr "Bestätige"
 
-#: konova/forms.py:84
+#: konova/forms.py:84 konova/forms.py:156
 msgid "Remove"
-msgstr "Entferne"
+msgstr "Löschen"
 
 #: konova/forms.py:86
 msgid "You are about to remove {} {}"
 msgstr "Sie sind dabei {} {} zu löschen"
 
-#: konova/forms.py:146
+#: konova/forms.py:157
+msgid "Are you sure?"
+msgstr ""
+
+#: konova/forms.py:181
 msgid "This will remove '{}'. Are you sure?"
 msgstr "Hiermit wird '{}' gelöscht. Sind Sie sicher?"
 
@@ -425,14 +476,10 @@ msgstr "Ökokonto"
 msgid "Withdraw"
 msgstr "Abbuchen"
 
-#: konova/views.py:133
+#: konova/views.py:131
 msgid "Document '{}' deleted"
 msgstr "Dokument '{}' gelöscht"
 
-#: konova/views.py:139
-msgid "There was an error on this form."
-msgstr "Es gab einen Fehler im Formular."
-
 #: news/templates/news/dashboard-news.html:12 news/templates/news/index.html:19
 msgid "Published on"
 msgstr "Veröffentlicht am"
@@ -461,19 +508,7 @@ msgstr ""
 msgid "Contact"
 msgstr "Kontakt"
 
-#: templates/generic_table_form.html:37
+#: templates/modal/modal_form.html:24
-msgid "Fields with * are required."
-msgstr "* sind Pflichtfelder."
-
-#: templates/generic_table_form.html:41
-msgid "Cancel"
-msgstr "Abbrechen"
-
-#: templates/generic_table_form.html:45
-msgid "Save"
-msgstr "Speichern"
-
-#: templates/modal/modal_form.html:30
 msgid "Continue"
 msgstr "Weiter"
 
@@ -549,6 +584,18 @@ msgstr ""
 msgid "Apply filter"
 msgstr "Filter anwenden"
 
+#: templates/table/generic_table_form.html:23
+msgid "Cancel"
+msgstr "Abbrechen"
+
+#: templates/table/generic_table_form.html:27
+msgid "Save"
+msgstr "Speichern"
+
+#: templates/table/generic_table_form_body.html:20
+msgid "Fields with * are required."
+msgstr "* sind Pflichtfelder."
+
 #: user/forms.py:23
 msgid "Notifications"
 msgstr "Benachrichtigungen"
@@ -1816,9 +1863,6 @@ msgstr ""
 #~ msgid "Last login on"
 #~ msgstr "Zuletzt eingeloggt am"
 
-#~ msgid "Add new intervention"
-#~ msgstr "Neuen Eingriff hinzufügen"
-
 #~ msgid "Delete intervention"
 #~ msgstr "Eingriff löschen"
 
@@ -1904,9 +1948,6 @@ msgstr ""
 #~ "Eingriffe müssen zu einem Vorgang gehören. Bitte geben SIe die fehlenden "
 #~ "Daten für den Vorgang ein."
 
-#~ msgid "Intervention {} removed"
-#~ msgstr "Eingriff {} gelöscht"
-
 #~ msgid "You are working as"
 #~ msgstr "Sie arbeiten gerade als "
 

templates/base.html

@@ -1,5 +1,5 @@
 <!DOCTYPE html>
-{% load static i18n fontawesome_5 bootstrap4 %}
+{% load static i18n l10n fontawesome_5 bootstrap4 %}
 <html lang="{{ language }}">
 <head>
     <meta charset="UTF-8">

templates/modal/modal_form.html

@@ -1,4 +1,4 @@
-{% load i18n %}
+{% load i18n l10n %}
 {% comment %}
     A generic modal form template which is based on django-bootstrap-modal-forms package
     https://pypi.org/project/django-bootstrap-modal-forms/
@@ -15,16 +15,10 @@
     </div>
 
     <div class="modal-body">
-        <article class="mb-5">{{form.form_caption}}</article>
+        <article>
-        {% for field in form %}
+            {{ form.form_caption }}
-        <div class="form-group{% if field.errors %} invalid{% endif %}">
+        </article>
-            <label for="{{ field.id_for_label }}">{{ field.label }}</label>
+        {% include 'table/generic_table_form_body.html' %}
-            {{ field }}
-            {% for error in field.errors %}
-            <p class="help-block">{{ error }}</p>
-            {% endfor %}
-        </div>
-        {% endfor %}
     </div>
     <div class="modal-footer">
         <button type="submit" class="btn btn-default">{% trans 'Continue' %}</button>

templates/table/generic_table_form.html

@@ -16,25 +16,7 @@
     {% endif %}
     <form method="post" action="{{ form.action_url }}">
         {% csrf_token %}
-        <table class="table">
+        {% include 'table/generic_table_form_body.html' %}
-            <tbody>
-                {% for field in form %}
-                <tr title="{{ field.help_text }}" class="{% if field.errors %}error{% endif %}">
-                    <th scope="row" class="col-sm-3">
-                        <div>{{ field.label }}<span class="label-required">{% if field.field.required %}*{% endif %}</span></div>
-                        <small>{{ field.help_text }}</small>
-                    </th>
-                    <td class="col-sm-9">
-                        {{ field }}
-                        {% for error in field.errors %}
-                        <strong>{{ error }}</strong>
-                        {% endfor %}
-                    </td>
-                </tr>
-                {% endfor %}
-            </tbody>
-        </table>
-        <small>{% trans 'Fields with * are required.' %}</small>
         <div class="row">
             <div class="col-md">
                 <a href="{{ form.cancel_redirect }}">

templates/table/generic_table_form_body.html

@@ -0,0 +1,20 @@
+{% load i18n %}
+        <table class="table">
+            <tbody>
+                {% for field in form %}
+                <tr title="{{ field.help_text }}" class="{% if field.errors %}error{% endif %}">
+                    <th scope="row" class="col-sm-3">
+                        <div>{{ field.label }}<span class="label-required">{% if field.field.required %}*{% endif %}</span></div>
+                        <small>{{ field.help_text }}</small>
+                    </th>
+                    <td class="col-sm-9">
+                        {{ field }}
+                        {% for error in field.errors %}
+                        <strong>{{ error }}</strong>
+                        {% endfor %}
+                    </td>
+                </tr>
+                {% endfor %}
+            </tbody>
+        </table>
+        <small>{% trans 'Fields with * are required.' %}</small>

user/templates/user/notifications.html

@@ -1,5 +1,5 @@
 {% extends 'base.html' %}
 
 {% block body %}
-    {% include 'generic_table_form.html' %}
+    {% include 'table/generic_table_form.html' %}
 {% endblock %}