From d327f26f3f22daa4ca9a424c08726aedfc01fe01 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Thu, 14 Mar 2019 13:15:01 +0000 Subject: [PATCH] Bug 21013: Missing itemtype for checkut makes patron summary print explode This patch moves retrieval of the item type description from the script to the template using the ItemTypes template plugin. To test, apply the patch and locate an item which is checked out to someone. Modify the database record for that item to remove the item type (items.itype). View the print summary for the patron who has that item checked out. The page should display correctly. Checked-out items which have an item type should show that item type description correctly. --- koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt | 3 ++- members/summary-print.pl | 3 --- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt index 10ee557..e5c6b8e 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt @@ -2,6 +2,7 @@ [% USE Branches %] [% USE KohaDates %] [% USE Price %] +[% USE ItemTypes %] [% SET footerjs = 1 %] [% PROCESS 'accounts.inc' %] @@ -55,7 +56,7 @@ [% issue.author | html %] [% issue.itemcallnumber | html %] - [% issue.itemtype_description | html %] + [% ItemTypes.GetDescription( issue.itype ) | html %] [% issue.date_due | $KohaDates as_due_date => 1 %] [% issue.barcode | html %] [% issue.charge | $Price %] diff --git a/members/summary-print.pl b/members/summary-print.pl index 76bb1d4..38127c9 100755 --- a/members/summary-print.pl +++ b/members/summary-print.pl @@ -93,9 +93,6 @@ sub build_issue_data { my ( $charge, $itemtype ) = GetIssuingCharges( $checkout->{itemnumber}, $borrowernumber ); - $itemtype = Koha::ItemTypes->find( $itemtype ); - $checkout->{itemtype_description} = $itemtype->description; #FIXME Should not it be translated_description - $checkout->{charge} = $charge; $checkout->{overdue} = $c->is_overdue; -- 2.1.4