View | Details | Raw Unified | Return to bug 15494
Collapse All | Expand All

(-)a/C4/SIP/ILS/Transaction/Renew.pm (+1 lines)
Lines 51-56 sub do_renew_for { Link Here
51
    } else {
51
    } else {
52
        $renewerror=~s/on_reserve/Item unavailable due to outstanding holds/;
52
        $renewerror=~s/on_reserve/Item unavailable due to outstanding holds/;
53
        $renewerror=~s/too_many/Item has reached maximum renewals/;
53
        $renewerror=~s/too_many/Item has reached maximum renewals/;
54
        $renewerror=~s/item_denied_renewal/Item renewal is not allowed/;
54
        $self->screen_msg($renewerror);
55
        $self->screen_msg($renewerror);
55
        $self->renewal_ok(0);
56
        $self->renewal_ok(0);
56
    }
57
    }
(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/strings.inc (+1 lines)
Lines 21-26 Link Here
21
    var NOT_RENEWABLE_AUTO_TOO_MUCH_OWEING = _("Automatic renewal failed, patron has unpaid fines");
21
    var NOT_RENEWABLE_AUTO_TOO_MUCH_OWEING = _("Automatic renewal failed, patron has unpaid fines");
22
    var NOT_RENEWABLE_AUTO_ACCOUNT_EXPIRED = _("Automatic renewal failed, account expired");
22
    var NOT_RENEWABLE_AUTO_ACCOUNT_EXPIRED = _("Automatic renewal failed, account expired");
23
    var NOT_RENEWABLE_AUTO_RENEW = _("Scheduled for automatic renewal");
23
    var NOT_RENEWABLE_AUTO_RENEW = _("Scheduled for automatic renewal");
24
    var NOT_RENEWABLE_DENIED = _("Renewal denied by syspref");
24
    var RENEWALS_REMAINING = _("%s of %s renewals remaining");
25
    var RENEWALS_REMAINING = _("%s of %s renewals remaining");
25
    var HOLD_IS_SUSPENDED = _("Hold is <strong>suspended</strong>");
26
    var HOLD_IS_SUSPENDED = _("Hold is <strong>suspended</strong>");
26
    var UNTIL = _("until %s");
27
    var UNTIL = _("until %s");
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt (+4 lines)
Lines 117-122 Link Here
117
117
118
                                <p>[% borrower.firstname | html %] [% borrower.surname | html %] ( <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrower.borrowernumber | html %]"> [% borrower.cardnumber | html %] </a> ) is currently restricted.</p>
118
                                <p>[% borrower.firstname | html %] [% borrower.surname | html %] ( <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrower.borrowernumber | html %]"> [% borrower.cardnumber | html %] </a> ) is currently restricted.</p>
119
119
120
                            [% ELSIF error == "item_denied_renewal" %]
121
122
                                <p>Item is not allowed renewal.</p>
123
120
                            [% ELSE %]
124
                            [% ELSE %]
121
125
122
                                [% error | html %]
126
                                [% error | html %]
(-)a/koha-tmpl/intranet-tmpl/prog/js/checkouts.js (+7 lines)
Lines 424-429 $(document).ready(function() { Link Here
424
                            span_class = "renewals-allowed";
424
                            span_class = "renewals-allowed";
425
                        } else if ( oObj.can_renew_error == "onsite_checkout" ) {
425
                        } else if ( oObj.can_renew_error == "onsite_checkout" ) {
426
                            // Don't display something if it's an onsite checkout
426
                            // Don't display something if it's an onsite checkout
427
                        } else if ( oObj.can_renew_error == "item_denied_renewal" ) {
428
                            content += "<span class='renewals-disabled'>"
429
                                    + NOT_RENEWABLE_DENIED
430
                                    + "</span>";
431
432
                            span_style = "display: none";
433
                            span_class = "renewals-allowed";
427
                        } else {
434
                        } else {
428
                            content += "<span class='renewals-disabled'>"
435
                            content += "<span class='renewals-disabled'>"
429
                                    + oObj.can_renew_error
436
                                    + oObj.can_renew_error
(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt (+4 lines)
Lines 83-88 Link Here
83
                                            <li>It is too soon after the checkout date for this item to be renewed.</li>
83
                                            <li>It is too soon after the checkout date for this item to be renewed.</li>
84
                                        [% ELSIF error == 'on_reserve' %]
84
                                        [% ELSIF error == 'on_reserve' %]
85
                                            <li>This item is on hold for another borrower.</li>
85
                                            <li>This item is on hold for another borrower.</li>
86
                                        [% ELSIF error == 'item_denied_renewal' %]
87
                                            <li>Item renewal is not allowed.</li>
86
                                        [% END %]
88
                                        [% END %]
87
                                    [% END %]
89
                                    [% END %]
88
                                </ul>
90
                                </ul>
Lines 307-312 Link Here
307
                                                    [% ELSIF ( ISSUE.too_soon ) %]
309
                                                    [% ELSIF ( ISSUE.too_soon ) %]
308
                                                        No renewal before [% ISSUE.soonestrenewdate | html %]
310
                                                        No renewal before [% ISSUE.soonestrenewdate | html %]
309
                                                        <span class="renewals">([% ISSUE.renewsleft | html %] of [% ISSUE.renewsallowed | html %] renewals remaining)</span>
311
                                                        <span class="renewals">([% ISSUE.renewsleft | html %] of [% ISSUE.renewsallowed | html %] renewals remaining)</span>
312
                                                    [% ELSIF ( ISSUE.item_denied_renewal ) %]
313
                                                        Renewal not allowed
310
                                                    [% END %]
314
                                                    [% END %]
311
                                                    </td>
315
                                                    </td>
312
                                                [% END %]
316
                                                [% END %]
(-)a/misc/cronjobs/automatic_renewals.pl (-1 / +2 lines)
Lines 88-94 while ( my $auto_renew = $auto_renews->next ) { Link Here
88
        or $error eq 'auto_account_expired'
88
        or $error eq 'auto_account_expired'
89
        or $error eq 'auto_too_late'
89
        or $error eq 'auto_too_late'
90
        or $error eq 'auto_too_much_oweing'
90
        or $error eq 'auto_too_much_oweing'
91
        or $error eq 'auto_too_soon' ) {
91
        or $error eq 'auto_too_soon'
92
        or $error eq 'item_denied_renewal' ) {
92
        if ( not $auto_renew->auto_renew_error or $error ne $auto_renew->auto_renew_error ) {
93
        if ( not $auto_renew->auto_renew_error or $error ne $auto_renew->auto_renew_error ) {
93
            $auto_renew->auto_renew_error($error)->store;
94
            $auto_renew->auto_renew_error($error)->store;
94
            push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew
95
            push @{ $report{ $auto_renew->borrowernumber } }, $auto_renew
(-)a/opac/opac-user.pl (-1 / +1 lines)
Lines 235-240 if ( $pending_checkouts->count ) { # Useless test Link Here
235
            $issue->{'auto_too_soon'}  = 1 if $renewerror eq 'auto_too_soon';
235
            $issue->{'auto_too_soon'}  = 1 if $renewerror eq 'auto_too_soon';
236
            $issue->{'auto_too_late'}  = 1 if $renewerror eq 'auto_too_late';
236
            $issue->{'auto_too_late'}  = 1 if $renewerror eq 'auto_too_late';
237
            $issue->{'auto_too_much_oweing'}  = 1 if $renewerror eq 'auto_too_much_oweing';
237
            $issue->{'auto_too_much_oweing'}  = 1 if $renewerror eq 'auto_too_much_oweing';
238
            $issue->{'item_denied_renewal'}  = 1 if $renewerror eq 'item_denied_renewal';
238
239
239
            if ( $renewerror eq 'too_soon' ) {
240
            if ( $renewerror eq 'too_soon' ) {
240
                $issue->{'too_soon'}         = 1;
241
                $issue->{'too_soon'}         = 1;
241
- 

Return to bug 15494