Transferring holds functions need more granular permissions

Bug #1350377 reported by Michele Morgan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Triaged
Wishlist
Unassigned

Bug Description

The UPDATE_HOLD permission controls the ability to edit individual holds. The same permission also allows a staff user to perform the following functions:

- Actions for this Record --> Transfer all title holds
- Actions for Selected Holds --> Transfer to Marked Title

Use case:

Circulation staff frequently assist patrons with holds so they need to be able to edit individual holds on behalf of patrons. Their duties do not require them to transfer holds in batch.

Technical services staff, when entering and deleting items, need to be able to select individual holds and transfer them to different titles whenever appropriate.

Transferring all title level holds to another record, outside of merging two records, is rare. Permission for this fuction, if granted at all, should be reserved only for administrators.

We have had situations where all title level holds have been transferred to a different title in error because the user intended to click the "Actions for selected holds", but instead clicked the "Actions for this record" button by mistake.

Related bug:

https://bugs.launchpad.net/evergreen/+bug/1312824

Related IRC discussion:

http://irc.evergreen-ils.org/evergreen/2014-07-29#i_113987

Revision history for this message
Kathy Lussier (klussier) wrote :

I really don't think this option is necessary since there is an alternate way to transfer a batch of holds in the client. I would advocate for removing that option altogether. I've posted a message to the list to see if there is support for removing it.

http://markmail.org/message/jsrbhxlwr2dshglv

Revision history for this message
Kathy Lussier (klussier) wrote :

Based on the feedback in the above-referenced e-mail thread, I have created a working branch that removes the "Transfer all title holds" option in both the xul client and the web client. I have left the transfer_holds_to_marked function in app.js because I didn't know if it would be needed later when adding this option to the record bucket interface in the web client.

If I should be removing the function, just let me know and I will do so.

Release notes entry has also been added.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1350377-remove-transfer-all-title-holds-option

tags: added: pullrequest
Changed in evergreen:
milestone: none → 2.next
importance: Undecided → Wishlist
status: New → Incomplete
status: Incomplete → Triaged
Revision history for this message
Kathy Lussier (klussier) wrote :

Removing pullrequest tag. I took a stab at making a YAOUS to control whether this menu option is hidden or not after receiving this request in the above-referenced email thread. However, I didn't have much luck getting it to work.

Changed in evergreen:
milestone: 2.next → none
tags: removed: pullrequest
Changed in evergreen:
assignee: nobody → Victoria Lewis (sykeslewis)
assignee: Victoria Lewis (sykeslewis) → nobody
tags: added: permissions
tags: added: holds
Dan Briem (dbriem)
tags: added: circ-holds
removed: holds
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.