Bugzilla – Attachment 57850 Details for
Bug 17671
Remove unused variables in Reserves.pm
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
remove unused variables
remove-unused-variables.patch (text/plain), 2.27 KB, created by
Alex Buckley
on 2016-11-30 23:47:52 UTC
(
hide
)
Description:
remove unused variables
Filename:
MIME Type:
Creator:
Alex Buckley
Created:
2016-11-30 23:47:52 UTC
Size:
2.27 KB
patch
obsolete
>From e9e00f35b33711cabd50e6f1990537bb4129fa0c Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Petter=20Goks=C3=B8yr=20=C3=85sen?= > <boutrosboutrosboutros@gmail.com> >Date: Wed, 23 Nov 2016 11:12:24 +0100 >Subject: [PATCH] remove unused variables > >https://bugs.koha-community.org/show_bug.cgi?id=17671 >Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz> >--- > C4/Reserves.pm | 10 +--------- > 1 file changed, 1 insertion(+), 9 deletions(-) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index 83903e8..9ef33ad 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -175,8 +175,6 @@ sub AddReserve { > $title, $checkitem, $found, $itemtype > ) = @_; > >- my $dbh = C4::Context->dbh; >- > $resdate = output_pref( { str => dt_from_string( $resdate ), dateonly => 1, dateformat => 'iso' }) > or output_pref({ dt => dt_from_string, dateonly => 1, dateformat => 'iso' }); > >@@ -1182,7 +1180,6 @@ sub ModReserve { > return unless ( $reserve_id || ( $borrowernumber && ( $biblionumber || $itemnumber ) ) ); > $reserve_id = GetReserveId({ biblionumber => $biblionumber, borrowernumber => $borrowernumber, itemnumber => $itemnumber }) unless ( $reserve_id ); > >- my $dbh = C4::Context->dbh; > if ( $rank eq "del" ) { > CancelReserve({ reserve_id => $reserve_id }); > } >@@ -1232,8 +1229,6 @@ sub ModReserveFill { > my ($res) = @_; > my $reserve_id = $res->{'reserve_id'}; > >- my $dbh = C4::Context->dbh; >- > my $hold = Koha::Holds->find($reserve_id); > > # get the priority on this record.... >@@ -1247,7 +1242,7 @@ sub ModReserveFill { > } > ); > >- my $old_hold = Koha::Old::Hold->new( $hold->unblessed() )->store(); >+ Koha::Old::Hold->new( $hold->unblessed() )->store(); > > $hold->delete(); > >@@ -1596,8 +1591,6 @@ Input: $where is 'up', 'down', 'top' or 'bottom'. Biblionumber, Date reserve was > sub AlterPriority { > my ( $where, $reserve_id ) = @_; > >- my $dbh = C4::Context->dbh; >- > my $reserve = GetReserve( $reserve_id ); > > if ( $reserve->{cancellationdate} ) { >@@ -2176,7 +2169,6 @@ sub MoveReserve { > return unless $res; > > my $biblionumber = $res->{biblionumber}; >- my $biblioitemnumber = $res->{biblioitemnumber}; > > if ($res->{borrowernumber} == $borrowernumber) { > ModReserveFill($res); >-- >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 17671
:
57726
|
57850
|
57883