Bug 9591 : Set Library changes after viewing Holds Queue >> reserves item

To test
1. Log in to the staff client. Note your logged in library.
2. Navigate to Circulation -> Holds queue.
3. Select a holds queue report for a library other than your own.
4. From the holds queue report page, click the "Check out" tab and
   enter the card number of a patron who has holds.
5. On the circulation page for that patron, click the "Holds" tab.
6. Click one of the titles in the list of holds.

Your logged in branch will now match the library you chose for the
holds queue report.

Alternate sequence:

4. From the holds queue report page, click the "Check out" tab and
   do a non-exact search (search which will return more than one result)
   for a patron who has holds.
5. Choose the correct patron from the list of results.

Your logged in branch will now match the library you chose for
the holds queue report.

Apply the patch and repeat, this time it should not change your logged
in branch

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Weird bug, the patch works for me.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Good catch. All tests and QA script pass.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This commit is contained in:
Chris Cormack 2013-04-08 21:20:51 +12:00
parent f5593842dc
commit 7925b98ee8
2 changed files with 2 additions and 2 deletions

View File

@ -53,7 +53,7 @@ my $itemtypeslimit = $params->{'itemtypeslimit'};
if ( $run_report ) {
my $items = GetHoldsQueueItems($branchlimit, $itemtypeslimit);
$template->param(
branch => $branchlimit,
branchlimit => $branchlimit,
total => scalar @$items,
itemsloop => $items,
run_report => $run_report,

View File

@ -34,7 +34,7 @@ $(document).ready(function() {
[% IF ( run_report ) %]
[% IF ( total ) %]
<div class="results">[% total %] items found for
[% IF ( branch ) %][% branch %][% ELSE %]ALL libraries[% END %]
[% IF ( branchlimit ) %][% branchlimit %][% ELSE %]ALL libraries[% END %]
</div>
[% ELSE %]
<div class="dialog message">No items found.</div>