Attachment 70115 Details for Bug 19812 – Bug 19812 - Holds count in "Already received" table has confusing and unexpected values

[patch] Bug 19812 - Holds count in "Already received" table has confusing and unexpected values

Bug-19812---Holds-count-in-Already-received-table-.patch (text/plain), 3.45 KB, created by Marcel de Rooy on 2017-12-22 11:03:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2017-12-22 11:03:53 UTC

Size: 3.45 KB

patch

obsolete

>From 9a549e42953d97357f10c97abb322883fe30cce4 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Thu, 14 Dec 2017 07:03:39 -0500
>Subject: [PATCH] Bug 19812 - Holds count in "Already received" table has
> confusing and unexpected values
>Content-Type: text/plain; charset=utf-8
>
>In acqui/parcel.pl both the "Pending orders" and "Already received" tables show how many holds there are for the given record. However, the count of holds in the "Pending orders" table confuses librarians because it only lists holds for the particular items in the orderline. Due to that, the holds column may show 0 holds even if there are a dozen record level holds for that bib! This is not what librarians seem to expect, instead it seems that the same total holds in the "Pending orders" table would be preferred.
>
>Test Plan:
>1) Find an invoice with an item in the "Already received" table
>2) Add one or more record level holds to the record
>3) Note the holds column does not count those holds
>4) Apply this patch
>5) Note the holds column now shows total holds and holds for just those ordered items
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>
>Signed-off-by: Nancy Keener <nkeener@washoecounty.us>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> acqui/parcel.pl | 3 ++-
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt | 12 +++++++++---
> 2 files changed, 11 insertions(+), 4 deletions(-)
>
>diff --git a/acqui/parcel.pl b/acqui/parcel.pl
>index b01e591..068f187 100755
>--- a/acqui/parcel.pl
>+++ b/acqui/parcel.pl
>@@ -142,7 +142,8 @@ for my $order ( @orders ) {
> $line{holds} = 0;
> my @itemnumbers = GetItemnumbersFromOrder( $order->{ordernumber} );
> my $biblio = Koha::Biblios->find( $line{biblionumber} );
>- $line{holds} = $biblio ? $biblio->current_holds->search(
>+ $line{total_holds} = $biblio ? $biblio->holds->count : 0;
>+ $line{item_holds} = $biblio ? $biblio->current_holds->search(
> {
> itemnumber => { -in => \@itemnumbers },
> }
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
>index ee4d8d6..6dc352f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/parcel.tt
>@@ -351,7 +351,7 @@
> <th>Basket</th>
> <th>Basket group</th>
> <th>Order line</th>
>- <th>Holds</th>
>+ <th title="Total holds / Item holds">Holds</th>
> <th>Summary</th>
> <th>View record</th>
> <th>Quantity</th>
>@@ -413,8 +413,14 @@
> [% END %]
> </td>
> <td>
>- [% IF order.holds > 0 %]
>- <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% order.biblionumber %]">[% order.holds %]</a>
>+ [% IF order.total_holds > 0 %]
>+ <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% order.biblionumber %]">[% order.total_holds %]</a>
>+ [% ELSE %]
>+ 0
>+ [% END %]
>+ /
>+ [% IF order.item_holds > 0 %]
>+ <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% order.biblionumber %]">[% order.item_holds %]</a>
> [% ELSE %]
> 0
> [% END %]
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 19812: 69799 | 69829 | 70115 | 73606 | 73607 | 73608