Bugzilla – Attachment 59484 Details for
Bug 17968
Remove useless variable in C4::Overdues::parse_overdues_letter
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17968: Remove useless variable $item_format in C4::Overdues::parse_overdues_letter
Bug-17968-Remove-useless-variable-itemformat-in-C4.patch (text/plain), 1.46 KB, created by
Mark Tompsett
on 2017-01-24 04:04:24 UTC
(
hide
)
Description:
Bug 17968: Remove useless variable $item_format in C4::Overdues::parse_overdues_letter
Filename:
MIME Type:
Creator:
Mark Tompsett
Created:
2017-01-24 04:04:24 UTC
Size:
1.46 KB
patch
obsolete
>From d56dd679f0d1800c10fa0c76cb982d0600c1884c Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Thu, 19 Jan 2017 18:03:55 +0100 >Subject: [PATCH] Bug 17968: Remove useless variable $item_format in > C4::Overdues::parse_overdues_letter > >The variable $item_format is not used and should be removed from this >subroutine. > >Moreover it the letter parameter, but it is never sent to this >subroutine. letter_code is expected instead. > >Test plan: >No test plan, just read the code and `git grep ` > >Signed-off-by: Mark Tompsett <mtompset@hotmail.com> >--- > C4/Overdues.pm | 6 ------ > 1 file changed, 6 deletions(-) > >diff --git a/C4/Overdues.pm b/C4/Overdues.pm >index 8300a1e..c26b5f0 100644 >--- a/C4/Overdues.pm >+++ b/C4/Overdues.pm >@@ -1009,14 +1009,8 @@ sub parse_overdues_letter { > > my @item_tables; > if ( my $i = $params->{'items'} ) { >- my $item_format = ''; > foreach my $item (@$i) { > my $fine = GetFine($item->{'itemnumber'}, $params->{'borrowernumber'}); >- if ( !$item_format and defined $params->{'letter'}->{'content'} ) { >- $params->{'letter'}->{'content'} =~ m/(<item>.*<\/item>)/; >- $item_format = $1; >- } >- > $item->{'fine'} = currency_format($currency_format, "$fine", FMT_SYMBOL); > # if active currency isn't correct ISO code fallback to sprintf > $item->{'fine'} = sprintf('%.2f', $fine) unless $item->{'fine'}; >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 17968
:
59410
|
59484
|
59602