SIP2 "Chargeable Loans"

Bug #806131 reported by Jason Stephenson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Bill Erickson
Declined for Main by Jason Stephenson

Bug Description

3M and likely other Self-check vendors support a mode called "chargeable loans." In this mode the ILS is able to report that a fee is required for the circulation transaction to complete to the SC. The SC then asks the patron if they accept the fee. If the patron accepts the fee, the SC tries the checkout again with the fee accepted field "BO" set to "Y".

I have been working on some code to implement this feature in Evergreen's SIP2 modules. It requires updates to Evergreen and to SIPServer in order to function properly.

I've only tested this with our own PHPSIP2 module, so it may not actually work as expected with an actual self-check. (I'm trying to get some consortium members to test it, but that is never as easy as it sounds.)

I'm posting this bug to call for anyone who'd like to try the code out and so that it doesn't get lost in the to and fro of other work.

You can get the SIPServer changes in the following branch:

git://git.evergreen-ils.org/working/SIPServer <email address hidden>/chargeable_loans
or at
git://git.mvlcstaff.org/jason/SIPServer chargeable_loans

The code necessary for Evergreen is at

git://git.evergreen-ils.org/working/Evergreen <email address hidden>/SIP2_chargeable_loans
or at
git://git.mvlcstaff.org/jason/ILS SIP2_chargeable_loans

Currently, the code does not conflict with the SIP23738 branch mentioned in another "bug," so you can test them independently of each other or at the same time.

description: updated
Revision history for this message
Jason Stephenson (jstephenson) wrote :

I went ahead and added a backport branch for rel_2_1 for this code. It was a clean cherry-pick of the relevant bits from master and the SIP2_chargeable_loans branch:

git://git.evergreen-ils.org/working/Evergreen.git <email address hidden>/SIP2_chargeable_loans_rel_2_1

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Something else that I forgot to mention in the details above:

If you are going to use this code and you have not already done so, you will need to add permissions for ITEM_RENTAL_FEE_REQUIRED.override and ITEM_DEPOSIT_FEE_REQUIRED.override and grant these to your SIP circulation group.

Without these permissions, the Evergreen circulation code will reject the checkout and not charge the fee.

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Did some testing on a 3M self check this afternoon, and we will be using this in production starting next week.

Revision history for this message
Jason Stephenson (jstephenson) wrote :

I am told by the members who are using it that this feature "just works."

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Steps to use this code:

First, get SIPServer:

git clone git://git.evergreen-ils.org/SIPServer.git
cd SIPServer
git remote add working git://git.evergreen-ils.org/working/SIPServer.git
git fetch working
git checkout -b chargeable_loans <email address hidden>/chargeable_loans
cd ..

Now, get Evergreen:
git clone git://git.evergreen-ils.org/Evergreen.git
cd Evergreen
git remote add working git://git.evergreen-ils.org/working/Evergreen.git
git fetch working
git checkout -b SIP2_chargeable_loan origin/master
git merge <email address hidden>/SIP2_chargeable_loans

Then, do the ./autogen.sh; configure; make ; make install dance.

Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (erickson-esilibrary)
Bill Erickson (berick)
Changed in evergreen:
status: New → In Progress
Bill Erickson (berick)
tags: removed: pullrequest
Revision history for this message
Bill Erickson (berick) wrote :

Both branches merged to their relative master repositories.

Changed in evergreen:
status: In Progress → Fix Committed
Changed in evergreen:
milestone: none → 2.2.0
Changed in evergreen:
status: Fix Committed → Fix Released
no longer affects: evergreen/2.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.