From d2a9630b048ef3a3a1721d7c1db2abb2902e1aff Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Kucharczyk?= <lukas@kucharczyk.xyz>
Date: Sun, 1 Oct 2023 21:51:32 +0200
Subject: [PATCH] Fix date range on game overview

---
 games/templates/view_game.html |  4 ++--
 games/views.py                 | 11 ++++++++---
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/games/templates/view_game.html b/games/templates/view_game.html
index 2667a93..1e2a210 100644
--- a/games/templates/view_game.html
+++ b/games/templates/view_game.html
@@ -7,7 +7,7 @@
 {% block content %}
     <div class="dark:text-white max-w-sm sm:max-w-xl lg:max-w-3xl mx-auto">
         <h1 class="text-4xl">{{ game.name }} (#{{ game.pk }})</h1>
-        <h2 class="text-lg my-2 ml-2">{{ total_playtime }} ({{ first_session.timestamp_end | date:"M Y"}} — {{ first_session.timestamp_end | date:"M Y"}}) </h2>
+        <h2 class="text-lg my-2 ml-2">{{ total_playtime }} ({{ first_session.timestamp_end | date:"M Y"}} — {{ last_session.timestamp_end | date:"M Y"}}) </h2>
         <hr class="border-slate-500">
         <h1 class="text-3xl mt-4 mb-1">Editions</h1>
         <ul>
@@ -33,7 +33,7 @@
         <h1 class="text-3xl mt-4 mb-1">Sessions</h1>
         <ul>
             {% for session in sessions %}
-            <li class="sm:pl-2">{{ session.timestamp_end | date:"d/m/Y" }} ({{ session.device.get_type_display | default:"Unknown" }}, {{ session.duration_formatted }})</li>
+            <li class="sm:pl-2">{{ session.timestamp_start | date:"d/m/Y" }} ({{ session.device.get_type_display | default:"Unknown" }}, {{ session.duration_formatted }})</li>
             {% endfor %}
         </ul>
         
diff --git a/games/views.py b/games/views.py
index 6c70d9f..4c111ac 100644
--- a/games/views.py
+++ b/games/views.py
@@ -98,10 +98,15 @@ def view_game(request, game_id=None):
     context["game"] = game
     context["editions"] = Edition.objects.filter(game_id=game_id)
     context["purchases"] = Purchase.objects.filter(edition__game_id=game_id)
-    context["sessions"] = Session.objects.filter(purchase__edition__game_id=game_id)
+    context["sessions"] = Session.objects.filter(
+        purchase__edition__game_id=game_id
+    ).order_by("-timestamp_start")
     context["total_playtime"] = context["sessions"].total_duration()
-    context["last_session"] = context["sessions"].last
-    context["first_session"] = context["sessions"].first
+    # here first and last is flipped
+    # because sessions are ordered from newest to oldest
+    # so the most recent are on top
+    context["last_session"] = context["sessions"].first()
+    context["first_session"] = context["sessions"].last()
     return render(request, "view_game.html", context)