PrivateCoffee/travelynx
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

remove sleep(1) workaround in checkout-before-checkin code path

Browse source 
The unique constraint on (user id, timestamp) is no longer present
This commit is contained in:
Daniel Friesel 2019-03-25 21:37:11 +01:00
parent a30970eb8b
commit 87a3ceb0c1
1 changed files with 2 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
lib/Travelynx.pm
View file

@ -536,23 +536,14 @@ qq{select * from pending_mails where email = ? and num_tries > 1;}
my $user = $self->get_user_status;
if ( $user->{checked_in} ) {
# If a user is already checked in, we assume that they forgot to
# check out and do it for them.
# XXX this is an ugly workaround for the UNIQUE constraint on
# (user id, action timestamp): Ensure that checkout and re-checkin
# work even if the previous checkin was less than a second ago.
sleep(1);
# If a user is already checked in, we assume that they forgot to
# check out and do it for them.
$self->checkout( $station, 1 );
# XXX same workaround: We can't checkin immediately after checkout.
sleep(1);
}
elsif ( $user->{cancelled} ) {
# Same
sleep(1);
$self->cancelled_to($station);
sleep(1);
}
my $success = $self->app->action_query->execute(