diff options
| author | Birte Kristina Friesel <derf@finalrewind.org> | 2023-07-23 12:21:37 +0200 |
|---|---|---|
| committer | Birte Kristina Friesel <derf@finalrewind.org> | 2023-07-23 12:21:37 +0200 |
| commit | 13277ad770b6710a9a5536334011d04fcf390919 (patch) | |
| tree | 8d065ea16afb6bd85cc9c23c04f533c9d7ba73b1 /lib/Travelynx | |
| parent | be607983c14dfc260ec20e92b97265d016198040 (diff) | |
checkout: remove with_related special case
travelynx now always calls checkout with the database-provided arr_eva,
so the edge case in which it provided a station that is not actually on the
route of a train can no longer appear.
Diffstat (limited to 'lib/Travelynx')
| -rwxr-xr-x | lib/Travelynx/Controller/Api.pm | 19 |
1 files changed, 13 insertions, 6 deletions
diff --git a/lib/Travelynx/Controller/Api.pm b/lib/Travelynx/Controller/Api.pm index 9685582..d0ebc1b 100755 --- a/lib/Travelynx/Controller/Api.pm +++ b/lib/Travelynx/Controller/Api.pm @@ -277,10 +277,14 @@ sub travel_v1 { ); } if ($to_station) { + + # the user may not have provided the correct to_station, so + # request related stations for checkout. my ( $train2, $error ) = $self->checkout( - station => $to_station, - force => 0, - uid => $uid + station => $to_station, + force => 0, + uid => $uid, + with_related => 1, ); if ($error) { return Mojo::Promise->reject($error); @@ -330,10 +334,13 @@ sub travel_v1 { ); } + # the user may not have provided the correct to_station, so + # request related stations for checkout. my ( $train, $error ) = $self->checkout( - station => $to_station, - force => $payload->{force} ? 1 : 0, - uid => $uid + station => $to_station, + force => $payload->{force} ? 1 : 0, + uid => $uid, + with_related => 1, ); if ($error) { $self->render( |