Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

PT 8243141: Koha batch changing hold pickup branch

If IndependantBranches is turned on, reserve/request.pl would show
only the current librarian's branch in the select box for all the
current reserves. Changing any of the reserve properties from that
screen would then re-save each of the pending reserves, calling
ModReserve with the pickup branch set to the librarian's branch.

Perhaps we should instead eliminate the display of these other
reserves altogether? There doesn't seem to be a single obvious
way to resolve that. At any rate, this patch simply calls
GetBranchesLoop() with "onlymine" forced to false. This at least
stops the data corruption that's going on.
  • Loading branch information...
commit 4b7f4fc5e425b743ea08c536b723987165f14607 1 parent 54d1f52
@pjones9 pjones9 authored
Showing with 1 addition and 1 deletion.
  1. +1 −1  reserve/request.pl
View
2  reserve/request.pl
@@ -602,7 +602,7 @@ =head1 request.pl
$reserve{'ccode'} = $res->{'ccode'};
$reserve{'barcode'} = $res->{'barcode'};
$reserve{'priority'} = $res->{'priority'};
- $reserve{'branchloop'} = GetBranchesLoop($res->{'branchcode'});
+ $reserve{'branchloop'} = GetBranchesLoop($res->{'branchcode'}, 0);
$reserve{'optionloop'} = \@optionloop;
$reserve{'reservenumber'} = $res->{'reservenumber'};
Please sign in to comment.
Something went wrong with that request. Please try again.