use C4::Members;
use C4::Reserves;
-use vars qw($VERSION @ISA @EXPORT @EXPORT_OK);
+our $VERSION = 3.07.00.049;
-BEGIN {
- $VERSION = 2.11;
- require Exporter;
- @ISA = qw(Exporter);
- @EXPORT_OK = qw();
-}
-
-=head2 EXAMPLE
+=head1 EXAMPLE
-our %item_db = (
+ our %item_db = (
'1565921879' => {
title => "Perl 5 desktop reference",
id => '1565921879',
],
},
'660' => {
- title => "Harry Potter y el cáliz de fuego",
+ title => "Harry Potter y el cáliz de fuego",
id => '660',
sip_media_type => '001',
magnetic_media => 0,
if (! $item) {
syslog("LOG_DEBUG", "new ILS::Item('%s'): not found", $item_id);
warn "new ILS::Item($item_id) : No item '$item_id'.";
- return undef;
+ return;
}
$item->{ 'itemnumber' } = $itemnumber;
$item->{ 'id' } = $item->{barcode}; # to SIP, the barcode IS the id.
# check if its on issue and if so get the borrower
my $issue = GetItemIssue($item->{'itemnumber'});
- my $borrower = GetMember($issue->{'borrowernumber'},'borrowernumber');
+ if ($issue) {
+ $item->{due_date} = $issue->{date_due};
+ }
+ my $borrower = GetMember(borrowernumber=>$issue->{'borrowernumber'});
$item->{patron} = $borrower->{'cardnumber'};
my ($whatever, $arrayref) = GetReservesFromBiblionumber($item->{biblionumber});
$item->{hold_queue} = [ sort priority_sort @$arrayref ];
);
sub next_hold {
- my $self = shift or return;
- foreach (@$self->{hold_shelf}) { # If this item was taken from the hold shelf, then that reserve still governs
+ my $self = shift;
+ # use Data::Dumper; warn "next_hold() hold_shelf: " . Dumper($self->{hold_shelf}); warn "next_hold() pending_queue: " . $self->{pending_queue};
+ foreach (@{$self->hold_shelf}) { # If this item was taken from the hold shelf, then that reserve still governs
next unless ($_->{itemnumber} and $_->{itemnumber} == $self->{itemnumber});
return $_;
}
- if (scalar @$self->{pending_queue}) { # Otherwise, if there is at least one hold, the first (best priority) gets it
+ if (scalar @{$self->{pending_queue}}) { # Otherwise, if there is at least one hold, the first (best priority) gets it
return $self->{pending_queue}->[0];
}
return;
}
+# hold_patron_id is NOT the barcode. It's the borrowernumber.
+# If a return triggers capture for a hold the borrowernumber is passed
+# and saved so that other hold info can be retrieved
sub hold_patron_id {
- my $self = shift or return;
- my $hold = $self->next_hold() or return;
- return $hold->{borrowernumber};
+ my $self = shift;
+ my $id = shift;
+ if ($id) {
+ $self->{hold}->{borrowernumber} = $id;
+ }
+ if ($self->{hold} ) {
+ return $self->{hold}->{borrowernumber};
+ }
+ return;
+
}
sub hold_patron_name {
- my $self = shift or return;
- # return $self->{hold_patron_name} if $self->{hold_patron_name}; TODO: consider caching
+ my $self = shift;
my $borrowernumber = (@_ ? shift: $self->hold_patron_id()) or return;
- my $holder = GetMember($borrowernumber, 'borrowernumber');
+ my $holder = GetMember(borrowernumber=>$borrowernumber);
unless ($holder) {
syslog("LOG_ERR", "While checking hold, GetMember failed for 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
- "" ; # niether populated, empty string
+ "" ; # neither populated, empty string
my $name = $holder->{firstname} ? $holder->{firstname} . ' ' : '';
$name .= $holder->{surname} . $extra;
- # $self->{hold_patron_name} = $name; # TODO: consider caching
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};
+ }
+ }
+ return;
+}
+
sub destination_loc {
- my $self = shift or return;
- my $hold = $self->next_hold();
- return ($hold ? $hold->{branchcode}
+ my $self = shift;
+ my $set_loc = shift;
+ if ($set_loc) {
+ $self->{dest_loc} = $set_loc;
+ }
+ if ($self->{dest_loc} ) {
+ return $self->{dest_loc};
+ }
+ return q{};
}
our $AUTOLOAD;
$status->{ok} = 1;
return $status;
}
-
+
sub title_id {
my $self = shift;
return $self->{title};
}
sub sip_security_marker {
+ my $self = shift;
return '02'; # FIXME? 00-other; 01-None; 02-Tattle-Tape Security Strip (3M); 03-Whisper Tape (3M)
}
sub sip_fee_type {
+ my $self = shift;
return '01'; # FIXME? 01-09 enumerated in spec. We just use O1-other/unknown.
}
return 0;
}
-sub _barcode_to_borrowernumber ($) {
+sub _barcode_to_borrowernumber {
my $known = shift;
- (defined($known)) or return undef;
- my $member = GetMember($known,'cardnumber') or return undef;
+ return unless defined $known;
+ my $member = GetMember(cardnumber=>$known) or return;
return $member->{borrowernumber};
}
-sub barcode_is_borrowernumber ($$$) { # because hold_queue only has borrowernumber...
- my $self = shift; # not really used
+sub barcode_is_borrowernumber { # because hold_queue only has borrowernumber...
+ my $self = shift;
my $barcode = shift;
- my $number = shift or return undef; # can't be zero
- (defined($barcode)) or return undef; # might be 0 or 000 or 000000
- my $converted = _barcode_to_borrowernumber($barcode) or return undef;
- return ($number eq $converted); # even though both *should* be numbers, eq is safer.
+ my $number = shift or return; # can't be zero
+ return unless defined $barcode; # might be 0 or 000 or 000000
+ my $converted = _barcode_to_borrowernumber($barcode);
+ return unless $converted;
+ return ($number == $converted);
}
-sub fill_reserve ($$) {
+sub fill_reserve {
my $self = shift;
- my $hold = shift or return undef;
+ my $hold = shift or return;
foreach (qw(biblionumber borrowernumber reservedate)) {
- $hold->{$_} or return undef;
+ $hold->{$_} or return;
}
return ModReserveFill($hold);
}