Created attachment 39061 [details] [review] Bug 14188 - branchtransfers GetAllTransfers and GetAllPendingTransfers functions to ease testing. Because there is no nice way of getting all branchtransfers and putting SQL into tests is VERY VERY BAD, introducing these branchtransfer API calls to conveniently get all transfers data nicely. Unit tests included. TEST PLAN: Run t/db_dependent/Circulation_transfers.t
Created attachment 40076 [details] [review] Bug 14188 - branchtransfers GetAllTransfers and GetAllPendingTransfers functions to ease testing. Because there is no nice way of getting all branchtransfers and putting SQL into tests is VERY VERY BAD, introducing these branchtransfer API calls to conveniently get all transfers data nicely. Unit tests included. TEST PLAN: Run t/db_dependent/Circulation_transfers.t
Wanted to test, but does not apply: Applying: Bug 14188 - branchtransfers GetAllTransfers and GetAllPendingTransfers functions to ease testing. fatal: sha1 information is lacking or useless (C4/Circulation.pm). Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 Bug 14188 - branchtransfers GetAllTransfers and GetAllPendingTransfers functions to ease testing.
Koha::Item::Transfers->search should be enough I think...
Agreeing with Joseph - just changing status as BLOCKED usually indicates we are waiting for something higher up in the dependency tree.