Add comments about CheckOutItem barcodes
authorJeff Godin <jgodin@tadl.org>
Wed, 11 Sep 2013 17:52:38 +0000 (13:52 -0400)
committerJeff Godin <jgodin@tadl.org>
Wed, 11 Sep 2013 17:52:38 +0000 (13:52 -0400)
Add comments about item barcode assumptions when responding in a
User Agency vs Item Agency CheckOutItem scenario.

Signed-off-by: Jeff Godin <jgodin@tadl.org>
iNCIPit.cgi

index 9e04dfe..703412d 100644 (file)
@@ -557,6 +557,7 @@ sub check_out_item {
     my $mdate = $doc->findvalue('/NCIPMessage/CheckOutItem/MandatedAction/DateEventOccurred');
     my $patron_barcode = "zyyyy";    # XXX: CUSTOMIZATION NEEDED XXX institution/eg_as_item_agency user lookup here
 
+    # For CheckOutItem and INN-REACH, this value will correspond with our local barcode
     my $barcode = $doc->findvalue('/NCIPMessage/CheckOutItem/UniqueItemId/ItemIdentifierValue');
 
     # TODO: watch for possible real ids here?
@@ -611,6 +612,7 @@ sub check_in_item {
     my $taidScheme = HTML::Entities::encode($taidSchemeX);
     my $taidValue  = $doc->find('/NCIPMessage/CheckInItem/InitiationHeader/ToAgencyId/UniqueAgencyId/Value');
 
+    # For CheckInItem and INN-REACH, this value will correspond with our local barcode
     my $barcode = $doc->findvalue('/NCIPMessage/CheckInItem/UniqueItemId/ItemIdentifierValue');
     my $r = checkin($barcode);
     fail($r) if $r =~ /^COPY_NOT_CHECKED_OUT/;