bitlair/revbank
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove overzealous checks from repeat plugin

Browse source 
There are some cases where repetition can be used with multi user transaction,
specifically if the repetition is done before any multi user stuff is added...
This commit is contained in:
Juerd Waalboer 2019-05-12 21:46:12 +02:00
parent de01c5fa23
commit 19691e07b8
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
plugins/repeat
View file

@ -32,9 +32,8 @@ sub command {
return NEXT if $pre and $post; # 123x123 -> invalid syntax
return REJECT, $err_multi if $cart->is_multi_user;
if ($post) {
return REJECT, $err_multi if $cart->is_multi_user;
return REJECT, $err_limit if $post > $limit;
return ABORT, "Can't repeat an empty transaction." if not $cart->size;
return REJECT, $err_stacked if $last->{_repeated};
@ -89,13 +88,12 @@ sub hook_added {
my ($self, $cart, $user, $item) = @_;
$cart->size >= 2 or return;
my @items = $cart->select_items;
my @planned = $cart->select_items('_repeat');
my @repeated = $cart->select_items('_repeated');
return ABORT, $err_multi if $cart->is_multi_user and @planned || @repeated;
return ABORT, $err_multi if $cart->is_multi_user and @planned;
return ABORT, "Multiple repeats queued; I'm confused." if @planned > 1;
return if not @planned;
my @items = $cart->select_items;
return ABORT, $err_pfand if grep $_->{is_pfand}, @items;
for my $i (0 .. $#items - 1) {