Bugzilla – Attachment 62535 Details for
Bug 13796
Alert in OPAC when renewing an item with a rental charge
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 13796: OPAC: Display rental charge for renewing
Bug-13796-OPAC-Display-rental-charge-for-renewing.patch (text/plain), 3.42 KB, created by
Marc Véron
on 2017-04-21 15:55:51 UTC
(
hide
)
Description:
Bug 13796: OPAC: Display rental charge for renewing
Filename:
MIME Type:
Creator:
Marc Véron
Created:
2017-04-21 15:55:51 UTC
Size:
3.42 KB
patch
obsolete
>From 68cc2392a52e94c9d1334225e2bb3132b8002556 Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch> >Date: Fri, 21 Apr 2017 17:50:45 +0200 >Subject: [PATCH] Bug 13796: OPAC: Display rental charge for renewing > >This patch displays the rental fee that will be applied for >renewing (near "Renew" link on patron's summary page) > >To test: >- Apply patch >- Enable system preference 'OpacRenewalAllowed' >- Verify that you have item types with and without rental fee > for testing >- Issue items with an without rental fee to a patron >- In OPAC, display patron's summary page >- Verify in table 'Checked out', items with rental charge display an information as > appropriate (near the link 'Renew') > >Patch rewritten because of merge issues with previous patches. 2017-04-21 mv >--- > koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt | 5 ++++- > opac/opac-user.pl | 1 + > 2 files changed, 5 insertions(+), 1 deletion(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >index a715178..9df6072 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >@@ -258,8 +258,11 @@ Using this account is not recommended because some parts of Koha will not functi > [% IF ( ISSUE.status ) %] > [% IF ( canrenew ) %] > <input type="checkbox" name="item" value="[% ISSUE.itemnumber %]"/> <a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&item=[% ISSUE.itemnumber %]&borrowernumber=[% ISSUE.borrowernumber %]">Renew</a> >+ <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span> >+ [% END %] >+ [% IF ISSUE.renewalfee > 0 %] >+ <span class="renewalfee">Fee for item type '[% ItemTypes.GetDescription( ISSUE.renewalitemtype) %]': [% ISSUE.renewalfee | $Price %]</span> > [% END %] >- <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span> > [% ELSIF ( ISSUE.on_reserve ) %] > Not renewable <span class="renewals">(on hold)</span> > [% ELSIF ( ISSUE.too_many ) %] >diff --git a/opac/opac-user.pl b/opac/opac-user.pl >index 3dc39bf..4f6784f 100755 >--- a/opac/opac-user.pl >+++ b/opac/opac-user.pl >@@ -204,6 +204,7 @@ if ($issues){ > # check if item is renewable > my ($status,$renewerror) = CanBookBeRenewed( $borrowernumber, $issue->{'itemnumber'} ); > ($issue->{'renewcount'},$issue->{'renewsallowed'},$issue->{'renewsleft'}) = GetRenewCount($borrowernumber, $issue->{'itemnumber'}); >+ ( $issue->{'renewalfee'}, $issue->{'renewalitemtype'} ) = GetIssuingCharges( $issue->{'itemnumber'}, $borrowernumber ); > if($status && C4::Context->preference("OpacRenewalAllowed")){ > $issue->{'status'} = $status; > } >-- >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 13796
:
52657
|
53051
|
53052
|
53476
|
54126
|
54188
|
54189
|
62535
|
65397
|
67272
|
67273
|
68742
|
68743