Selfcheck: Printing Fines Prints All Open Transactions

Bug #1551447 reported by Terran McCanna
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.8
Fix Released
Medium
Unassigned
2.9
Fix Released
Medium
Unassigned

Bug Description

When printing the list of fines from the self-check, it is printing all open billable transactions regardless of whether or not the transactions have any fines on them (see attachment).

(I have a proposed fix for this that we have working in our production environment - I'll post tomorrow.)

Evergreen 2.9

Revision history for this message
Terran McCanna (tmccanna) wrote :
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Terran McCanna (tmccanna) wrote :

I've submitted a proposed patch for this:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=ea9fda2c9e5142d1976f92788e8aa89de7757b38

To test, log in to the selfcheck interface ( /eg/circ/selfcheck/main ) through a browser, log in with an account that has permissions to set the workstation, and then log in with Concerto patron data to test printing from Fines tab:

- patrons with no items out, no bills (should not launch print function)
- patrons with items out, but no bills (should not launch print function)
- patrons with bills, but with no items out (should launch print function and print a bullet for each bill)
- patrons will bills, and with items out (should launch print function and only print bullets for the bills, and not print bullets for the open transactions that do not have bills)

tags: added: pullrequest
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

I have tested this code and consent to signing off on it with my name, Jennifer Pringle and email address, <email address hidden>.

tags: added: signedoff
Kathy Lussier (klussier)
Changed in evergreen:
milestone: none → 2.10-rc
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed all the way to rel_2_8. Thanks, Terran!

Changed in evergreen:
status: New → Confirmed
status: Confirmed → Fix Committed
importance: Undecided → Medium
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.