From 26de35af3432b083a71e02037625a6cbd31a1282 Mon Sep 17 00:00:00 2001 From: Bill Erickson Date: Tue, 23 Jan 2018 17:56:51 -0500 Subject: [PATCH] LP#1743608 Replace barcode discards NULL/accidental entries When replacing a patron's barcode in the patron editor, discard any previous replacement attempts done in the current edit session. This prevents cases where secondary, etc. replacement cards are accidentally created or NULL cards that result in ugly error messages. Signed-off-by: Bill Erickson Signed-off-by: Chris Sharp --- Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js b/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js index b80e307f88..8b389ae6ba 100644 --- a/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js +++ b/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js @@ -1449,6 +1449,11 @@ function($scope , $routeParams , $q , $uibModal , $window , egCore , new_card.active = true; new_card._primary = 'on'; $scope.patron.card = new_card; + + // Remove any previous attempts to replace the card, since they + // may be incomplete or created by accident. + $scope.patron.cards = + $scope.patron.cards.filter(function(c) {return !c.isnew}) $scope.patron.cards.push(new_card); } -- 2.11.0