list editions

games/editionviews.py

@@ -0,0 +1,109 @@
+from typing import Any
+
+from django.contrib.auth.decorators import login_required
+from django.core.paginator import Paginator
+from django.http import HttpRequest, HttpResponse
+from django.shortcuts import get_object_or_404, redirect, render
+from django.template.loader import render_to_string
+from django.urls import reverse
+
+from common.utils import truncate_with_popover
+from games.forms import EditionForm
+from games.models import Edition
+from games.views import dateformat
+
+
+@login_required
+def list_editions(request: HttpRequest) -> HttpResponse:
+    context: dict[Any, Any] = {}
+    page_number = request.GET.get("page", 1)
+    limit = request.GET.get("limit", 10)
+    editions = Edition.objects.order_by("-created_at")
+    page_obj = None
+    if int(limit) != 0:
+        paginator = Paginator(editions, limit)
+        page_obj = paginator.get_page(page_number)
+        editions = page_obj.object_list
+
+    context = {
+        "title": "Manage editions",
+        "page_obj": page_obj or None,
+        "elided_page_range": (
+            page_obj.paginator.get_elided_page_range(
+                page_number, on_each_side=1, on_ends=1
+            )
+            if page_obj
+            else None
+        ),
+        "data": {
+            "columns": [
+                "Game",
+                "Name",
+                "Sort Name",
+                "Platform",
+                "Year",
+                "Wikidata",
+                "Created",
+                "Actions",
+            ],
+            "rows": [
+                [
+                    truncate_with_popover(edition.game.name),
+                    truncate_with_popover(
+                        edition.name
+                        if edition.game.name != edition.name
+                        else "(identical)"
+                    ),
+                    truncate_with_popover(
+                        edition.sort_name
+                        if edition.sort_name is not None
+                        and edition.game.name != edition.sort_name
+                        else "(identical)"
+                    ),
+                    truncate_with_popover(str(edition.platform)),
+                    edition.year_released,
+                    edition.wikidata,
+                    edition.created_at.strftime(dateformat),
+                    render_to_string(
+                        "components/button_group_sm.html",
+                        {
+                            "buttons": [
+                                {
+                                    "href": reverse("edit_edition", args=[edition.pk]),
+                                    "text": "Edit",
+                                },
+                                {
+                                    "href": reverse(
+                                        "delete_edition", args=[edition.pk]
+                                    ),
+                                    "text": "Delete",
+                                    "color": "red",
+                                },
+                            ]
+                        },
+                    ),
+                ]
+                for edition in editions
+            ],
+        },
+    }
+    return render(request, "list_purchases.html", context)
+
+
+@login_required
+def edit_device(request: HttpRequest, edition_id: int = 0) -> HttpResponse:
+    edition = get_object_or_404(Edition, id=edition_id)
+    form = EditionForm(request.POST or None, instance=edition)
+    if form.is_valid():
+        form.save()
+        return redirect("list_editions")
+
+    context: dict[str, Any] = {"form": form, "title": "Edit edition"}
+    return render(request, "add.html", context)
+
+
+@login_required
+def delete_edition(request: HttpRequest, edition_id: int) -> HttpResponse:
+    edition = get_object_or_404(Edition, id=edition_id)
+    edition.delete()
+    return redirect("list_editions")

games/migrations/0036_alter_edition_platform.py

@@ -0,0 +1,19 @@
+# Generated by Django 5.1 on 2024-08-11 16:48
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('games', '0035_alter_session_device'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='edition',
+            name='platform',
+            field=models.ForeignKey(blank=True, default=None, null=True, on_delete=django.db.models.deletion.SET_DEFAULT, to='games.platform'),
+        ),
+    ]

games/models.py

@@ -39,7 +39,7 @@ class Edition(models.Model):
     name = models.CharField(max_length=255)
     sort_name = models.CharField(max_length=255, null=True, blank=True, default=None)
     platform = models.ForeignKey(
-        Platform, on_delete=models.CASCADE, null=True, blank=True, default=None
+        Platform, on_delete=models.SET_DEFAULT, null=True, blank=True, default=None
     )
     year_released = models.IntegerField(null=True, blank=True, default=None)
     wikidata = models.CharField(max_length=50, null=True, blank=True, default=None)

games/templates/components/popover.html

@@ -1,6 +1,3 @@
-<!-- needs data-popover-target on triggering block -->
-<!-- id -->
-<!-- children -->
 <div data-popover
      id="{{ id }}"
      role="tooltip"

games/templates/components/table_row.html

@@ -1,17 +1,8 @@
 {% fragment as default_content %}
-{% load randomid %}
 {% for td in data %}
     {% if forloop.first %}
         <th scope="row"
-            class="px-6 py-4 font-medium text-gray-900 whitespace-nowrap dark:text-white min-w-30char">
-            {% randomid td as th_popover_id %}
-            <span data-popover-target="{{ th_popover_id }}">{{ td|truncatechars:30 }}</span>
-            {% if td|length > 30 %}
-                {% #popover id=th_popover_id %}
-                {{ td }}
-                {% /popover %}
-            {% endif %}
-        </th>
+            class="px-6 py-4 font-medium text-gray-900 whitespace-nowrap dark:text-white">{{ td }}</th>
     {% else %}
         {% #table_td %}
         {{ td }}

games/templates/navbar.html

@@ -103,7 +103,7 @@
                                    class="block px-4 py-2 hover:bg-gray-100 dark:hover:bg-gray-600 dark:hover:text-white">Games</a>
                             </li>
                             <li>
-                                <a href="{% url 'add_edition' %}"
+                                <a href="{% url 'list_editions' %}"
                                    class="block px-4 py-2 hover:bg-gray-100 dark:hover:bg-gray-600 dark:hover:text-white">Editions</a>
                             </li>
                             <li>

games/urls.py

@@ -2,6 +2,7 @@ from django.urls import path
 
 from games import (
     deviceviews,
+    editionviews,
     gameviews,
     platformviews,
     purchaseviews,
@@ -24,6 +25,12 @@ urlpatterns = [
         name="add_edition_for_game",
     ),
     path("edition/<int:edition_id>/edit", views.edit_edition, name="edit_edition"),
+    path("edition/list", editionviews.list_editions, name="list_editions"),
+    path(
+        "edition/<int:edition_id>/delete",
+        editionviews.delete_edition,
+        name="delete_edition",
+    ),
     path("game/add", views.add_game, name="add_game"),
     path("game/<int:game_id>/edit", views.edit_game, name="edit_game"),
     path("game/<int:game_id>/view", views.view_game, name="view_game"),