Default hold transit slip should not include patron information

Bug #1735847 reported by Blake GH
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.2
Fix Released
Medium
Unassigned
3.3
Fix Released
Medium
Unassigned

Bug Description

It's been brought to our attention that courier services could potentially violate patron privacy while items are in transit. We should not include patron information on the slips being put into transit.

Revision history for this message
Blake GH (bmagic) wrote :
Blake GH (bmagic)
tags: added: pullrequest
Revision history for this message
Blake GH (bmagic) wrote :

I went ahead and committed the change to the xul client as well. Same git link above.

Michele Morgan (mmorgan)
Changed in evergreen:
assignee: nobody → Michele Morgan (mmorgan)
Revision history for this message
Michele Morgan (mmorgan) wrote :

The fix changes the hold_transit_slip from this:

This item needs to be routed to
BR2
Example Branch 2
BR2 - Holds and ILL
234 Side St.
Anywhere, GA 30304

Barcode: G880510001549
Title: Harry Potter and the goblet of fire Author: Rowling, J. K.
Hold for patron Anderson, Leon
Barcode: 99999376864
Notify by phone: 9

Request Date: 9/12/2018 1:15 PM
Slip Date: 9/12/2018 1:16 PM
Printed by Administrator at BR1

To this:

This item needs to be routed to
BR2
Example Branch 2
BR2 - Holds and ILL
234 Side St.
Anywhere, GA 30304

Barcode: G880510001549
Title: Harry Potter and the goblet of fire Author: Rowling, J. K.
Hold for patron 99999376864

Request Date: 9/12/2018 11:37 AM
Slip Date: 9/12/2018 1:25 PM
Printed by Administrator at BR1

I'm just wondering if it's necessary to have the patron's barcode printed or if that can be removed as well.

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

Works as advertised.

user/dpearl/LP1735847_Default_hold_transit_slip_should_not_include_patron_information-signoff

tags: added: signedoff
Changed in evergreen:
assignee: Dan Pearl (dpearl) → nobody
Revision history for this message
Jason Boyer (jboyer) wrote :

Given that this is a step in the right direction I've gone ahead and committed these changes to 3.1 to master. Another thing to consider in the future re: Michele's comments is whether we want to have the requesting patron's information even made available to the transit template at all. That's a discussion we can have in another bug.

Changed in evergreen:
status: New → Fix Committed
Revision history for this message
Jason Boyer (jboyer) wrote :

Forgot to mention, thanks Blake and Dan!

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

I think making it impossible to include the requester's information might be a step too far, as there's the potential for internal paging within a corporate library or the like to need it.

Changed in evergreen:
importance: Undecided → Medium
milestone: none → 3.3.4
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.