From 47f76da4f8cc31146d2834dfdf9731d330288c9d Mon Sep 17 00:00:00 2001 From: Birte Kristina Friesel Date: Fri, 26 Jul 2024 18:55:58 +0200 Subject: Multi-backend support Squashed commit of the following: commit 92518024ba295456358618c0e8180bd8e996fdf1 Author: Birte Kristina Friesel Date: Fri Jul 26 18:39:46 2024 +0200 add_or_update station: remove superfluos 'new backend id := old backend id' commit df21c20c6e4c86454f8a9ac69121404415217f2a Author: Birte Kristina Friesel Date: Fri Jul 26 18:35:51 2024 +0200 revert connection targets min_count to 3 commit be335cef07d0b42874f5fc1de4a1d13396e8e807 Author: Birte Kristina Friesel Date: Fri Jul 26 18:20:05 2024 +0200 mention backend selection in API documentation commit 9f41828fb4f18fd707e0087def3032e8d4c8d7d8 Author: Birte Kristina Friesel Date: Thu Jul 25 20:19:23 2024 +0200 use_history: not all backends provide route data in departure monitor commit 09714b4d89684b8331d0e96f564a4c7432318f70 Author: Birte Kristina Friesel Date: Thu Jul 25 20:11:44 2024 +0200 disambiguation: pass correct hafas parameter commit 8cdf1120fc32155dc6525be64601b7c10a9c7f52 Author: Birte Kristina Friesel Date: Thu Jul 25 20:11:28 2024 +0200 _checked_in: hide Zuglauf link for non-db checkins commit 7455653f541198e0e0a6d11aed421487ffdb6285 Author: Birte Kristina Friesel Date: Thu Jul 25 20:01:47 2024 +0200 debug output commit b9cda07f85601a58ea32dbdacdd5399f302db52b Author: Birte Kristina Friesel Date: Thu Jul 25 19:09:07 2024 +0200 fix remaining get_connection_targets / get_connecting_trains_p invocations commit 2759d7258c37c7498905cfe19f6b4c4f6d16bd21 Author: Birte Kristina Friesel Date: Wed Jul 24 20:50:12 2024 +0200 support non-DB HAFAS backends (WiP) --- templates/_checked_in.html.ep | 9 ++++++-- templates/_checked_out.html.ep | 2 +- templates/_connections_hafas.html.ep | 3 ++- templates/_departures_hafas.html.ep | 1 + templates/api_documentation.html.ep | 6 +++++ templates/departures.html.ep | 39 ++++++++++++++++---------------- templates/disambiguation.html.ep | 2 +- templates/exception.html.ep | 11 +++++++-- templates/landingpage.html.ep | 43 ++++++++++++++++++++++-------------- templates/layouts/default.html.ep | 2 +- templates/select_backend.html.ep | 34 ++++++++++++++++++++++++++++ templates/traewelling.html.ep | 27 +++++++++------------- templates/use_history.html.ep | 6 ++++- 13 files changed, 124 insertions(+), 61 deletions(-) create mode 100644 templates/select_backend.html.ep (limited to 'templates') diff --git a/templates/_checked_in.html.ep b/templates/_checked_in.html.ep index 19a94bb..2d71d7e 100644 --- a/templates/_checked_in.html.ep +++ b/templates/_checked_in.html.ep @@ -343,9 +343,14 @@ % else { % $url = $url . $journey->{train_type} . ' ' . $journey->{train_no} . '/' . $journey->{sched_departure}->epoch . '000?station=' . $journey->{dep_eva}; % } - Zuglauf + % if ($journey->{backend_id} <= 1) { + Zuglauf + % } + % else { +   + % } % if ($journey->{extra_data}{trip_id}) { - Karte + Karte % } diff --git a/templates/_checked_out.html.ep b/templates/_checked_out.html.ep index 5a944dc..21db335 100644 --- a/templates/_checked_out.html.ep +++ b/templates/_checked_out.html.ep @@ -3,7 +3,7 @@ Ausgecheckt

Aus %= include '_format_train', journey => $journey - bis <%= $journey->{arr_name} %>

+ bis <%= $journey->{arr_name} %>

% if (@{stash('connections_iris') // [] } or @{stash('connections_hafas') // []}) { Verbindungen

Fahrt auswählen zum Einchecken mit Zielwahl.

diff --git a/templates/_connections_hafas.html.ep b/templates/_connections_hafas.html.ep index dcf7ec9..43198e2 100644 --- a/templates/_connections_hafas.html.ep +++ b/templates/_connections_hafas.html.ep @@ -1,6 +1,6 @@
    % for my $res (@{$connections}) { - % my ($train, $via, $via_arr) = @{$res}; + % my ($train, $via, $via_arr, $hafas_service) = @{$res}; % $via_arr = $via_arr ? $via_arr->strftime('%H:%M') : q{}; % my $row_class = ''; % my $link_class = 'action-checkin'; @@ -10,6 +10,7 @@ % } % if ($checkin_from) {
  • % }
  • "checkedIn" : true / false,
    "comment": "Kommentar",
    + "backend": {
    + "id": 1,
    + "name": "DB",
    + "type": "HAFAS",
    + },
    "fromStation" : { (letzter Checkin)
    "name" : "Essen Hbf",
    "ds100" : "EE", (ggf. null)
    @@ -122,6 +127,7 @@ {
    "token" : "<%= $uid %>-<%= $token->{travel} // 'TOKEN' %>",
    "action" : "checkin",
    + "hafas" : "DB", (HAFAS-Instanz – Default: Deutsche Bahn)
    "train" : {
    "journeyID" : "1|1426396|4|80|19082023",
    }
    diff --git a/templates/departures.html.ep b/templates/departures.html.ep index 9fd79e6..71dd0ad 100644 --- a/templates/departures.html.ep +++ b/templates/departures.html.ep @@ -1,26 +1,27 @@
    -
    -

    - +
    + <%= $station %> -

    + % for my $related_station (sort { $a->{name} cmp $b->{name} } @{$related_stations}) { + <%= $related_station->{name} %>
    % }
    -
    -% if ($api_link) { -
    -
    - % if (param('hafas')) { - zum Schienenverkehr - % } - % else { - zum Nahverkehr - % } +
    + % my $self_link = url_for('sstation', station => param('station')); + % if (param('hafas')) { + <%= param('hafas') %> (HAFAS) + % } + % else { + % if ($user->{backend_id}) { + <%= $user->{backend_name} %> (<%= $user->{backend_hafas} ? 'HAFAS' : q{} %>) + % } + % else { + DB (IRIS-TTS) + % } + % }
    -% } % my $have_connections = 0; % if ($user_status->{checked_in}) { @@ -40,10 +41,10 @@
    % if ($can_check_out) { - + Rückgängig - + Hier auschecken % } @@ -51,7 +52,7 @@ Rückgängig - + Hier auschecken @@ -139,7 +140,7 @@

    % if (not $user_status->{checked_in} or ($can_check_out and $user_status->{arr_eva} and $user_status->{arrival_countdown} <= 0)) { % if ($hafas) { - %= include '_departures_hafas', results => $results; + %= include '_departures_hafas', results => $results, hafas => $hafas; % } % else { %= include '_departures_iris', results => $results; diff --git a/templates/disambiguation.html.ep b/templates/disambiguation.html.ep index 9fc1e4c..af7d1dd 100644 --- a/templates/disambiguation.html.ep +++ b/templates/disambiguation.html.ep @@ -13,7 +13,7 @@
    diff --git a/templates/exception.html.ep b/templates/exception.html.ep index ec01ad2..9b8697c 100644 --- a/templates/exception.html.ep +++ b/templates/exception.html.ep @@ -20,8 +20,15 @@ Timestamp: %= DateTime->now(time_zone => 'Europe/Berlin')->strftime("%d/%b/%Y:%H:%M:%S %z")

    - Message: - %= ref($exception) ? (split(qr{\n}, $exception->message))[0] : $exception + % if (ref($exception)) { + Trace:
    + % for my $line (split(qr{\n}, $exception->message)) { + <%= $line %>
    + % } + % } + % else { + Message: <%= $exception %> + % }

    diff --git a/templates/landingpage.html.ep b/templates/landingpage.html.ep index 45bfb21..22478ed 100644 --- a/templates/landingpage.html.ep +++ b/templates/landingpage.html.ep @@ -1,5 +1,6 @@ % if (is_user_authenticated()) { % my $status = stash('user_status'); + % my $user = stash('user'); % if (stash('error')) {
    @@ -51,32 +52,40 @@ % if ( @{stash('timeline') // [] } ) { %= include '_timeline_link', timeline => stash('timeline') % } -
    -
    - Hallo, <%= current_user->{name} %>! -

    Du bist gerade nicht eingecheckt.

    -
    - -
    - %= form_for 'list_departures' => begin + %= form_for 'list_departures' => begin +
    +
    + Hallo, <%= $user->{name} %>! +

    Du bist gerade nicht eingecheckt.

    +

    +

    {backend_id} %>"> + +
    %= text_field 'station', id => 'station', class => 'autocomplete contrast-color-text', autocomplete => 'off', required => undef
    -
    - -
    - %= end +

    +
    +
    + % if ($user->{backend_id}) { + <%= $user->{backend_name} %> (<%= $user->{backend_hafas} ? 'HAFAS' : q{} %>) + % } + % else { + DB (IRIS-TTS) + % } + +
    -
    + %= end % }

    Letzte Fahrten

    - %= include '_history_trains', date_format => '%d.%m.%Y', journeys => [journeys->get(uid => current_user->{id}, limit => 5, with_datetime => 1)]; + %= include '_history_trains', date_format => '%d.%m.%Y', journeys => [journeys->get(uid => $user->{id}, limit => 5, with_datetime => 1)]; % } % else {
    diff --git a/templates/layouts/default.html.ep b/templates/layouts/default.html.ep index 01b8018..4edf805 100644 --- a/templates/layouts/default.html.ep +++ b/templates/layouts/default.html.ep @@ -62,7 +62,7 @@ %= javascript "/static/${av}/js/geolocation${min}.js" % } % if (stash('with_autocomplete')) { - %= javascript "/dyn/${av}/autocomplete.js", defer => undef + %= javascript "/dyn/${av}/autocomplete.js?backend_id=" . (stash('backend_id') // 1), defer => undef % } % if (stash('with_map')) { %= javascript "/static/${av}/leaflet/leaflet.js" diff --git a/templates/select_backend.html.ep b/templates/select_backend.html.ep new file mode 100644 index 0000000..4eb4cc9 --- /dev/null +++ b/templates/select_backend.html.ep @@ -0,0 +1,34 @@ +
    +
    +

    Backend auswählen

    +

    + Das ausgewählte Backend bestimmt die Datenquelle für Fahrten in travelynx. + DB (HAFAS) ist eine gute Wahl für Nah-, Regional- und Fernverkehr in Deutschland und (teilweise) Nachbarländern. + IRIS-TTS unterstützt ausschließlich Schienenverkehr; im Gegensatz zum HAFAS sind hier detaillierte Verspätungsgründe verfügbar. + Die restlichen Backends können sich für Fahrten außerhalb Deutschlands sowie in einzelnen innerdeutschen Verkehrsverbünden lohnen. + Fahrten im Ausland fehlen im DB-HAFAS oft; Fahrten im Inland können mit spezifischen Backends genauere Daten haben – z.B. liefert das AVV-HAFAS im Gegensatz zum DB-HAFAS Kartendaten für dortige Buslinien. +

    +
    +
    +%= form_for '/account/select_backend' => (method => 'POST') => begin + % if (stash('redirect_to')) { + %= hidden_field 'redirect_to' => stash('redirect_to') + % } + % for my $backend (@{ stash('backends') // [] }) { +
    +
    + +
    +
    + % if ($backend->{longname}) { + <%= $backend->{longname} %> + % } + % if ($backend->{id} == $user->{backend_id}) { + (aktuell ausgewählt) + % } +
    +
    + % } +%= end diff --git a/templates/traewelling.html.ep b/templates/traewelling.html.ep index c1f2b7d..1e0d65d 100644 --- a/templates/traewelling.html.ep +++ b/templates/traewelling.html.ep @@ -157,26 +157,21 @@
    -
    -
    -

    Die Synchronisierung erfolgt spätestens drei Minuten nach der - Zielwahl. Beachte, dass die Synchronisierung travelynx - → Träwelling unabhängig von der eingestellten Sichtbarkeit - des Checkins erfolgt. travelynx reicht die Sichtbarkeit - aber an Träwelling weiter. - Träwelling-Checkins können von travelynx aktuell nicht - rückgängig gemacht werden. Eine nachträgliche Änderung der - Zielstation wird nicht übernommen. Mastodon und Twitter beziehen - sich auf die in den Träwelling-Einstellungen - verknüpften Accounts.

    + verknüpften Account.

    diff --git a/templates/use_history.html.ep b/templates/use_history.html.ep index 9b76e98..f91ca16 100644 --- a/templates/use_history.html.ep +++ b/templates/use_history.html.ep @@ -5,9 +5,13 @@ Travelynx kann anhand deiner vergangenen Fahrten Verbindungen zum Einchecken vorschlagen. Fährst zu z.B regelmäßig von Dortmund Hbf nach Essen Hbf, werden dir in Dortmund bevorzugt Fahrten angezeigt, die - Essen passieren. Bei Auswahl dieser wird nicht nur in die Fahrt eingecheckt, + Essen passieren. Bei Auswahl dieser wird nicht nur in die Fahrt eingecheckt, sondern auch direkt Essen Hbf als Ziel eingetragen.

    +

    + Beachte, dass nicht alle von travelynx unterstützten Backends die + für dieses Feature notwendigen Daten bereitstellen. +