carriage formation: directly refer to Travel::Status::DE::DBRIS

Also removes old has_wagonorder / get_wagonorder split. That used to be
sensible when we had multiple backends, but has long since lost its meaning.

1 files changed, 15 insertions, 120 deletions

diff --git a/lib/Travelynx/Helper/DBRIS.pm b/lib/Travelynx/Helper/DBRIS.pm
index 966a78d..b5f30b2 100644
--- a/lib/Travelynx/Helper/DBRIS.pm
+++ b/lib/Travelynx/Helper/DBRIS.pm
@@ -160,130 +160,25 @@ sub get_journey_p {
 	return $promise;
 }
 
-sub has_wagonorder_p {
-	my ( $self, %opt ) = @_;
-
-	$opt{train_type} //= q{};
-	my $datetime = $opt{datetime}->clone->set_time_zone('UTC');
-	my %param    = (
-		administrationId => 80,
-		category         => $opt{train_type},
-		date             => $datetime->strftime('%Y-%m-%d'),
-		evaNumber        => $opt{eva},
-		number           => $opt{train_no},
-		time             => $datetime->rfc3339 =~ s{(?=Z)}{.000}r
-	);
-
-	my $url = sprintf( '%s?%s',
-'https://www.bahn.de/web/api/reisebegleitung/wagenreihung/vehicle-sequence',
-		join( '&', map { $_ . '=' . $param{$_} } sort keys %param ) );
-
-	my $promise = Mojo::Promise->new;
-	my $debug_prefix
-	  = "has_wagonorder_p($opt{train_type} $opt{train_no} @ $opt{eva})";
-
-	if ( my $content = $self->{main_cache}->get("HEAD $url")
-		// $self->{realtime_cache}->get("HEAD $url") )
-	{
-		if ( $content eq 'n' ) {
-			$self->{log}->debug("${debug_prefix}: n (cached)");
-			return $promise->reject;
-		}
-		else {
-			$self->{log}->debug("${debug_prefix}: ${content} (cached)");
-			return $promise->resolve($content);
-		}
-	}
-
-	$self->get_agent->request_timeout(5)
-	  ->get_p( $url => $self->{header} )
-	  ->then(
-		sub {
-			my ($tx) = @_;
-			if ( $tx->result->is_success ) {
-				$self->{log}->debug("${debug_prefix}: a");
-				$self->{main_cache}->set( "HEAD $url", 'a' );
-				my $body = decode( 'utf-8', $tx->res->body );
-				my $json = JSON->new->decode($body);
-				$self->{main_cache}->freeze( $url, $json );
-				$promise->resolve('a');
-			}
-			else {
-				my $code = $tx->res->code;
-				$self->{log}->debug("${debug_prefix}: n (HTTP $code)");
-				$self->{realtime_cache}->set( "HEAD $url", 'n' );
-				$promise->reject;
-			}
-			return;
-		}
-	  )->catch(
-		sub {
-			my ($err) = @_;
-			$self->{log}->debug("${debug_prefix}: n ($err)");
-			$self->{realtime_cache}->set( "HEAD $url", 'n' );
-			$promise->reject;
-			return;
-		}
-	  )->wait;
-	return $promise;
-}
-
 sub get_wagonorder_p {
 	my ( $self, %opt ) = @_;
 
-	my $datetime = $opt{datetime}->clone->set_time_zone('UTC');
-	my %param    = (
-		administrationId => 80,
-		category         => $opt{train_type},
-		date             => $datetime->strftime('%Y-%m-%d'),
-		evaNumber        => $opt{eva},
-		number           => $opt{train_no},
-		time             => $datetime->rfc3339 =~ s{(?=Z)}{.000}r
-	);
-
-	my $url = sprintf( '%s?%s',
-'https://www.bahn.de/web/api/reisebegleitung/wagenreihung/vehicle-sequence',
-		join( '&', map { $_ . '=' . $param{$_} } sort keys %param ) );
-	my $debug_prefix
-	  = "get_wagonorder_p($opt{train_type} $opt{train_no} @ $opt{eva})";
-
-	my $promise = Mojo::Promise->new;
-
-	if ( my $content = $self->{main_cache}->thaw($url) ) {
-		$self->{log}->debug("${debug_prefix}: (cached)");
-		$promise->resolve($content);
-		return $promise;
-	}
+	$self->{log}
+	  ->debug("get_wagonorder_p($opt{train_type} $opt{train_no} @ $opt{eva})");
 
-	$self->get_agent->request_timeout(5)
-	  ->get_p( $url => $self->{header} )
-	  ->then(
-		sub {
-			my ($tx) = @_;
-
-			if ( $tx->result->is_success ) {
-				my $body = decode( 'utf-8', $tx->res->body );
-				my $json = JSON->new->decode($body);
-				$self->{log}->debug("${debug_prefix}: success");
-				$self->{main_cache}->freeze( $url, $json );
-				$promise->resolve($json);
-			}
-			else {
-				my $code = $tx->res->code;
-				$self->{log}->debug("${debug_prefix}: HTTP ${code}");
-				$promise->reject("HTTP ${code}");
-			}
-			return;
-		}
-	  )->catch(
-		sub {
-			my ($err) = @_;
-			$self->{log}->debug("${debug_prefix}: error ${err}");
-			$promise->reject($err);
-			return;
-		}
-	  )->wait;
-	return $promise;
+	return Travel::Status::DE::DBRIS->new_p(
+		cache         => $self->{main_cache},
+		failure_cache => $self->{realtime_cache},
+		promise       => 'Mojo::Promise',
+		user_agent    => $self->get_agent->request_timeout(10),
+		formation     => {
+			departure    => $opt{datetime},
+			eva          => $opt{eva},
+			train_type   => $opt{train_type},
+			train_number => $opt{train_no}
+		},
+		developer_mode => $self->{log}->is_level('debug') ? 1 : 0,
+	);
 }
 
 1;