Bug 29987: (follow-up) Fix missing payment type

The payment type include has changed since the patch was first written,
we now require a 'type' is passed to properly set the field name.

This should fix the 'bankable' issue raised.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit a059392490)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 6d0af80314)
Signed-off-by: Arthur Suzuki <arthur.suzuki@biblibre.com>
This commit is contained in:
Martin Renvoize 2022-08-24 11:23:25 +01:00 committed by Arthur Suzuki
parent 1cb69bc22a
commit 07d8edefd9
2 changed files with 7 additions and 7 deletions

View File

@ -69,7 +69,7 @@
<li><label for="desc">Description: </label><input type="text" name="desc" size="50" id="desc" /></li>
<li><label for="note">Note: </label><input type="text" name="note" size="50" id="note" /></li>
<li><label for="amount">Amount: </label><input type="text" inputmode="decimal" pattern="^\d+(\.\d{2})?$" name="amount" id="amount" required="required" min="0" value=""/> Example: 5.00</li>
[% INCLUDE 'transaction_types.inc' %]
[% INCLUDE 'transaction_types.inc' type="credit" %]
[% IF Koha.Preference('UseCashRegisters') %]
<li>
[% IF Koha.Preference('RequireCashRegister') %]

View File

@ -85,11 +85,11 @@ if ($add) {
my $item = Koha::Items->find( { barcode => $barcode } );
$item_id = $item->itemnumber if $item;
}
my $description = $input->param('desc');
my $note = $input->param('note');
my $amount = $input->param('amount') || 0;
my $type = $input->param('type');
my $payment_type = $input->param('payment_type');
my $description = $input->param('desc');
my $note = $input->param('note');
my $amount = $input->param('amount') || 0;
my $type = $input->param('type');
my $credit_type = $input->param('credit_type');
my $cash_register_id = $input->param('cash_register');
$patron->account->add_credit(
@ -102,7 +102,7 @@ if ($add) {
type => $type,
user_id => $logged_in_user->id,
interface => C4::Context->interface,
payment_type => $payment_type,
payment_type => $credit_type,
cash_register => $cash_register_id
}
);