Attachment 95476 Details for Bug 20948 – Bug 20948: Simplify existing code

[patch] Bug 20948: Simplify existing code

Bug-20948-Simplify-existing-code.patch (text/plain), 3.47 KB, created by Jonathan Druart on 2019-11-18 09:38:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2019-11-18 09:38:53 UTC

Size: 3.47 KB

patch

obsolete

>From 7492a7deb7950dba549e2438ec3ca8c5d4b07a95 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 18 Nov 2019 10:10:56 +0100
>Subject: [PATCH] Bug 20948: Simplify existing code
>
>No changes expected here.
>---
> catalogue/detail.pl                                         |  8 +-------
> koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 13 +++++++------
> 2 files changed, 8 insertions(+), 13 deletions(-)
>
>diff --git a/catalogue/detail.pl b/catalogue/detail.pl
>index 511855e9fc..249a7417cb 100755
>--- a/catalogue/detail.pl
>+++ b/catalogue/detail.pl
>@@ -314,13 +314,7 @@ foreach my $item (@items) {
>     my $item_object = Koha::Items->find( $item->{itemnumber} );
>     my $holds = $item_object->current_holds;
>     if ( my $first_hold = $holds->next ) {
>-        my $patron = Koha::Patrons->find( $first_hold->borrowernumber );
>-        $item->{backgroundcolor} = 'reserved';
>-        $item->{reservedate}     = $first_hold->reservedate;
>-        $item->{ReservedFor}     = $patron,
>-        $item->{ExpectedAtLibrary}      = $first_hold->branchcode;
>-        # Check waiting status
>-        $item->{waitingdate} = $first_hold->waitingdate;
>+        $item->{first_hold} = $first_hold;
>     }
> 
>     if ( my $checkout = $item_object->checkout ) {
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>index 571f3b2395..615d09eaef 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>@@ -355,18 +355,19 @@
>                             [% END %]
>                         [% END %]
> 
>-                        [% IF ( item.reservedate ) %]
>-                            [% IF ( item.waitingdate ) %]
>-                                Waiting at [% Branches.GetName( item.ExpectedAtLibrary ) | html %] since [% item.waitingdate | $KohaDates %].
>+                        [% SET hold = item.first_hold %]
>+                        [% IF hold %]
>+                            [% IF hold.waitingdate %]
>+                                Waiting at [% Branches.GetName( hold.branchcode ) | html %] since [% hold.waitingdate | $KohaDates %].
>                             [% ELSE %]
>-                                Item-level hold (placed [% item.reservedate | $KohaDates %]) for delivery at [% Branches.GetName( item.ExpectedAtLibrary ) | html %].
>+                                Item-level hold (placed [% hold.reservedate | $KohaDates %]) for delivery at [% Branches.GetName( hold.branchcode ) | html %].
>                             [% END %]
>                             [% IF ( canreservefromotherbranches ) %]
>                                 Hold for:
>-                                [% INCLUDE 'patron-title.inc' patron=item.ReservedFor hide_patron_infos_if_needed=1 %]
>+                                [% INCLUDE 'patron-title.inc' patron=hold.borrower hide_patron_infos_if_needed=1 %]
>                             [% END %]
>                         [% END %]
>-                        [% UNLESS ( item.itemnotforloan || item.notforloan_per_itemtype || item.onloan || item.itemlost || item.withdrawn || item.damaged || item.transfertwhen || item.reservedate ) %]
>+                        [% UNLESS ( item.itemnotforloan || item.notforloan_per_itemtype || item.onloan || item.itemlost || item.withdrawn || item.damaged || item.transfertwhen || hold ) %]
>                             Available
>                         [% END %]
> 
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 20948: 76145 | 76216 | 95476 | 95477 | 95636 | 95637 | 95870 | 95871 | 96137