From 38f2e10c6df06fe12adf7456e68d4ca9cbb697b6 Mon Sep 17 00:00:00 2001
From: Edward Betts <edward@4angle.com>
Date: Mon, 27 May 2024 10:16:03 +0200
Subject: [PATCH] Show distances for all past and future trips.

---
 templates/trip/list.html | 31 +++++++++++++++++++++++++------
 web_view.py              | 26 ++++++++++++++++++++++++++
 2 files changed, 51 insertions(+), 6 deletions(-)

diff --git a/templates/trip/list.html b/templates/trip/list.html
index a35ac5b..c39a516 100644
--- a/templates/trip/list.html
+++ b/templates/trip/list.html
@@ -1,6 +1,6 @@
 {% extends "base.html" %}
 
-{% from "macros.html" import trip_link, display_date_no_year, display_date, display_datetime, display_time with context %}
+{% from "macros.html" import trip_link, display_date_no_year, display_date, display_datetime, display_time, format_distance with context %}
 
 {% block title %}{{ heading }} - Edward Betts{% endblock %}
 
@@ -52,6 +52,18 @@
     {% set items = item_list | list %}
     <div class="heading"><h2>{{ heading }}</h2></div>
     <p>{{ items | count }} trips</p>
+
+    <div>Total distance: {{ format_distance(total_distance) }}</div>
+
+    {% for transport_type, distance in distances_by_transport_type %}
+      <div>
+        {{ transport_type | title }}
+        distance: {{format_distance(distance) }}
+      </div>
+    {% endfor %}
+
+
+
     {% for trip in items %}
       {% set distances_by_transport_type = trip.distances_by_transport_type() %}
       {% set total_distance = trip.total_distance() %}
@@ -74,15 +86,17 @@
           <div>Start: {{ display_date_no_year(trip.start) }} (end date missing)</div>
         {% endif %}
         {% if total_distance %}
-          <div>Total distance:
-          {{ "{:,.0f} km / {:,.0f} miles".format(total_distance, total_distance / 1.60934) }}
+          <div>
+            Total distance:
+            {{ format_distance(total_distance) }}
           </div>
         {% endif %}
 
         {% if distances_by_transport_type %}
           {% for transport_type, distance in distances_by_transport_type %}
-            <div>{{ transport_type | title }} distance:
-              {{ "{:,.0f} km / {:,.0f} miles".format(distance, distance / 1.60934) }}
+            <div>
+              {{ transport_type | title }}
+              distance: {{format_distance(distance) }}
             </div>
           {% endfor %}
         {% endif %}
@@ -108,9 +122,14 @@
                   {% if e.element_type == "flight" %}
                     <span class="text-nowrap"><strong>airline:</strong> {{ e.detail.airline_name }}</span>
                     <span class="text-nowrap"><strong>flight number:</strong> {{ e.detail.airline }}{{ e.detail.flight_number }}</span>
-                    <span class="text-nowrap"><strong>duration:</strong> {{ e.detail.duration }}</span>
+                    {% if e.detail.duration %}
+                      <span class="text-nowrap"><strong>duration:</strong> {{ e.detail.duration }}</span>
+                    {% endif %}
                     {# <pre>{{ e.detail | pprint }}</pre> #}
                   {% endif %}
+                  {% if e.detail.distance %}
+                    <span class="text-nowrap"><strong>distance:</strong> {{ format_distance(e.detail.distance) }}</span>
+                  {% endif %}
                 </div>
             {% endif %}
             {% endfor %}
diff --git a/web_view.py b/web_view.py
index 286d4ad..8c30bbd 100755
--- a/web_view.py
+++ b/web_view.py
@@ -8,6 +8,7 @@ import operator
 import os.path
 import sys
 import traceback
+from collections import defaultdict
 from datetime import date, datetime, timedelta
 
 import flask
@@ -286,6 +287,27 @@ def trip_list() -> werkzeug.Response:
     return flask.redirect(flask.url_for("trip_future_list"))
 
 
+def calc_total_distance(trips: list[Trip]) -> float:
+    """Total distance for trips."""
+    total = 0.0
+    for item in trips:
+        dist = item.total_distance()
+        if dist:
+            total += dist
+
+    return total
+
+
+def sum_distances_by_transport_type(trips: list[Trip]) -> list[tuple[str, float]]:
+    """Sum distances by transport type."""
+    distances_by_transport_type: defaultdict[str, float] = defaultdict(float)
+    for trip in trips:
+        for transport_type, dist in trip.distances_by_transport_type():
+            distances_by_transport_type[transport_type] += dist
+
+    return list(distances_by_transport_type.items())
+
+
 @app.route("/trip/past")
 def trip_past_list() -> str:
     """Page showing a list of past trips."""
@@ -307,6 +329,8 @@ def trip_past_list() -> str:
         get_country=agenda.get_country,
         format_list_with_ampersand=format_list_with_ampersand,
         fx_rate=agenda.fx.get_rates(app.config),
+        total_distance=calc_total_distance(past),
+        distances_by_transport_type=sum_distances_by_transport_type(past),
     )
 
 
@@ -337,6 +361,8 @@ def trip_future_list() -> str:
         get_country=agenda.get_country,
         format_list_with_ampersand=format_list_with_ampersand,
         fx_rate=agenda.fx.get_rates(app.config),
+        total_distance=calc_total_distance(current + future),
+        distances_by_transport_type=sum_distances_by_transport_type(current + future),
     )