Bug 33010: Do not filter checkins with logged in patrons branch in printslip.pl

If one tries to print checkin slip for checkins that
have been checked out from different branch, list
of checkins is empty. One has to change their branch
as checkout library to be able to print checkins.

This happens because we filter (or rather search) patrons
old checkouts with logged in patrons branch. This patch
removes this search so that checkout are filtered using
just filter_by_todays_checkins.

To test:
1. Checkout items for patron from branch A.
2. Switch to branch B and checkin items.
3. Print checkin slip.
=> Checkins list is empty.
4. Switch back to branch A.
5. Print checkin slip again.
=> Checkins are printed.
6. Apply this patch, restart services if needed.
7. Switch back to branch B and print checkin slip.
=> Checkins should now print.

Sponsored-by: Koha-Suomi Oy
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Emmi Takkinen 2023-02-20 10:26:29 +02:00 committed by Tomas Cohen Arazi
parent a40bd8d05d
commit b5f8c52657
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F

View file

@ -71,9 +71,7 @@ if ( $print eq 'checkinslip' ) {
my $checkinslip_branch = $session->param('branch') ? $session->param('branch') : $branch;
# get today's checkins
my @issue_ids = $patron->old_checkouts->search( { branchcode => $checkinslip_branch } )
->filter_by_todays_checkins->get_column('issue_id');
my @issue_ids = $patron->old_checkouts->filter_by_todays_checkins->get_column('issue_id');
my %loops = (
old_issues => \@issue_ids,
);