Bugzilla – Attachment 121064 Details for
Bug 28294
C4::Circulation::updateWrongTransfer should be moved into Koha::
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 28294: Convert updateWrongTransfer to use request_transfer
Bug-28294-Convert-updateWrongTransfer-to-use-reque.patch (text/plain), 1.78 KB, created by
Martin Renvoize (ashimema)
on 2021-05-17 16:22:32 UTC
(
hide
)
Description:
Bug 28294: Convert updateWrongTransfer to use request_transfer
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2021-05-17 16:22:32 UTC
Size:
1.78 KB
patch
obsolete
>From 950114e24c2a327cbf3297ca448a8454f869beff Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Mon, 17 May 2021 17:14:16 +0100 >Subject: [PATCH] Bug 28294: Convert updateWrongTransfer to use > request_transfer > >This patch is an interim patch to prove that the updated >request_transfer method achieves the same as updateWrongTransfer. > >Test plan >1/ Run t/db_dependant/Circulation.t before and after applying this >patch. >--- > C4/Circulation.pm | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index c42277d3b5..c56934c800 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -3591,19 +3591,17 @@ This function validate the line of brachtransfer but with the wrong destination > sub updateWrongTransfer { > my ( $itemNumber,$waitingAtLibrary,$FromLibrary ) = @_; > >- # first step: cancel the original transfer >- my $item = Koha::Items->find($itemNumber); >- my $transfer = $item->get_transfer; >- $transfer->set({ datecancelled => dt_from_string, cancellation_reason => 'WrongTransfer' })->store(); >+ my $to_library = Koha::Libraries->find($waitingAtLibrary); > >- # second step: create a new transfer to the right location >+ # Request a new transfer, replacing the existing one. >+ my $item = Koha::Items->find($itemNumber); > my $new_transfer = $item->request_transfer( > { >- to => $transfer->to_library, >- reason => $transfer->reason, >- comment => $transfer->comments, >+ to => $to_library, >+ reason => 'WrongTransfer', > ignore_limits => 1, >- enqueue => 1 >+ replace => 1, >+ force => 1 > } > ); > >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 28294
:
120637
|
121062
|
121063
|
121064
|
121077
|
121078
|
121079
|
121095
|
121096
|
121097
|
121714
|
121715
|
121716
|
130082
|
130083
|
130084
|
130085
|
130086
|
130087
|
132212
|
132213
|
132214
|
132215
|
132216
|
132217
|
160516
|
160517
|
160518
|
160519
|
160520
|
160521
|
160536
|
160692
|
168281
|
172778
|
172781
|
173856
|
174805