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

Remove dead code

Browse source 
As a side effect of a7a5f14e, "123 x <product>" (with a space between the
number and the operator) is no longer supported. Breaking that was
unintentional, but since it was an undocumented feature and unintuitive
hack anyway, the feature won't come back.

When it still worked, it was implemented by *converting* a raw amount
(withdrawal or unlisted product, which you could still enter as just a
number -- since the aforementioned commit you need to be explicit and
use `withdraw` or `unlisted`) into a stub that would finally apply a
repetition when adding something else.
This commit is contained in:
Juerd Waalboer 2023-01-29 22:52:02 +01:00
parent af5567da8b
commit 5e5c27a203
1 changed files with 2 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

10
plugins/repeat
View file

@ -1,7 +1,7 @@
#!perl
HELP "*<N>, x<N>, <N>x, <N>*" => "Set quantity of previous/next product";
HELP "-<N>, +<N>" => "Decrease/increase quantity of previous product";
HELP "<N>x, <N>*" => "Set quantity of previous/next product";
HELP "-<N>, +<N>, *<N>, x<N>" => "Change quantity of previous product";
my $err_stacked = "Stacked repetition is not supported.";
my $err_pfand = "Plugins 'pfand' and 'repeat' cannot be combined.";
@ -60,12 +60,6 @@ sub command($self, $cart, $command, @) {
$self->{op} = $op;
return "$op how many?", \&plusminus;
}
if ($last->has_attribute('is_withdrawal')) {
$lhs = $last->{amount}->abs->float;
$lhs == int $lhs or return REJECT, "Repeat only works on integers.";
$cart->delete($last);
}
}
if ($lhs) {