Trim away whitespace from barcodes

Bug #1197050 reported by Pasi Kallinen
22
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

*EG master

In checkout, when pasting a barcode it may have accidental spaces or tabs in the beginning and end. Trim those away.

Branch at user/paxed/trim-whitespace-barcode-checkout

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

Sounds good in principle, but I know we unfortunately have "legitimate" barcodes employed in our system that have used space as a character. Adding this would break those barcodes from scanning.

Worth exploring though, and perhaps something we can eventually make use of.

Revision history for this message
Pasi Kallinen (paxed) wrote :

Ben, you've got whitespace in the beginning or end of barcodes? That's ... amazing.

Revision history for this message
Tim Spindler (tspindler-cwmars) wrote :

Not allowing any whitespace for us at C/W MARS would be a good thing.

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

Since we're the only wacky people so far, I'll focus on cleaning up our database of strangeness and not stand in the way of this proposed feature.... marking bug target for 2.5 consideration.

Changed in evergreen:
milestone: none → 2.5.0-alpha1
status: New → Triaged
Revision history for this message
Ben Shum (bshum) wrote :

Pushed to master. Because this changes behavior, I included a short release note entry in the Circulation group.

Changed in evergreen:
status: Triaged → 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.

Other bug subscribers

Remote bug watches

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