Attachment 106132 Details for Bug 25699 – Bug 25699: Add edition information to "Holds to pull" report

[patch] Bug 25699: Add edition information to "Holds to pull" report

Bug-25699-Add-edition-information-to-Holds-to-pull.patch (text/plain), 5.19 KB, created by Katrin Fischer on 2020-06-21 16:11:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2020-06-21 16:11:12 UTC

Size: 5.19 KB

patch

obsolete

>From 69d1ce9975f4ad539b7a90a998724089559d0c1f Mon Sep 17 00:00:00 2001
>From: Petro Vashchuk <stalkernoid@gmail.com>
>Date: Wed, 10 Jun 2020 15:36:01 +0300
>Subject: [PATCH] Bug 25699: Add edition information to "Holds to pull" report
>
>Added a feature that displays edition information of the book
>together with title in "Holds to pull" report.
>
>Edition information is fetched from "biblioitem" table
>as "editionstatement" and transferred to template.
>
>Those changes don't have performance impact as all tables
>including "biblioitem" were already used and joined in this request.
>
>Mentored-by: Andrew Nugged <nugged@gmail.com>
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> circ/pendingreserves.pl | 52 +++++++++++-----------
> .../prog/en/modules/circ/pendingreserves.tt | 1 +
> 2 files changed, 28 insertions(+), 25 deletions(-)
>
>diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl
>index 7f2062942a..22c93f7130 100755
>--- a/circ/pendingreserves.pl
>+++ b/circ/pendingreserves.pl
>@@ -206,6 +206,7 @@ my $strsth =
> biblio.part_number,
> biblio.part_name,
> biblio.author,
>+ biblioitems.editionstatement,
> count(DISTINCT items.itemnumber) as icount,
> count(DISTINCT reserves.borrowernumber) as rcount,
> borrowers.firstname,
>@@ -246,31 +247,32 @@ $sth->execute(@query_params);
> while ( my $data = $sth->fetchrow_hashref ) {
> push(
> @reservedata, {
>- reservedate => $data->{l_reservedate},
>- firstname => $data->{firstname} || '',
>- surname => $data->{surname},
>- title => $data->{title},
>- subtitle => $data->{subtitle},
>- medium => $data->{medium},
>- part_number => $data->{part_number},
>- part_name => $data->{part_name},
>- author => $data->{author},
>- borrowernumber => $data->{borrowernumber},
>- biblionumber => $data->{biblionumber},
>- holdingbranches => [split('\|', $data->{l_holdingbranch})],
>- branch => $data->{l_branch},
>- itemcallnumber => [split('\|', $data->{l_itemcallnumber})],
>- enumchron => [split('\|', $data->{l_enumchron})],
>- copyno => [split('\|', $data->{l_copynumber})],
>- count => $data->{icount},
>- rcount => $data->{rcount},
>- pullcount => $data->{icount} <= $data->{rcount} ? $data->{icount} : $data->{rcount},
>- itemTypes => [split('\|', $data->{l_item_type})],
>- locations => [split('\|', $data->{l_location})],
>- reserve_id => $data->{reserve_id},
>- holdingbranch => $data->{holdingbranch},
>- homebranch => $data->{homebranch},
>- itemnumber => $data->{itemnumber},
>+ reservedate => $data->{l_reservedate},
>+ firstname => $data->{firstname} || '',
>+ surname => $data->{surname},
>+ title => $data->{title},
>+ editionstatement => $data->{editionstatement},
>+ subtitle => $data->{subtitle},
>+ medium => $data->{medium},
>+ part_number => $data->{part_number},
>+ part_name => $data->{part_name},
>+ author => $data->{author},
>+ borrowernumber => $data->{borrowernumber},
>+ biblionumber => $data->{biblionumber},
>+ holdingbranches => [split('\|', $data->{l_holdingbranch})],
>+ branch => $data->{l_branch},
>+ itemcallnumber => [split('\|', $data->{l_itemcallnumber})],
>+ enumchron => [split('\|', $data->{l_enumchron})],
>+ copyno => [split('\|', $data->{l_copynumber})],
>+ count => $data->{icount},
>+ rcount => $data->{rcount},
>+ pullcount => $data->{icount} <= $data->{rcount} ? $data->{icount} : $data->{rcount},
>+ itemTypes => [split('\|', $data->{l_item_type})],
>+ locations => [split('\|', $data->{l_location})],
>+ reserve_id => $data->{reserve_id},
>+ holdingbranch => $data->{holdingbranch},
>+ homebranch => $data->{homebranch},
>+ itemnumber => $data->{itemnumber},
> }
> );
> }
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>index 1510f81ed9..72d4c875a4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>@@ -80,6 +80,7 @@
> [% INCLUDE 'biblio-title.inc' biblio=reserveloo %]
> </a>
> [% IF ( reserveloo.author ) %] by [% reserveloo.author | html %][% END %]
>+ [% IF ( reserveloo.editionstatement ) %][% reserveloo.editionstatement | html %][% END %]
> </td>
> [% ELSE %]
> <td>"</td>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 25699: 105691 | 105786 | 106006 | 106132