Tpac - placing holds and barcode scanners

Bug #1004093 reported by Ben Shum
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.2
Fix Released
Medium
Unassigned
2.3
Fix Released
Medium
Unassigned

Bug Description

When using the staff client to place a hold using Tpac, the interface asks for the patron's barcode to begin. As one enters this information, it detects user preferences, etc. for the hold and updates the same page accordingly. However, if a staff member uses a barcode scanner to read the barcode and scan it into the place hold field, the auto-return from the scanner can automatically trigger the attempt to place the hold without giving staff the opportunity to review options and/or change hold settings (such as notification options / pickup library, etc.) This skips them straight to hold success or failure and they have to find the hold via view holds on the title or via the patron record itself to set options as actually desired.

In contrast, the workflow in JSPac hold placement was three screens:

1 - Enter recipient barcode
2 - Set hold options
3 - Hold successfully placed (or not) message

That initial screen added extra clicks to the process, but allowed staff to use barcode scanners without incident.

For Tpac, I'm not sure what the best approach would be to solving this particular user problem yet, just reporting for now.

Revision history for this message
Sarah Childs (sarahc) wrote :

Related problem: When you enter the barcode it loads user preferences, but doesn't display the user's name.

The Set hold options screen in the JSPac displayed the name of the patron, giving staff the chance to confirm that the hold is being placed for the correct person.

If you are keying in the patron number, and miskey (or correctly enter a wrong number mistakenly provided by a patron) a valid number, you won't know that the hold was placed for the wrong person until the complaints roll in.

Ben Shum (bshum)
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Thomas Berezansky (tsbere) wrote :

I don't consider the patron's name not showing up to be closely related enough to warrant sharing this ticket. I suggest that be opened as a new ticket.

The branch below should prevent enter from submitting from that entry box, though.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/no_hold_autosubmit

tags: added: pullrequest
Revision history for this message
Sarah Childs (sarahc) wrote :

OK--here's the new bug report. https://bugs.launchpad.net/evergreen/+bug/1053635

To me they were related because both problems were the result no longer having the intervening confirmation screen.

Revision history for this message
Ben Shum (bshum) wrote :

Tested Thomas' fix and works for us. Thanks!

Sign-off: user/bshum/no_hold_autosubmit

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/no_hold_autosubmit

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Thanks to all three of you. Pushed to master and back through rel_2_2.

Changed in evergreen:
milestone: none → 2.4.0-alpha
status: Confirmed → Fix Committed
Ben Shum (bshum)
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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