Validation

project/main/apps.py

@@ -13,7 +13,7 @@ class MainConfig(AppConfig):
         """copy template database if db not exists"""
 
         db_fname = settings.DATABASES["default"]["NAME"]
-        template_db_fname = settings.BASE_DIR / "db_template.sqlite3"
+        template_db_fname = settings.BASE_DIR / "db.template.sqlite3"
 
         if os.path.exists(db_fname):
             return

project/main/migrations/0004_alter_person_address_alter_person_city_and_more.py

@@ -0,0 +1,46 @@
+# Generated by Django 5.0.4 on 2024-05-16 05:33
+
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("main", "0003_person"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="person",
+            name="address",
+            field=models.CharField(
+                max_length=100,
+                validators=[
+                    django.core.validators.MinLengthValidator(2),
+                    django.core.validators.MaxLengthValidator(100),
+                ],
+            ),
+        ),
+        migrations.AlterField(
+            model_name="person",
+            name="city",
+            field=models.CharField(
+                max_length=100,
+                validators=[
+                    django.core.validators.MinLengthValidator(2),
+                    django.core.validators.MaxLengthValidator(100),
+                ],
+            ),
+        ),
+        migrations.AlterField(
+            model_name="person",
+            name="name",
+            field=models.CharField(
+                max_length=100,
+                validators=[
+                    django.core.validators.MinLengthValidator(2),
+                    django.core.validators.MaxLengthValidator(100),
+                ],
+            ),
+        ),
+    ]

project/main/models/person.py

@@ -1,10 +1,26 @@
+from django.core.validators import MaxLengthValidator, MinLengthValidator
 from django.db import models
 
 
 class Person(models.Model):
-    name = models.CharField(max_length=100)
-    address = models.CharField(max_length=100)
-    city = models.CharField(max_length=100)
+    name = models.CharField(
+        max_length=100,
+        blank=False,
+        null=False,
+        validators=[MinLengthValidator(2), MaxLengthValidator(100)],
+    )
+    address = models.CharField(
+        max_length=100,
+        blank=False,
+        null=False,
+        validators=[MinLengthValidator(2), MaxLengthValidator(100)],
+    )
+    city = models.CharField(
+        max_length=100,
+        blank=False,
+        null=False,
+        validators=[MinLengthValidator(2), MaxLengthValidator(100)],
+    )
 
     class Meta:
         db_table = "persons"

project/main/templates/main/components/inline_table_row.html

@@ -1,20 +1,30 @@
-{% macro inline_table_row(person, is_editing) %}
-    <tr hx-target="this" hx-swap="outerHTML">
-        {% if is_editing %}
+{% macro inline_table_row(person, is_editing, errors={}) %}
+    {% macro render_input(field_name, value) %}
+        {% set has_error = field_name in errors %}
+        <input
+            class="form-control {% if has_error %}is-invalid{% endif %}"
+            name="{{ field_name }}"
+            value="{{ value }}"
+            {% if has_error %}title="{{ errors[field_name] }}"{% endif %}
+        >
+    {% endmacro %}
+
+    {% if is_editing %}
+        <tr id="person-row-{{ person.pk }}" hx-target="this" hx-swap="outerHTML">
             <td>
-                <input class="form-control" name="name" value="{{ person.name }}">
+                {{ render_input(field_name="name", value=person.name) }}
             </td>
             <td>
-                <input class="form-control" name="address" value="{{ person.address }}">
+                {{ render_input(field_name="address", value=person.address) }}
             </td>
             <td>
-                <input class="form-control" name="city" value="{{ person.city }}">
+                {{ render_input(field_name="city", value=person.city) }}
             </td>
             <td>
                 <button
                     class="btn btn-outline-success"
-                    hx-get="{{ url("table-inline-edit-row", pk=person.pk) }}"
-                    hx-vals='{"action": "save"}'
+                    hx-post="{{ url("table-inline-edit-row", pk=person.pk) }}"
+                    hx-include="#person-row-{{ person.pk }} input"
                 >
                     <i class="bi bi-check-circle-fill"></i>
                 </button>
@@ -26,7 +36,9 @@
                     <i class="bi bi-x-circle-fill"></i>
                 </button>
             </td>
-        {% else %}
+        </tr>
+    {% else %}
+        <tr hx-target="this" hx-swap="outerHTML">
             <td>{{ person.name }}</td>
             <td>{{ person.address }}</td>
             <td>{{ person.city }}</td>
@@ -34,10 +46,11 @@
                 <button
                     class="btn btn-outline-primary"
                     hx-get="{{ url("table-inline-edit-row", pk=person.pk) }}"
+
                 >
                     <i class="bi bi-pencil-square"></i>
                 </button>
             </td>
-        {% endif %}
-    </tr>
+        </tr>
+    {% endif %}
 {% endmacro %}

project/main/templates/main/table_inline_table_row.html

@@ -1,3 +1,3 @@
 {% from "main/components/inline_table_row.html" import inline_table_row %}
 
-{{ inline_table_row(person, is_editing=is_editing) }}
+{{ inline_table_row(person, is_editing=is_editing, errors=errors) }}

project/main/views/table_inline_edit.py

@@ -1,11 +1,21 @@
-from typing import Any
+from typing import Any, Optional
 
-from django.http import Http404
+from django.core.exceptions import ValidationError
+from django.core.handlers.wsgi import WSGIRequest
+from django.http import Http404, HttpResponse
+from django.shortcuts import render
 
 from project.main.models import Person
 from project.main.views.demo_view_base import DemoViewBase
 
 
+def get_person(pk: int) -> Person:
+    try:
+        return Person.objects.get(pk=pk)
+    except Person.DoesNotExist:
+        raise Http404("Person not found")
+
+
 class TableInlineEditView(DemoViewBase):
     template_name = "main/table_inline_edit.html"
     active_section = "table-inline-edit"
@@ -30,17 +40,40 @@ class TableInlineEditRowView(DemoViewBase):
     def get_context_data(self, **kwargs) -> dict[str, Any]:
         context_data = super().get_context_data(**kwargs)
 
-        try:
-            person = Person.objects.get(pk=kwargs.get("pk"))
-        except Person.DoesNotExist:
-            raise Http404("Person not found")
-
-        action = self.request.GET.get("action")
+        person = get_person(pk=kwargs.get("pk"))
+        action = self.request.GET.get("action", "edit")
 
         context_data.update(
             {
                 "person": person,
-                "is_editing": action != "cancel",
+                "is_editing": action == "edit",
             }
         )
         return context_data
+
+    def post(self, request: WSGIRequest, *args, **kwargs) -> HttpResponse:
+        errors: Optional[dict[str, str]] = None
+
+        person = get_person(pk=kwargs.get("pk"))
+        person.name = request.POST.get("name")
+        person.address = request.POST.get("address")
+        person.city = request.POST.get("city")
+
+        try:
+            person.clean_fields()
+        except ValidationError as ex:
+            errors = {key: value[0].message for key, value in ex.error_dict.items()}
+        else:
+            person.save()
+
+        print(errors)
+
+        return render(
+            context={
+                "person": person,
+                "errors": errors,
+                "is_editing": errors is not None,
+            },
+            template_name=self.template_name,
+            request=request,
+        )