diff --git a/agenda/holidays.py b/agenda/holidays.py index a93ec5d..620b9ab 100644 --- a/agenda/holidays.py +++ b/agenda/holidays.py @@ -3,30 +3,11 @@ import collections from datetime import date, timedelta -import flask - import agenda.uk_holiday import holidays from .event import Event -from .types import Holiday, Trip - - -def get_trip_holidays(trip: Trip) -> list[Holiday]: - """Get holidays happening during trip.""" - if not trip.end: - return [] - countries = {c.alpha_2 for c in trip.countries} - return sorted( - ( - hol - for hol in get_all( - trip.start, trip.end, flask.current_app.config["DATA_DIR"] - ) - if hol.country.upper() in countries - ), - key=lambda item: (item.date, item.country), - ) +from .types import Holiday def us_holidays(start_date: date, end_date: date) -> list[Holiday]: diff --git a/agenda/utils.py b/agenda/utils.py index 0c17610..974f554 100644 --- a/agenda/utils.py +++ b/agenda/utils.py @@ -42,11 +42,6 @@ def timedelta_display(delta: timedelta) -> str: ) -def plural(value: int, unit: str) -> str: - """Value + unit with unit written as singular or plural as appropriate.""" - return f"{value} {unit}{'s' if value > 1 else ''}" - - def human_readable_delta(future_date: date) -> str | None: """ Calculate the human-readable time delta for a given future date. @@ -69,11 +64,14 @@ def human_readable_delta(future_date: date) -> str | None: weeks, days = divmod(days, 7) # Formatting the output - parts = [ - plural(value, unit) - for value, unit in ((months, "month"), (weeks, "week"), (days, "days")) - if value > 0 - ] + parts = [] + if months > 0: + parts.append(f"{months} month{'s' if months > 1 else ''}") + if weeks > 0: + parts.append(f"{weeks} week{'s' if weeks > 1 else ''}") + if days > 0: + parts.append(f"{days} day{'s' if days > 1 else ''}") + return " ".join(parts) if parts else None diff --git a/templates/trip_page.html b/templates/trip_page.html index b7733f3..bf961fa 100644 --- a/templates/trip_page.html +++ b/templates/trip_page.html @@ -53,7 +53,7 @@ } .half-map { - height: 90vh; + height: 100vh; } .full-window-map { diff --git a/tests/test_agenda.py b/tests/test_agenda.py index 35b3102..51d6643 100644 --- a/tests/test_agenda.py +++ b/tests/test_agenda.py @@ -5,6 +5,7 @@ from decimal import Decimal import pytest from agenda import ( + get_gbpusd, get_next_bank_holiday, get_next_timezone_transition, next_economist, @@ -13,7 +14,6 @@ from agenda import ( timedelta_display, uk_financial_year_end, ) -from agenda.fx import get_gbpusd @pytest.fixture diff --git a/web_view.py b/web_view.py index c485657..a73a1b1 100755 --- a/web_view.py +++ b/web_view.py @@ -486,15 +486,13 @@ def get_prev_current_and_next_trip( """Get previous trip, this trip and next trip.""" trip_iter = iter(trip_list) prev_trip = None - current_trip = None for trip in trip_iter: if trip.start.isoformat() == start: - current_trip = trip break prev_trip = trip next_trip = next(trip_iter, None) - return (prev_trip, current_trip, next_trip) + return (prev_trip, trip, next_trip) @app.route("/trip/") @@ -507,6 +505,8 @@ def trip_page(start: str) -> str: if not trip: flask.abort(404) + today = date.today() + coordinates = agenda.trip.collect_trip_coordinates(trip) routes = agenda.trip.get_trip_routes(trip, app.config["PERSONAL_DATA"]) @@ -518,17 +518,30 @@ def trip_page(start: str) -> str: app.config["PERSONAL_DATA"], route.pop("geojson_filename") ) + if trip.end: + countries = {c.alpha_2 for c in trip.countries} + holidays = [ + hol + for hol in agenda.holidays.get_all( + trip.start, trip.end, app.config["DATA_DIR"] + ) + if hol.country.upper() in countries + ] + holidays.sort(key=lambda item: (item.date, item.country)) + else: + holidays = [] + return flask.render_template( "trip_page.html", trip=trip, prev_trip=prev_trip, next_trip=next_trip, - today=date.today(), + today=today, coordinates=coordinates, routes=routes, get_country=agenda.get_country, format_list_with_ampersand=format_list_with_ampersand, - holidays=agenda.holidays.get_trip_holidays(trip), + holidays=holidays, human_readable_delta=agenda.utils.human_readable_delta, ) @@ -567,6 +580,9 @@ def trip_stats() -> str: route_distances = agenda.travel.load_route_distances(app.config["DATA_DIR"]) trip_list = get_trip_list(route_distances) + # today = date.today() + # past = [item for item in trip_list if (item.end or item.start) < today] + conferences = sum(len(item.conferences) for item in trip_list) yearly_stats = agenda.stats.calculate_yearly_stats(trip_list)