Bug 15253: Add Koha::Logger based logging for SIP2
[koha-equinox.git] / C4 / SIP / ILS / Item.pm
index 4586fee..7e246bc 100644 (file)
@@ -9,8 +9,9 @@ package C4::SIP::ILS::Item;
 use strict;
 use warnings;
 
-use Sys::Syslog qw(syslog);
+use C4::SIP::Sip qw(syslog);
 use Carp;
+use Template;
 
 use C4::SIP::ILS::Transaction;
 
@@ -22,8 +23,11 @@ use C4::Circulation;
 use C4::Members;
 use C4::Reserves;
 use Koha::Database;
-
-our $VERSION = 3.07.00.049;
+use Koha::Biblios;
+use Koha::Checkouts;
+use Koha::DateUtils;
+use Koha::Patrons;
+use Koha::Items;
 
 =encoding UTF-8
 
@@ -64,51 +68,46 @@ our $VERSION = 3.07.00.049;
         hold_queue => [],
     },
 );
-=cut
 
-sub priority_sort {
-    defined $a->{priority} or return -1;
-    defined $b->{priority} or return 1;
-    return $a->{priority} <=> $b->{priority};
-}
+=cut
 
 sub new {
-       my ($class, $item_id) = @_;
-       my $type = ref($class) || $class;
-       my $self;
-    my $itemnumber = GetItemnumberFromBarcode($item_id);
-       my $item = GetBiblioFromItemNumber($itemnumber);    # actually biblio.*, biblioitems.* AND items.*  (overkill)
-       if (! $item) {
-               syslog("LOG_DEBUG", "new ILS::Item('%s'): not found", $item_id);
-               warn "new ILS::Item($item_id) : No item '$item_id'.";
+    my ($class, $item_id) = @_;
+    my $type = ref($class) || $class;
+    my $item = Koha::Items->find( { barcode => $item_id } );
+    unless ( $item ) {
+        syslog("LOG_DEBUG", "new ILS::Item('%s'): not found", $item_id);
+        warn "new ILS::Item($item_id) : No item '$item_id'.";
         return;
-       }
-    $item->{  'itemnumber'   } = $itemnumber;
-    $item->{      'id'       } = $item->{barcode};     # to SIP, the barcode IS the id.
-    $item->{permanent_location}= $item->{homebranch};
-    $item->{'collection_code'} = $item->{ccode};
-    $item->{  'call_number'  } = $item->{itemcallnumber};
+    }
+    my $self = $item->unblessed;
+    $self->{      'id'       } = $item->barcode;     # to SIP, the barcode IS the id.
+    $self->{permanent_location}= $item->homebranch;
+    $self->{'collection_code'} = $item->ccode;
+    $self->{  'call_number'  } = $item->itemcallnumber;
 
-    my $it = C4::Context->preference('item-level_itypes') ? $item->{itype} : $item->{itemtype};
+    my $it = $item->effective_itemtype;
     my $itemtype = Koha::Database->new()->schema()->resultset('Itemtype')->find( $it );
-    $item->{sip_media_type} = $itemtype->sip_media_type() if $itemtype;
+    $self->{sip_media_type} = $itemtype->sip_media_type() if $itemtype;
 
-       # check if its on issue and if so get the borrower
-       my $issue = GetItemIssue($item->{'itemnumber'});
+    # check if its on issue and if so get the borrower
+    my $issue = Koha::Checkouts->find( { itemnumber => $item->itemnumber } );
     if ($issue) {
-        $item->{due_date} = $issue->{date_due};
+        $self->{due_date} = dt_from_string( $issue->date_due, 'sql' )->truncate( to => 'minute' );
+        my $patron = Koha::Patrons->find( $issue->borrowernumber );
+        $self->{patron} = $patron->cardnumber;
     }
-       my $borrower = GetMember(borrowernumber=>$issue->{'borrowernumber'});
-       $item->{patron} = $borrower->{'cardnumber'};
-    my $reserves = GetReservesFromBiblionumber({ biblionumber => $item->{biblionumber} });
-    $item->{hold_queue} = [ sort priority_sort @$reserves ];
-       $item->{hold_shelf}    = [( grep {   defined $_->{found}  and $_->{found} eq 'W' } @{$item->{hold_queue}} )];
-       $item->{pending_queue} = [( grep {(! defined $_->{found}) or  $_->{found} ne 'W' } @{$item->{hold_queue}} )];
-       $self = $item;
-       bless $self, $type;
-
-    syslog("LOG_DEBUG", "new ILS::Item('%s'): found with title '%s'",
-          $item_id, $self->{title});
+    my $biblio = Koha::Biblios->find( $self->{biblionumber} );
+    my $holds = $biblio->current_holds->unblessed;
+    $self->{hold_queue} = $holds;
+    $self->{hold_shelf}    = [( grep {   defined $_->{found}  and $_->{found} eq 'W' } @{$self->{hold_queue}} )];
+    $self->{pending_queue} = [( grep {(! defined $_->{found}) or  $_->{found} ne 'W' } @{$self->{hold_queue}} )];
+    $self->{title} = $biblio->title;
+    $self->{author} = $biblio->author;
+    bless $self, $type;
+
+    syslog( "LOG_DEBUG", "new ILS::Item('%s'): found with title '%s'",
+        $item_id, $self->{title} // '' );
 
     return $self;
 }
@@ -166,31 +165,40 @@ sub hold_patron_id {
 
 }
 sub hold_patron_name {
-    my $self = shift;
-    my $borrowernumber = (@_ ? shift: $self->hold_patron_id()) or return;
-    my $holder = GetMember(borrowernumber=>$borrowernumber);
+    my ( $self, $template ) = @_;
+    my $borrowernumber = $self->hold_patron_id() or return;
+
+    if ($template) {
+        my $tt = Template->new();
+
+        my $patron = Koha::Patrons->find($borrowernumber);
+
+        my $output;
+        $tt->process( \$template, { patron => $patron }, \$output );
+        return $output;
+    }
+
+    my $holder = Koha::Patrons->find( $borrowernumber );
     unless ($holder) {
-        syslog("LOG_ERR", "While checking hold, GetMember failed for borrowernumber '$borrowernumber'");
+        syslog("LOG_ERR", "While checking hold, failed to retrieve the patron with borrowernumber '$borrowernumber'");
         return;
     }
-    my $email = $holder->{email} || '';
-    my $phone = $holder->{phone} || '';
+    my $email = $holder->email || '';
+    my $phone = $holder->phone || '';
     my $extra = ($email and $phone) ? " ($email, $phone)" :  # both populated, employ comma
                 ($email or  $phone) ? " ($email$phone)"   :  # only 1 populated, we don't care which: no comma
                 "" ;                                         # neither populated, empty string
-    my $name = $holder->{firstname} ? $holder->{firstname} . ' ' : '';
-    $name .= $holder->{surname} . $extra;
+    my $name = $holder->firstname ? $holder->firstname . ' ' : '';
+    $name .= $holder->surname . $extra;
     return $name;
 }
 
 sub hold_patron_bcode {
     my $self = shift;
     my $borrowernumber = (@_ ? shift: $self->hold_patron_id()) or return;
-    my $holder = GetMember(borrowernumber => $borrowernumber);
-    if ($holder) {
-        if ($holder->{cardnumber}) {
-            return $holder->{cardnumber};
-        }
+    my $holder = Koha::Patrons->find( $borrowernumber );
+    if ($holder and $holder->cardnumber ) {
+        return $holder->cardnumber;
     }
     return;
 }
@@ -245,13 +253,15 @@ sub title_id {
 
 sub sip_circulation_status {
     my $self = shift;
-    if ($self->{patron}) {
-               return '04';    # charged
-    } elsif (scalar @{$self->{hold_queue}}) {
-               return '08';    # waiting on hold shelf
-    } else {
-               return '03';    # available
-    }                   # FIXME: 01-13 enumerated in spec.
+    if ( $self->{patron} ) {
+        return '04';    # charged
+    }
+    elsif ( grep { $_->{itemnumber} == $self->{itemnumber}  } @{ $self->{hold_shelf} } ) {
+        return '08';    # waiting on hold shelf
+    }
+    else {
+        return '03';    # available
+    }    # FIXME: 01-13 enumerated in spec.
 }
 
 sub sip_security_marker {
@@ -350,8 +360,8 @@ sub available {
 sub _barcode_to_borrowernumber {
     my $known = shift;
     return unless defined $known;
-    my $member = GetMember(cardnumber=>$known) or return;
-    return $member->{borrowernumber};
+    my $patron = Koha::Patrons->find( { cardnumber => $known } ) or return;
+    return $patron->borrowernumber
 }
 sub barcode_is_borrowernumber {    # because hold_queue only has borrowernumber...
     my $self = shift;