Enhance SIP support for fine item detail

Bug #1731021 reported by Dan Pearl
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

There is existing support in SIP to generate fine item detail. This is hardwired to provide a 3M-style of output. While the 3M format is practically a defacto standard, other providers of self-check equipment have specified their own, incompatible formats.

This is a wishlist item to provide configurable support for other manufacturer's equipment.

This requires a change to the SIP product and to the Evergreen product.

Revision history for this message
Dan Pearl (dpearl) wrote :

I have implementation currently in internal review.

Changed in evergreen:
assignee: nobody → Dan Pearl (dpearl)
Revision history for this message
Dan Pearl (dpearl) wrote :

As it turns out, there was no change needed to the SIP product (just the Evergreen)

Internal review and testing complete.

Branch at working
user/dpearl/LP1731021_fine_item_detail

tags: added: pullrequest
Changed in evergreen:
assignee: Dan Pearl (dpearl) → nobody
Changed in evergreen:
importance: Undecided → Wishlist
Revision history for this message
Jason Stephenson (jstephenson) wrote :

I am assigning myself to see if the vendor is still interested in testing this before it goes in.

Changed in evergreen:
assignee: nobody → Jason Stephenson (jstephenson)
tags: removed: pullrequest
Revision history for this message
Mike Rylander (mrylander) wrote :

If this does end up going in, I think the setting should probably be moved to be an attribute on the <login> element. That ties the format to the specific client, rather that setting it for an entire <institution>. Opinions?

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

So, I was not thinking straight when I took this bug over. I don't have time to look at it right now. I'm assigning it to Dan Pearl and will ask him to do the testing with the vendor and to make the changes from Mike's comment #4.

Changed in evergreen:
assignee: Jason Stephenson (jstephenson) → nobody
assignee: nobody → Dan Pearl (dpearl)
Changed in evergreen:
assignee: Dan Pearl (dpearl) → nobody
assignee: nobody → Jason Stephenson (jstephenson)
Changed in evergreen:
assignee: Jason Stephenson (jstephenson) → nobody
Revision history for this message
Jason Stephenson (jstephenson) wrote :

I am reworking Dan's branch and may just start this over.

Changed in evergreen:
status: New → In Progress
assignee: nobody → Jason Stephenson (jstephenson)
Revision history for this message
Jason Stephenson (jstephenson) wrote :

And, I'm not really working on this any more.

Changed in evergreen:
assignee: Jason Stephenson (jstephenson) → nobody
status: In Progress → Won't Fix
status: Won't Fix → Triaged
tags: added: sip
Revision history for this message
Bill Erickson (berick) wrote :

Kicking the dust off this one. Will post a follow-up branch soon.

Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
status: Triaged → In Progress
Revision history for this message
Bill Erickson (berick) wrote :

New branch pushed:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1731021-sip-fine-item-detail

* Adds a sign-off to Dan's first commit.
* Implements Dan's second commit manually since the working branch was not in a usable state (merge conflict markers, unintended deletions, etc.)
* Migrates the title/author retrieval code to use metabib display fields instead of MVR's.
* General code cleanup

As an example, here's a debug excerpt from SIP call (using pysip2) to a server configured to use "3m" AV format:

[AV] fine items : 487 $3.33 "FINE" Misc test test
[AV] fine items : 313 $0.20 "FINE" Overdue materials Piano concerto no. 2, op. 83, B flat major/Brahms, Johannes 1833-1897
[AV] fine items : 315 $0.90 "FINE" Overdue materials Four concertos with oboe and string orchestra/Handel, George Frideric 1685-1759
[AV] fine items : 316 $3.00 "FINE" Overdue materials The Baroque concerto/Hutchings, Arthur 1906-
[AV] fine items : 318 $2.30 "FINE" Overdue materials Complete transcriptions, cadenzas, and exercises, for solo piano/Brahms, Johannes 1833-1897
[AV] fine items : 314 $1.70 "FINE" Overdue materials Concerto, op. 4, no. 6, B flat major ; Concerto, op. 4, no. 5, F major ; Concerto grosso, C major (Alexander's feast)/Handel, George Frideric 1685-1759

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.4-beta1
assignee: Bill Erickson (berick) → nobody
status: In Progress → Confirmed
Revision history for this message
Bill Erickson (berick) wrote :

I have pushed another commit to this branch integrating some additional features coded by Jason Stephenson (see working/user/dyrcona/sip2-envisionware-fid) to support specifying the fee type as fine/fee/lost depending on the last billing type of each transaction.

Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master for inclusion in 3.4. Thanks, Dan and Bill!

Changed in evergreen:
status: Confirmed → Fix Committed
Galen Charlton (gmc)
Changed in evergreen:
status: Fix Committed → Fix Released
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.