projects
/
koha_gimpoz
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
bug 7408 follow-up DBRev number
[koha_gimpoz]
/
reserve
/
request.pl
diff --git
a/reserve/request.pl
b/reserve/request.pl
index
5d59e2f
..
24557f4
100755
(executable)
--- a/
reserve/request.pl
+++ b/
reserve/request.pl
@@
-112,26
+112,24
@@
if ( $action eq 'move' ) {
}
if ($findborrower) {
}
if ($findborrower) {
- my ( $count, $borrowers ) =
- SearchMember($findborrower, 'cardnumber', 'web' );
+ my $borrowers = Search($findborrower, 'cardnumber');
- my @borrowers = @$borrowers;
-
- if ( !@borrowers ) {
+ if ($borrowers && @$borrowers) {
+ if ( @$borrowers == 1 ) {
+ $borrowernumber_hold = $borrowers->[0]->{'borrowernumber'};
+ }
+ else {
+ $borrowerslist = $borrowers;
+ }
+ } else {
$messageborrower = "'$findborrower'";
}
$messageborrower = "'$findborrower'";
}
- elsif ( @borrowers == 1 ) {
- $borrowernumber_hold = $borrowers[0]->{'borrowernumber'};
- }
- else {
- $borrowerslist = \@borrowers;
- }
}
# If we have the borrowernumber because we've performed an action, then we
# don't want to try to place another reserve.
if ($borrowernumber_hold && !$action) {
}
# If we have the borrowernumber because we've performed an action, then we
# don't want to try to place another reserve.
if ($borrowernumber_hold && !$action) {
- my $borrowerinfo = GetMember
Details(
$borrowernumber_hold );
+ my $borrowerinfo = GetMember
( borrowernumber =>
$borrowernumber_hold );
my $diffbranch;
my @getreservloop;
my $count_reserv = 0;
my $diffbranch;
my @getreservloop;
my $count_reserv = 0;
@@
-194,9
+192,9
@@
if ($borrowerslist) {
foreach my $borrower (
sort {
foreach my $borrower (
sort {
- $a->{surname}
- . $a->{firstname}
cmp
$b->{surname}
- . $b->{firstname}
+
uc(
$a->{surname}
+ . $a->{firstname}
) cmp uc(
$b->{surname}
+ . $b->{firstname}
)
} @{$borrowerslist}
)
{
} @{$borrowerslist}
)
{
@@
-219,8
+217,8
@@
if ($borrowerslist) {
);
}
);
}
-# FIXME launch another time GetMember
Details
perhaps until
-my $borrowerinfo = GetMember
Details(
$borrowernumber_hold );
+# FIXME launch another time GetMember perhaps until
+my $borrowerinfo = GetMember
( borrowernumber =>
$borrowernumber_hold );
my @biblionumbers = ();
my $biblionumbers = $input->param('biblionumbers');
my @biblionumbers = ();
my $biblionumbers = $input->param('biblionumbers');
@@
-375,7
+373,7
@@
foreach my $biblionumber (@biblionumbers) {
# checking reserve
my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($itemnumber);
# checking reserve
my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($itemnumber);
- my $ItemBorrowerReserveInfo = GetMember
Details( $reservedfor, 0
);
+ my $ItemBorrowerReserveInfo = GetMember
( borrowernumber => $reservedfor
);
if ( defined $reservedate ) {
$item->{backgroundcolor} = 'reserved';
if ( defined $reservedate ) {
$item->{backgroundcolor} = 'reserved';
@@
-544,7
+542,7
@@
foreach my $biblionumber (@biblionumbers) {
}
# get borrowers reserve info
}
# get borrowers reserve info
- my $reserveborrowerinfo = GetMember
Details( $res->{'borrowernumber'}, 0
);
+ my $reserveborrowerinfo = GetMember
( borrowernumber => $res->{'borrowernumber'}
);
if (C4::Context->preference('HidePatronName')){
$reserve{'hidename'} = 1;
$reserve{'cardnumber'} = $reserveborrowerinfo->{'cardnumber'};
if (C4::Context->preference('HidePatronName')){
$reserve{'hidename'} = 1;
$reserve{'cardnumber'} = $reserveborrowerinfo->{'cardnumber'};