LP#1777698 Duplicate barcode alert appearing on new barcodes user/dbriem/lp1777698_duplicate_warning_on_new_barcodes
authorDan Briem <dbriem@wlsmail.org>
Sun, 11 Aug 2019 01:02:00 +0000 (21:02 -0400)
committerDan Briem <dbriem@wlsmail.org>
Sun, 11 Aug 2019 01:02:00 +0000 (21:02 -0400)
commit67eb0767170176b75f4fb304ece2ef8a5acbf5a8
treef91dffc658e50501427bf368e149f8359d129be3
parent53dd40504d92f3092e0bf98a94fe4592a0dc49c0
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>
Open-ILS/web/js/ui/default/staff/cat/volcopy/app.js