Bugzilla – Attachment 130435 Details for
Bug 24295
C4::Circulation::GetTransfers should be removed, use Koha::Item->get_transfer instead
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24295: Remove GetTransfers from detail.pl
Bug-24295-Remove-GetTransfers-from-detailpl.patch (text/plain), 2.36 KB, created by
Martin Renvoize (ashimema)
on 2022-02-10 12:01:15 UTC
(
hide
)
Description:
Bug 24295: Remove GetTransfers from detail.pl
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2022-02-10 12:01:15 UTC
Size:
2.36 KB
patch
obsolete
>From 60676a8fc395cb072d03a14ded4db59375e75b78 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Mon, 15 Mar 2021 16:18:36 +0000 >Subject: [PATCH] Bug 24295: Remove GetTransfers from detail.pl > >This patch replaces the call to GetTransfers in detail.pl with the >get_transfer method available from the Koha::Item object > >Test plan >1/ Trigger the transfer of the item >2/ Check the detail page for the display of the current transfer > status >3/ Apply patch >4/ Repeat step 2 >--- > catalogue/detail.pl | 13 ++++++------- > 1 file changed, 6 insertions(+), 7 deletions(-) > >diff --git a/catalogue/detail.pl b/catalogue/detail.pl >index 75aeed0e96..0d5a91e140 100755 >--- a/catalogue/detail.pl >+++ b/catalogue/detail.pl >@@ -34,7 +34,6 @@ use C4::Serials qw( CountSubscriptionFromBiblionumber SearchSubscriptions GetLat > use C4::Output qw( output_html_with_http_headers ); > use C4::Biblio qw( GetBiblioData GetFrameworkCode GetMarcBiblio ); > use C4::Items qw( GetAnalyticsCount GetHostItemsInfo GetItemsInfo ); >-use C4::Circulation qw( GetTransfers ); > use C4::Reserves; > use C4::Serials qw( CountSubscriptionFromBiblionumber SearchSubscriptions GetLatestSerials ); > use C4::XISBN qw( get_xisbns ); >@@ -42,7 +41,7 @@ use C4::External::Amazon qw( get_amazon_tld ); > use C4::Search qw( z3950_search_args enabled_staff_search_views new_record_from_zebra ); > use C4::Tags qw( get_tags ); > use C4::XSLT qw( XSLTParse4Display ); >-use Koha::DateUtils qw( format_sqldatetime ); >+use Koha::DateUtils qw( format_sqldatetime output_pref ); > use C4::HTML5Media; > use C4::CourseReserves qw( GetItemCourseReservesInfo ); > use Koha::AuthorisedValues; >@@ -345,11 +344,11 @@ foreach my $item (@items) { > } > > # Check the transit status >- my ( $transfertwhen, $transfertfrom, $transfertto ) = GetTransfers($item->{itemnumber}); >- if ( defined( $transfertwhen ) && ( $transfertwhen ne '' ) ) { >- $item->{transfertwhen} = $transfertwhen; >- $item->{transfertfrom} = $transfertfrom; >- $item->{transfertto} = $transfertto; >+ my $transfer = $item_object->get_transfer; >+ if ( $transfer && $transfer->in_transit ) { >+ $item->{transfertwhen} = output_pref({ dt => dt_from_string($transfer->datesent) }); >+ $item->{transfertfrom} = $transfer->frombranch; >+ $item->{transfertto} = $transfer->tobranch; > $item->{nocancel} = 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 24295
:
118242
|
118243
|
118244
|
118245
|
118246
|
118247
|
118248
|
118249
|
118250
|
118287
|
118295
|
118296
|
118297
|
118298
|
118299
|
118300
|
118301
|
118302
|
118303
|
118304
|
118305
|
118306
|
118307
|
118308
|
118309
|
118310
|
119854
|
119855
|
119856
|
119857
|
119858
|
119859
|
119860
|
119861
|
119862
|
119863
|
119864
|
119866
|
119867
|
127465
|
127466
|
127467
|
127468
|
127469
|
127470
|
127471
|
127472
|
127473
|
127474
|
127475
|
127476
|
127477
|
130430
|
130431
|
130432
|
130433
|
130434
|
130435
|
130436
|
130437
|
130438
|
130439
|
130440
|
130441
|
130442
|
139546
|
139547
|
139548
|
139549
|
139550
|
139551
|
139552
|
139553
|
139554
|
139555
|
139556
|
139557
|
139558
|
139655
|
139656
|
139657
|
139658
|
139659
|
139660
|
139661
|
139662
|
139663
|
139664
|
139665
|
139666
|
139667