Bugzilla – Attachment 18842 Details for
Bug 10445
SIP Server doesn't respect maxoutstanding system preference
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10445 - SIP Server doesn't respect maxoutstanding system preference
Bug-10445---SIP-Server-doesnt-respect-maxoutstandi.patch (text/plain), 2.00 KB, created by
Kyle M Hall (khall)
on 2013-06-11 14:04:28 UTC
(
hide
)
Description:
Bug 10445 - SIP Server doesn't respect maxoutstanding system preference
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2013-06-11 14:04:28 UTC
Size:
2.00 KB
patch
obsolete
>From 87e6e5d31b872b7d05c67b0052ab20a945455851 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Tue, 11 Jun 2013 10:00:58 -0400 >Subject: [PATCH] Bug 10445 - SIP Server doesn't respect maxoutstanding system preference > >The system preference 'maxoutstanding' is defined as the maximum amount >of fees owed by the patron before Koha should block placing holds ( >terrible naming on this one ). > >However, although the Koha OPAC respects this preference, placing holds >via a SIP2 device will not. > >Test Plan: >1) Set maxoutstanding to $10 >2) Pick a patron owning more than $10 in fees >3) Attempt to place a hold for this patron from a SIP2 device > This attempt should succeed >4) Apply this patch >5) Restart your SIP2 server >6) Attempt to place a hold for this patron from a SIP2 device again > This attempt should now fail >--- > C4/SIP/ILS.pm | 4 ++++ > C4/SIP/ILS/Patron.pm | 6 ++++++ > 2 files changed, 10 insertions(+), 0 deletions(-) > >diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm >index 5d59850..703bb96 100644 >--- a/C4/SIP/ILS.pm >+++ b/C4/SIP/ILS.pm >@@ -254,6 +254,10 @@ sub add_hold { > return $trans; > } > >+ if ( $patron->holds_blocked_by_excessive_fees() ) { >+ $trans->screen_msg("Excessive fees blocking placement of hold."); >+ } >+ > if ($item->fee and $fee_ack ne 'Y') { > $trans->screen_msg = "Fee required to place hold."; > return $trans; >diff --git a/C4/SIP/ILS/Patron.pm b/C4/SIP/ILS/Patron.pm >index ac24d2f..da4290b 100644 >--- a/C4/SIP/ILS/Patron.pm >+++ b/C4/SIP/ILS/Patron.pm >@@ -321,10 +321,16 @@ sub excessive_fees { > my $self = shift; > return ($self->fee_amount and $self->fee_amount > $self->fee_limit); > } >+ > sub excessive_fines { > my $self = shift; > return $self->excessive_fees; # excessive_fines is the same thing as excessive_fees for Koha > } >+ >+sub holds_blocked_by_excessive_fees { >+ return ( $self->fee_amount >+ && $self->fee_amount > C4::Context->preference("maxoutstanding") ); >+} > > sub library_name { > my $self = shift; >-- >1.7.2.5
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 10445
:
18841
|
18842
|
19203
|
22143
|
22145
|
22146
|
22147