From d213a3d35d16f071b2489ad82f76bb4a52fc9185 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Kucharczyk?= <lukas@kucharczyk.xyz>
Date: Thu, 30 Jan 2025 17:54:42 +0100
Subject: [PATCH] Improve purchase view

---
 games/models.py                    | 19 ++++++++++++++++---
 games/templates/view_purchase.html | 11 ++++++++++-
 games/views/purchase.py            |  6 +++++-
 3 files changed, 31 insertions(+), 5 deletions(-)

diff --git a/games/models.py b/games/models.py
index 15e0cd3..eba5848 100644
--- a/games/models.py
+++ b/games/models.py
@@ -3,7 +3,7 @@ from datetime import timedelta
 from django.core.exceptions import ValidationError
 from django.db import models
 from django.db.models import F, Sum
-from django.template.defaultfilters import floatformat, slugify
+from django.template.defaultfilters import floatformat, pluralize, slugify
 from django.utils import timezone
 
 from common.time import format_duration
@@ -148,14 +148,27 @@ class Purchase(models.Model):
 
     @property
     def standardized_name(self):
-        return self.name if self.name else self.first_game.name
+        return self.name or self.first_game.name
 
     @property
     def first_game(self):
         return self.games.first()
 
     def __str__(self):
-        return f"{self.standardized_name} ({self.num_purchases}, {self.date_purchased}, {self.standardized_price})"
+        return self.standardized_name
+
+    @property
+    def full_name(self):
+        additional_info = [
+            str(item)
+            for item in [
+                f"{self.num_purchases} game{pluralize(self.num_purchases)}",
+                self.date_purchased,
+                self.standardized_price,
+            ]
+            if item
+        ]
+        return f"{self.standardized_name} ({', '.join(additional_info)})"
 
     def is_game(self):
         return self.type == self.GAME
diff --git a/games/templates/view_purchase.html b/games/templates/view_purchase.html
index 040c06d..fd1ead0 100644
--- a/games/templates/view_purchase.html
+++ b/games/templates/view_purchase.html
@@ -2,8 +2,17 @@
     <div class="dark:text-white max-w-sm sm:max-w-xl lg:max-w-3xl mx-auto">
     
     <div class="flex flex-col gap-5 mb-3">
+        <div class="font-bold font-serif text-slate-500 text-2xl">
+        {% if not purchase.name %}
+            Unnamed purchase
+        {% else %}
+            {{ purchase.name }}
+        {% endif %}
+        </div>
     <span class="text-balance max-w-[30rem] text-4xl">
-        <span class="font-bold font-serif">{% if purchase.name %}{{ purchase.name }}{% else %}Unnamed purchase{% endif %}</span> <span class="text-slate-500 text-2xl">({{ purchase.games.count }} games)</span>
+        <span class="font-bold font-serif">
+            {{ purchase.date_purchased }} ({{ purchase.num_purchases }} game{{ purchase.num_purchases|pluralize}})
+        </span>
     </span>
     <div class="inline-flex rounded-md shadow-sm mb-3" role="group">
         <a href="{% url 'edit_purchase' purchase.id %}">
diff --git a/games/views/purchase.py b/games/views/purchase.py
index c7962b7..2af5ee6 100644
--- a/games/views/purchase.py
+++ b/games/views/purchase.py
@@ -200,7 +200,11 @@ def delete_purchase(request: HttpRequest, purchase_id: int) -> HttpResponse:
 @login_required
 def view_purchase(request: HttpRequest, purchase_id: int) -> HttpResponse:
     purchase = get_object_or_404(Purchase, id=purchase_id)
-    return render(request, "view_purchase.html", {"purchase": purchase})
+    return render(
+        request,
+        "view_purchase.html",
+        {"purchase": purchase, "title": f"Purchase: {purchase.full_name}"},
+    )
 
 
 @login_required