Bugzilla – Attachment 53052 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.11 KB, created by
Marc Véron
on 2016-07-03 13:10:31 UTC
(
hide
)
Description:
Bug 13796: OPAC: Display rental charge for renewing
Filename:
MIME Type:
Creator:
Marc Véron
Created:
2016-07-03 13:10:31 UTC
Size:
3.11 KB
patch
obsolete
>From 72e9458d0f829aa7f43a5e87672f48cffbadd162 Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch> >Date: Tue, 21 Jun 2016 18:28:47 +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 >- 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') >--- > koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt | 4 ++++ > opac/opac-user.pl | 1 + > 2 files changed, 5 insertions(+) > >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 264641f..96f83e9 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt >@@ -2,6 +2,7 @@ > [% USE KohaDates %] > [% USE Branches %] > [% USE ItemTypes %] >+[% USE Price %] > > [% INCLUDE 'doc-head-open.inc' %] > <title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle %][% ELSE %]Koha online[% END %] catalog › Your library home</title> >@@ -235,6 +236,9 @@ > [% 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> >+ [% IF ISSUE.renewalfee > 0 %] >+ <span class="renewalfee">Fee for [% ItemTypes.GetDescription( ISSUE.renewalitemtype) %] [% ISSUE.renewalfee | $Price %]</span> >+ [% END %] > [% END %] > <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span> > [% ELSIF ( ISSUE.too_many ) %] >diff --git a/opac/opac-user.pl b/opac/opac-user.pl >index 413c26d..64afe08 100755 >--- a/opac/opac-user.pl >+++ b/opac/opac-user.pl >@@ -192,6 +192,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; > } >-- >1.7.10.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