]> git.evergreen-ils.org Git - working/Evergreen.git/commit
LP#1777698 Duplicate barcode alert appearing on new barcodes
authorDan Briem <dbriem@wlsmail.org>
Sun, 11 Aug 2019 01:02:00 +0000 (21:02 -0400)
committerMike Rylander <mrylander@gmail.com>
Mon, 21 Oct 2019 13:39:36 +0000 (09:39 -0400)
commit43662fd8cabc3d25edf61edc3bca8c954982ccf6
tree11773f1aad5623a49748afa98cc49667017acd1a
parent5dadc54f88e5ef3771664bded5312fb5d9131c4f
LP#1777698 Duplicate barcode alert appearing on new barcodes

When scanning new copy barcodes, ng-change quickly fires multiple times
and sends promises to check for duplicates. If 1234 exists and you scan
12345, if the 1234 promise resolves after 12345, it will indicate it's a
duplicate. This patch keeps a count of the promises sent and only the
latest promise will set the boolean to indicate if it's a duplicate.

Hard to test because the promises usually resolve in order:
1. Add a new holding with the first few characters of an unused barcode
2. Scan the new barcode multiple times until you see the dup error
3. Apply patch and scan the new barcode multiple times (no error)

Signed-off-by: Dan Briem <dbriem@wlsmail.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js