LP#1778783 - Circulate.pm fix log_me correct barcode var
authorJosh Stompro <stompro@stompro.org>
Thu, 28 Feb 2019 15:43:49 +0000 (09:43 -0600)
committerBill Erickson <berickxx@gmail.com>
Thu, 7 Apr 2022 15:53:53 +0000 (11:53 -0400)
The log_me sub wasn't using the correct variable for showing the entered
barcode.  It should be $self->copy_barcode instead of $self->barcode.

Testing Notes:

Before fix:
1. Watch the logs with something like
  tail -f osrfsys.log | fgrep 'circulator: do_permit()'
2. Perform a checkout of a non-existant barcode.
3. Notice that the entered barcode is missing after 'copy='

After fix:
1. Restart the circ openils service
  osrf_control -l --service open-ils.circ --restart
2. Watch the logs.
3. Perform a checkout of a non-existant barcode.
4. Notice that the barcode is now shown after 'copy='

Signed-off-by: Josh Stompro <stompro@stompro.org>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm

index c2047e3..0e417c5 100644 (file)
@@ -4040,7 +4040,7 @@ sub checkin_flesh_events {
 sub log_me {
     my( $self, $msg ) = @_;
     my $bc = ($self->copy) ? $self->copy->barcode :
-        $self->barcode;
+        $self->copy_barcode;
     $bc ||= "";
     my $usr = ($self->patron) ? $self->patron->id : "";
     $logger->info("circulator: $msg requestor=".$self->editor->requestor->id.