Attachment 115291 Details for Bug 27011 – Bug 27011 removed $name variable

[patch] Bug 27011 removed $name variable

Bug-27011-removed-name-variable.patch (text/plain), 1.55 KB, created by Amy King on 2021-01-19 03:30:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Amy King

Created: 2021-01-19 03:30:45 UTC

Size: 1.55 KB

patch

obsolete

>From d1520f1690ecdf5c8af7c2d8b09b37831b97ca31 Mon Sep 17 00:00:00 2001
>From: Amy King <amyking386@gmail.com>
>Date: Tue, 19 Jan 2021 03:27:27 +0000
>Subject: [PATCH] Bug 27011 removed $name variable
>
>Test Plan
>1. Check out a book
>2. Check in a book
>3. Note what happens
>4. Apply patch
>5. Repeat step 1-2
>6. Note that it doesn't break and everything still works
>---
> circ/returns.pl | 3 ---
> 1 file changed, 3 deletions(-)
>
>diff --git a/circ/returns.pl b/circ/returns.pl
>index 116ac3c15e..4e9662024c 100755
>--- a/circ/returns.pl
>+++ b/circ/returns.pl
>@@ -162,13 +162,11 @@ if ( $query->param('reserve_id') ) {
>     my ( $messages, $nextreservinfo ) = GetOtherReserves($itemnumber);
> 
>     my $patron = Koha::Patrons->find( $nextreservinfo );
>-    my $name   = $patron ? $patron->surname . ", " . $patron->title . " " . $patron->firstname : '';
>     if ( $messages->{'transfert'} ) {
>         $template->param(
>             itemtitle      => $biblio->title,
>             itembiblionumber => $biblio->biblionumber,
>             iteminfo       => $biblio->author,
>-            name           => $name,
>             patron         => $patron,
>             diffbranch     => 1,
>         );
>@@ -400,7 +398,6 @@ if ( $messages->{'WrongTransfer'} and not $messages->{'WasTransfered'}) {
>     my $reserve    = $messages->{'ResFound'};
>     if ( $reserve ) {
>         my $patron = Koha::Patrons->find( $reserve->{'borrowernumber'} );
>-        my $name = $patron->surname . ", " . $patron->title . " " . $patron->firstname;
>         $template->param(
>             patron => $patron,
>         );
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 27011: 115291 | 115301 | 115319 | 115523