Bug 40771 - Wrong transfer breaking check in for holds when using library transfer limits
Summary: Wrong transfer breaking check in for holds when using library transfer limits
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on: 39750
Blocks:
  Show dependency treegraph
 
Reported: 2025-09-08 21:33 UTC by Nick Clemens (kidclamp)
Modified: 2025-09-08 21:33 UTC (History)
2 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2025-09-08 21:33:17 UTC
Bug 39750 fixed the case of a transfer with no hold - it seems the same problem can occur when the transfer is for a hold:


Line 178 of returns.pl
        my $hold = Koha::Holds->find($reserve_id);
        if ($diffBranchSend) {
            my $tobranch = $hold->pickup_library();

            # Add transfer, enqueue if one is already in the queue, and immediately set to in transit
            my $transfer = $item->request_transfer( { to => $tobranch, reason => 'Reserve', enqueue => 1 } );
            $transfer->transit;
        }