[% IF ( userdebarred ) %]
@@ -196,7 +195,7 @@ $.tablesorter.addParser({
[% IF ( show_barcode ) %]Barcode [% END %]
Call No.
[% IF ( OpacRenewalAllowed ) %]
- [% UNLESS patron_flagged %]Renew [% END %]
+ Renew
[% END %]
[% IF ( OPACFinesTab ) %]
Fines
@@ -244,12 +243,10 @@ $.tablesorter.addParser({
[% IF ( show_barcode ) %][% ISSUE.barcode %] [% END %]
[% ISSUE.itemcallnumber %]
[% IF ( OpacRenewalAllowed ) %]
- [% UNLESS patron_flagged %]
- [% IF ( ISSUE.status ) %] Renew ([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)
+ [% IF ( ISSUE.status ) %][% IF ( canrenew ) %] Renew [% END %] ([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)
[% ELSE %]
Not renewable[% IF ( ISSUE.too_many ) %] ([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining) [% ELSE %][% IF ( ISSUE.on_reserve ) %] (On hold) [% END %][% END %]
[% END %]
- [% END %]
[% END %]
[% IF ( OPACFinesTab ) %]
[% IF ( ISSUE.charges ) %]Yes[% ELSE %]No[% END %]
@@ -271,7 +268,7 @@ $.tablesorter.addParser({
[% FOREACH ISSUE IN ISSUES %]
[% END %]
- [% UNLESS patron_flagged %] [% END %]
+
[% END %]
[% ELSE %]
@@ -293,7 +290,8 @@ $.tablesorter.addParser({
Call No.
Due
[% IF ( OpacRenewalAllowed ) %]
- [% UNLESS patron_flagged %]Renew [% END %][% END %]
+ Renew
+[% END %]
[% IF ( OPACFinesTab ) %]
Fines
[% END %]
@@ -325,15 +323,14 @@ $.tablesorter.addParser({
[% OVERDUE.itemcallnumber %]
[% OVERDUE.date_due %]
[% IF ( OpacRenewalAllowed ) %]
- [% UNLESS patron_flagged %]
[% IF ( OVERDUE.debarred ) %]Account Frozen
[% ELSIF ( OVERDUE.status ) %]
-Renew ([% OVERDUE.renewsleft %] of [% OVERDUE.renewsallowed %] renewals remaining)
-[% ELSIF ( OVERDUE.onreserve ) %]On Reserve
+[% IF ( canrenew ) %]Renew [% END %] ([% OVERDUE.renewsleft %] of [% OVERDUE.renewsallowed %] renewals remaining)
+[% ELSIF ( OVERDUE.onreserve ) %]On hold
[% ELSE %]No renewals left
[% END %]
- [% END %][% END %]
+[% END %]
[% IF ( OPACFinesTab ) %]
[% IF ( OVERDUE.charges ) %]Yes[% ELSE %]No[% END %]
[% END %]
--- a/opac/opac-user.pl
+++ a/opac/opac-user.pl
@@ -67,6 +67,7 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
my $OPACDisplayRequestPriority = (C4::Context->preference("OPACDisplayRequestPriority")) ? 1 : 0;
my $patronupdate = $query->param('patronupdate');
+my $canrenew = 1;
# get borrower information ....
my ( $borr ) = GetMemberDetails( $borrowernumber );
@@ -92,6 +93,7 @@ if ($debar) {
if ( $userdebarred || $borr->{'gonenoaddress'} || $borr->{'lost'} ) {
$borr->{'flagged'} = 1;
+ $canrenew = 0;
}
if ( $borr->{'amountoutstanding'} > 5 ) {
@@ -105,6 +107,7 @@ $no_renewal_amt ||= 0;
if ( $borr->{amountoutstanding} > $no_renewal_amt ) {
$borr->{'flagged'} = 1;
+ $canrenew = 0;
$template->param(
renewal_blocked_fines => sprintf( '%.02f', $no_renewal_amt ),
);
@@ -144,13 +147,11 @@ $template->param( BORROWER_INFO => \@bordat,
#get issued items ....
my $count = 0;
-my $toggle = 0;
my $overdues_count = 0;
my @overdues;
my @issuedat;
my $itemtypes = GetItemTypes();
my ($issues) = GetPendingIssues($borrowernumber);
-my $canrenew = 0;
if ($issues){
foreach my $issue ( sort { $b->{'date_due'} cmp $a->{'date_due'} } @$issues ) {
# check for reserves
@@ -181,7 +182,6 @@ if ($issues){
($issue->{'renewcount'},$issue->{'renewsallowed'},$issue->{'renewsleft'}) = GetRenewCount($borrowernumber, $issue->{'itemnumber'});
if($status && C4::Context->preference("OpacRenewalAllowed")){
$issue->{'status'} = $status;
- $canrenew = 1;
}
$issue->{'too_many'} = 1 if $renewerror and $renewerror eq 'too_many';
$issue->{'on_reserve'} = 1 if $renewerror and $renewerror eq 'on_reserve';
--