diff options
author | Daniel Friesel <derf@finalrewind.org> | 2019-07-14 17:00:48 +0200 |
---|---|---|
committer | Daniel Friesel <derf@finalrewind.org> | 2019-07-14 17:00:48 +0200 |
commit | 2b71512e0d484bca2178722a50facf8ae1e3bebd (patch) | |
tree | 5258f6fcae5683ac19f845e740406e36b3fb4a41 /lib | |
parent | d7ec56ddfd7f4579e4695c2079233b4542e34b72 (diff) |
get_route_timestamps: handle ambiguous train numbers2.3.5
Diffstat (limited to 'lib')
-rw-r--r-- | lib/DBInfoscreen/Controller/Stationboard.pm | 34 |
1 files changed, 27 insertions, 7 deletions
diff --git a/lib/DBInfoscreen/Controller/Stationboard.pm b/lib/DBInfoscreen/Controller/Stationboard.pm index 5b1b95c..9c8248c 100644 --- a/lib/DBInfoscreen/Controller/Stationboard.pm +++ b/lib/DBInfoscreen/Controller/Stationboard.pm @@ -196,15 +196,26 @@ sub hafas_xml_req { # quick&dirty, will be cleaned up later sub get_route_timestamps { - my ( $ua, $cache_main, $cache_rt, $train ) = @_; + my ( $ua, $cache_main, $cache_rt, $opt ) = @_; $ua->request_timeout(3); my $base = 'https://reiseauskunft.bahn.de/bin/trainsearch.exe/dn?L=vs_json&start=yes&rt=1'; - my $date_yy = $train->start->strftime('%d.%m.%y'); - my $date_yyyy = $train->start->strftime('%d.%m.%Y'); - my $train_no = $train->type . ' ' . $train->train_no; + my ( $date_yy, $date_yyyy, $train_no, $train_origin ); + + if ( $opt->{train} ) { + $date_yy = $opt->{train}->start->strftime('%d.%m.%y'); + $date_yyyy = $opt->{train}->start->strftime('%d.%m.%Y'); + $train_no = $opt->{train}->type . ' ' . $opt->{train}->train_no; + $train_origin = $opt->{train}->origin; + } + else { + my $now = DateTime->now( time_zone => 'Europe/Berlin' ); + $date_yy = $now->strftime('%d.%m.%y'); + $date_yyyy = $now->strftime('%d.%m.%Y'); + $train_no = $opt->{train_no}; + } my $trainsearch = hafas_json_req( $ua, $cache_main, "${base}&date=${date_yy}&trainname=${train_no}" ); @@ -223,8 +234,16 @@ sub get_route_timestamps { if ( $suggestion->{depDate} eq $date_yy or $suggestion->{depDate} eq $date_yyyy ) { - $trainlink = $suggestion->{trainLink}; - last; + # Train numbers are not unique, e.g. IC 149 refers both to the + # InterCity service Amsterdam -> Berlin and to the InterCity service + # Koebenhavns Lufthavn st -> Aarhus. One workaround is making + # requests with the stationFilter=80 parameter. Checking the origin + # station seems to be the more generic solution, so we do that + # instead. + if ( $train_origin and $suggestion->{dep} eq $train_origin ) { + $trainlink = $suggestion->{trainLink}; + last; + } } } @@ -918,7 +937,8 @@ sub handle_request { my ( $route_ts, $route_info ) = get_route_timestamps( $self->ua, $self->app->cache_iris_main, - $self->app->cache_iris_rt, $result + $self->app->cache_iris_rt, + { train => $result } ); # If a train number changes on the way, IRIS routes are incomplete, |