Bugzilla – Attachment 106056 Details for
Bug 25812
Fines can be displayed on SIP checkin/checkout
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 25812: Display overdue fines for the document on SIP checkin.
Bug-25812-Display-overdue-fines-for-the-document-o.patch (text/plain), 1.67 KB, created by
Matthias Meusburger
on 2020-06-19 08:59:10 UTC
(
hide
)
Description:
Bug 25812: Display overdue fines for the document on SIP checkin.
Filename:
MIME Type:
Creator:
Matthias Meusburger
Created:
2020-06-19 08:59:10 UTC
Size:
1.67 KB
patch
obsolete
>From 20e67d93e111e2526cad89607d5f2a6361c7d96e Mon Sep 17 00:00:00 2001 >From: Matthias Meusburger <matthias.meusburger@biblibre.com> >Date: Wed, 25 Sep 2019 15:30:51 +0200 >Subject: [PATCH] Bug 25812: Display overdue fines for the document on SIP > checkin. > >Test plan: > > - Using SIP, checkin an item that has one or several overdue fines. > - Check that the following message is displayed (AF field): > "You owe {correctly formatted price} for this document." >--- > C4/SIP/ILS.pm | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > >diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm >index 7cff9c0579..5a7cef0335 100644 >--- a/C4/SIP/ILS.pm >+++ b/C4/SIP/ILS.pm >@@ -239,6 +239,21 @@ sub checkin { > delete $item->{borrowernumber}; > delete $item->{due_date}; > $patron->{items} = [ grep { $_ ne $item_id } @{ $patron->{items} } ]; >+ # Check for overdue fines to display >+ my $kohaitem = Koha::Items->find( { barcode => $item_id } ); >+ if ($kohaitem) { >+ my $charges = Koha::Account::Lines->search( >+ { >+ borrowernumber => $patron->{borrowernumber}, >+ amountoutstanding => { '>' => 0 }, >+ accounttype => [ 'OVERDUE' ], >+ itemnumber => $kohaitem->itemnumber >+ }, >+ ); >+ if ($charges) { >+ $circ->screen_msg("You owe " . Koha::Number::Price->new( $charges->total_outstanding )->format({ with_symbol => 1}) . " for this document."); >+ } >+ } > } else { > # Checkin failed: Wrongbranch or withdrawn? > # Bug 10748 with pref BlockReturnOfLostItems adds another case to come >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 25812
:
106056
|
106340
|
139690
|
144217
|
144476