Evergreen

Item Status - allow paste of multiple barcodes in CSV format into scan box

Bug #1819367 reported by Josh Stompro
30
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned
Evergreen 3.6-beta

Bug Description

EG 3.2

I would like to be able to paste a string consisting of multiple barcodes in csv format into the item status scan box, and have item status load all the barcodes.

I'm often sending and receiving lists of item barcodes to take a look at, from reports where the items are summarized in a string of comma separated barcodes.

It would be a time saver to be able to just copy and paste the the string of barcodes into the scan box, vs needing to format them as a file of barcodes to upload, or pasting them one at a time.

I would also like to see this same ability in the retrieve bib record by ID form, and the retrieve patron by ID form, to cover the cases of lists of bibs and patrons that get passed around.

Josh

Revision history for this message
Elaine Hardy (ehardy) wrote :

+1

Changed in evergreen:
status: New → Confirmed
Terran McCanna (tmccanna)
Changed in evergreen:
importance: Undecided → Wishlist
Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

Branch at user/stompro/lp1819367_item_status_csv_paste

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/stompro/lp1819367_item_status_csv_paste

This branch adds the ability to paste in a string of barcodes seperated by commas to the item status scan box.

It checks to see if splitting the barcode by commas results in multiple barcodes, and passed the barcodes to the file upload handler if multiple barcodes seem to be there.

If not, then it continues with the original barcode.

So pasting a string like
33500012542726, 35500004468045, 35500005727696 , 35500006028581, , 35500004587208,33500013019179,35500003474093,

Is equivalent to uploading a file with the contents of.
33500012542726
35500004468045
35500005727696
35500006028581
35500004587208
33500013019179
35500003474093

A barcode with only a trailing comma will fail, since if there is only one barcode the original value is used.

So "35500004468045," will fail.

That could be addressed, but I just don't see it as a big issue. If you only have one barcode, just paste the barcode.

Josh

Josh Stompro (u-launchpad-stompro-org)
tags: added: pullrequest
Revision history for this message
Elaine Hardy (ehardy) wrote :

Tested on https://bugs.launchpad.net/evergreen/+bug/1819367

I was unable to get a pasted list to work. Whether I used comma space, comma, space, semicolon, error message Item Not found was returned and no barcodes displayed

Revision history for this message
Elaine Hardy (ehardy) wrote :

Looks like the fix may not have been added to that sandbox... will test again when we verify it has

Revision history for this message
Elaine Hardy (ehardy) wrote :

Tested again and works as expected. I tested a list of barcodes from a single record and barcodes from multiple record and all were retrieved!

I also added a precat barcode and a made up barcode. The list loaded all the existing barcodes including the precat one. There was no error message for the nonexisting barcode.

Revision history for this message
Elaine Hardy (ehardy) wrote :

I have tested this code and consent to signing off on it with my name, Elaine Hardy and by email: <email address hidden>

Revision history for this message
Terran McCanna (tmccanna) wrote :

The New Developers Working Group also tested this and signed off:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1819367_item_status-csv_paste_signoff

tags: added: signedoff
Terran McCanna (tmccanna)
Changed in evergreen:
milestone: none → 3.5-alpha
Evergreen Bug Maintenance (bugmaster)
Changed in evergreen:
milestone: 3.5-beta → 3.5.0
Evergreen Bug Maintenance (bugmaster)
Changed in evergreen:
milestone: 3.5.0 → 3.5.1
Galen Charlton (gmc)
Changed in evergreen:
milestone: 3.5.1 → 3.6-beta
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
Revision history for this message
Bill Erickson (berick) wrote :

Works great for me. A nice addition. Merged to master for 3.6. Thanks!

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Bill Erickson (berick) → nobody
Revision history for this message
Jason Etheridge (phasefx) wrote :

just noting that on bug 1332651 I'm going to offer a branch to tweak the whitespace stripping behavior

Evergreen Bug Maintenance (bugmaster)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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