Bugzilla – Attachment 102818 Details for
Bug 24612
expirationdate blank if patron has more than one item from bib on hold
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24612: Make hold-transfer-slip take reserve_id
Bug-24612-Make-hold-transfer-slip-take-reserveid.patch (text/plain), 10.05 KB, created by
Jonathan Druart
on 2020-04-13 11:51:37 UTC
(
hide
)
Description:
Bug 24612: Make hold-transfer-slip take reserve_id
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2020-04-13 11:51:37 UTC
Size:
10.05 KB
patch
obsolete
>From 260198161032f816a0db326f2ab2d1c9988a1090 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Wed, 12 Feb 2020 10:02:56 +0100 >Subject: [PATCH] Bug 24612: Make hold-transfer-slip take reserve_id > >To make sure we are going to display the correct hold's info we need to >pass the reserve_id. > >See comment 7 and 8 for test plans >--- > C4/Circulation.pm | 2 ++ > C4/Reserves.pm | 36 ++++--------------- > circ/circulation.pl | 3 +- > circ/hold-transfer-slip.pl | 10 ++---- > circ/returns.pl | 11 ++---- > .../prog/en/modules/circ/circulation.tt | 4 +-- > .../prog/en/modules/circ/returns.tt | 4 +-- > 7 files changed, 18 insertions(+), 52 deletions(-) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index 747d660ea6..11de1d39b6 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -1038,6 +1038,7 @@ sub CanBookBeIssued { > $needsconfirmation{'resborrowernumber'} = $patron->borrowernumber; > $needsconfirmation{'resbranchcode'} = $res->{branchcode}; > $needsconfirmation{'reswaitingdate'} = $res->{'waitingdate'}; >+ $needsconfirmation{'reserve_id'} = $res->{reserve_id}; > } > elsif ( $restype eq "Reserved" ) { > # The item is on reserve for someone else. >@@ -1048,6 +1049,7 @@ sub CanBookBeIssued { > $needsconfirmation{'resborrowernumber'} = $patron->borrowernumber; > $needsconfirmation{'resbranchcode'} = $patron->branchcode; > $needsconfirmation{'resreservedate'} = $res->{reservedate}; >+ $needsconfirmation{'reserve_id'} = $res->{reserve_id}; > } > } > } >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index 08b16893da..c0a6665dcd 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -604,7 +604,7 @@ sub GetOtherReserves { > ModReserveStatus($itemnumber,'W'); > } > >- $nextreservinfo = $checkreserves->{'borrowernumber'}; >+ $nextreservinfo = $checkreserves; > } > > return ( $messages, $nextreservinfo ); >@@ -1173,7 +1173,7 @@ sub ModReserveCancelAll { > #step 2 launch the subroutine of the others reserves > ( $messages, $nextreservinfo ) = GetOtherReserves($itemnumber); > >- return ( $messages, $nextreservinfo ); >+ return ( $messages, $nextreservinfo->{borrowernumber} ); > } > > =head2 ModReserveMinusPriority >@@ -2037,36 +2037,12 @@ available within the slip: > sub ReserveSlip { > my ($args) = @_; > my $branchcode = $args->{branchcode}; >- my $borrowernumber = $args->{borrowernumber}; >- my $biblionumber = $args->{biblionumber}; >- my $itemnumber = $args->{itemnumber}; >- my $barcode = $args->{barcode}; >- >- >- my $patron = Koha::Patrons->find($borrowernumber); >- >- my $hold; >- if ($itemnumber || $barcode ) { >- $itemnumber ||= Koha::Items->find( { barcode => $barcode } )->itemnumber; >- >- $hold = Koha::Holds->search( >- { >- biblionumber => $biblionumber, >- borrowernumber => $borrowernumber, >- itemnumber => $itemnumber >- } >- )->next; >- } >- else { >- $hold = Koha::Holds->search( >- { >- biblionumber => $biblionumber, >- borrowernumber => $borrowernumber >- } >- )->next; >- } >+ my $reserve_id = $args->{reserve_id}; > >+ my $hold = Koha::Holds->find($reserve_id); > return unless $hold; >+ >+ my $patron = $hold->borrower; > my $reserve = $hold->unblessed; > > return C4::Letters::GetPreparedLetter ( >diff --git a/circ/circulation.pl b/circ/circulation.pl >index 19feb305f5..b2e0d08ac7 100755 >--- a/circ/circulation.pl >+++ b/circ/circulation.pl >@@ -419,7 +419,8 @@ if (@$barcodes) { > > if ($question->{RESERVE_WAITING} or $question->{RESERVED}){ > $template->param( >- reserveborrowernumber => $question->{'resborrowernumber'} >+ reserveborrowernumber => $question->{'resborrowernumber'}, >+ reserve_id => $question->{reserve_id}, > ); > } > >diff --git a/circ/hold-transfer-slip.pl b/circ/hold-transfer-slip.pl >index 328f4811f3..a0cb0ef1ea 100755 >--- a/circ/hold-transfer-slip.pl >+++ b/circ/hold-transfer-slip.pl >@@ -35,10 +35,7 @@ my $input = new CGI; > my $sessionID = $input->cookie("CGISESSID"); > my $session = get_session($sessionID); > >-my $biblionumber = $input->param('biblionumber'); >-my $borrowernumber = $input->param('borrowernumber'); >-my $itemnumber = $input->param('itemnumber'); >-my $barcode = $input->param('barcode'); >+my $reserve_id = $input->param('reserve_id'); > > my ( $template, $loggedinuser, $cookie ) = get_template_and_user( > { >@@ -55,10 +52,7 @@ my $userenv = C4::Context->userenv; > my ($slip, $is_html); > if ( my $letter = ReserveSlip ({ > branchcode => $session->param('branch') || $userenv->{branch}, >- borrowernumber => $borrowernumber, >- biblionumber => $biblionumber, >- itemnumber => $itemnumber, >- barcode => $barcode, >+ reserve_id => $reserve_id, > }) ) { > $slip = $letter->{content}; > $is_html = $letter->{is_html}; >diff --git a/circ/returns.pl b/circ/returns.pl >index f0e4abe9fc..affbda5877 100755 >--- a/circ/returns.pl >+++ b/circ/returns.pl >@@ -77,9 +77,7 @@ my $session = get_session($sessionID); > if ( $query->param('print_slip') ) { > $template->param( > print_slip => 1, >- borrowernumber => scalar $query->param('borrowernumber'), # FIXME We should send a Koha::Patron and raise an error if not exist. >- biblionumber => scalar $query->param('biblionumber'), >- itemnumber => scalar $query->param('itemnumber'), >+ reserve_id => scalar $query->param('reserve_id'), > ); > } > >@@ -419,15 +417,10 @@ if ( $messages->{'ResFound'}) { > my $diffBranchSend = !$branchCheck ? $reserve->{branchcode} : undef; > ModReserveAffect( $reserve->{itemnumber}, $reserve->{borrowernumber}, $diffBranchSend, $reserve->{reserve_id} ); > my ( $messages, $nextreservinfo ) = GetOtherReserves($reserve->{itemnumber}); >- >- my $patron = Koha::Patrons->find( $nextreservinfo ); >- > $template->param( > hold_auto_filled => 1, > print_slip => C4::Context->preference('HoldsAutoFillPrintSlip'), >- patron => $patron, >- borrowernumber => $patron->id, >- biblionumber => $biblio->id, >+ reserve_id => $nextreservinfo->{reserve_id}, > ); > > if ( $messages->{'transfert'} ) { >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >index 9087ccb03d..f7189eb687 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -263,7 +263,7 @@ > <input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" /> > <input type="hidden" name="duedatespec" value="[% duedatespec | html %]" /> > <input type="hidden" name="stickyduedate" value="[% stickyduedate | html %]" /> >- <button class="print" type="submit" onclick="Dopop('hold-transfer-slip.pl?borrowernumber=[% reserveborrowernumber | uri %]&biblionumber=[% itembiblionumber | uri %]&itemnumber=[% item.itemnumber | uri %]&op=slip');this.form.submit();"><i class="fa fa-print"></i> Don't check out and print slip (P)</button> >+ <button class="print" type="submit" onclick="Dopop('hold-transfer-slip.pl?reserve_id=[% reserve_id | uri %]');this.form.submit();"><i class="fa fa-print"></i> Don't check out and print slip (P)</button> > </form> > [% END %] > >@@ -273,7 +273,7 @@ > <input type="hidden" name="borrowernumber" value="[% patron.borrowernumber | html %]" /> > <input type="hidden" name="duedatespec" value="[% duedatespec | html %]" /> > <input type="hidden" name="stickyduedate" value="[% stickyduedate | html %]" /> >- <button class="print" type="submit" onclick="Dopop('hold-transfer-slip.pl?borrowernumber=[% reserveborrowernumber | uri %]&biblionumber=[% itembiblionumber | uri %]&itemnumber=[% item.itemnumber | uri %]&op=slip');this.form.submit();"><i class="fa fa-print"></i> Don't check out and print slip (P)</button> >+ <button class="print" type="submit" onclick="Dopop('hold-transfer-slip.pl?reserve_id=[% reserve_id | uri %]');this.form.submit();"><i class="fa fa-print"></i> Don't check out and print slip (P)</button> > </form> > [% END %] > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >index 50580d54e8..734aab6234 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >@@ -936,11 +936,11 @@ > > $(".print-slip").on('click', function(e) { > e.preventDefault(); >- Dopop('hold-transfer-slip.pl?borrowernumber=[% patron.borrowernumber | uri %]&biblionumber=[% biblionumber | uri %]'); >+ Dopop('hold-transfer-slip.pl?reserve_id=[% reserve_id | uri %]'); > }); > > [% IF print_slip %] >- Dopop('hold-transfer-slip.pl?borrowernumber=[% borrowernumber | uri %]&biblionumber=[% biblionumber | uri %]&itemnumber=[% itemnumber | uri %]'); >+ Dopop('hold-transfer-slip.pl?reserve_id=[% reserve_id | uri %]'); > [% END %] > > var columns_settings = [% ColumnsSettings.GetColumns( 'circ', 'returns', 'checkedintable', 'json' ) | $raw %] >-- >2.20.1
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 24612
:
98738
|
98739
|
102818
|
102819
|
102820
|
104803
|
104804
|
104805
|
105202
|
105203
|
105204
|
105253
|
105254
|
105775