Bugzilla – Attachment 18841 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), 1.33 KB, created by
Kyle M Hall (khall)
on 2013-06-11 14:03:32 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:03:32 UTC
Size:
1.33 KB
patch
obsolete
>From 4e77682ef958083c13ae2fb760a29069608deb10 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 ++++ > 1 files changed, 4 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; >-- >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