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

give: Don't do checkout with description as username

Browse source 
Originally, this command didn't have a description parameter. Foo would
use `give xyzzy 10 foo`. Then, a description parameter was added. For
backwards compatibility, if you would enter a username (like `foo` in
this example) in the place of the description, it would finalize the
transaction using that.

However, as the user base grows, several reasonable descriptions exist as
user account names, and that would finalize the transaction under the
wrong user.

It's time to break backward compatibility. If you don't want to leave a
message (it's still optional), that can be done with `x` (like the
`donate` command), or in advanced mode, with `""`.

Because it's likely that people are still very much used to just leaving
the description out, if you enter something that happens to match an
existing username, the input will be rejected.

The current equivalent command line would be `give xyzzy 10 ""; foo` or
`give xyzzy 10 x; foo`. If the `;` (new since v5.0.0) is left out, the
trailing `foo` has to be confirmed with a second press of the enter key.
This commit is contained in:
Juerd Waalboer 2023-12-26 05:15:43 +01:00
parent 243b34e295
commit 09411bb6c0
1 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
plugins/give
View file

@ -19,21 +19,22 @@ sub amount($self, $cart, $input, @) {
$self->{amount} = parse_amount($input)
or return REJECT, "$input: Invalid amount.";
return "Why are you giving $self->{amount} to $self->{beneficiary}, or enter your username to end", \&reason;
return "Short description ('x' for no message)", \&reason;
}
sub reason :Tab(whatevah) ($self, $cart, $input, @) {
return REJECT, "'$input' is a username, not a description :)."
if parse_user($input);
return REJECT, "'$input' is an amount, not a description :)."
if parse_amount($input);
my $beneficiary = $self->{beneficiary};
my $amount = $self->{amount};
my $user = parse_user($input);
my $reason = $user ? "" : " ($input)";
my $reason = $input =~ /^x?$/ ? "" : " ($input)";
$cart
->add(-$amount, "Given to $beneficiary" . $reason)
->add_contra($beneficiary, +$amount, "Received from \$you" . $reason);
$cart->checkout($user) if $user;
return ACCEPT;
}