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

(-)a/C4/Circulation.pm (-2 / +3 lines)
Lines 2424-2430 already renewed the loan. $error will contain the reason the renewal can not pro Link Here
2424
sub CanBookBeRenewed {
2424
sub CanBookBeRenewed {
2425
2425
2426
    # check renewal status
2426
    # check renewal status
2427
    my ( $borrowernumber, $itemnumber, $override_limit ) = @_;
2427
    my ( $borrowernumber, $itemnumber, $override_limit, $overdue, $overduesblockrenew ) = @_;
2428
    my $dbh       = C4::Context->dbh;
2428
    my $dbh       = C4::Context->dbh;
2429
    my $renews    = 1;
2429
    my $renews    = 1;
2430
    my $renewokay = 0;
2430
    my $renewokay = 0;
Lines 2473-2479 sub CanBookBeRenewed { Link Here
2473
    if ( my $data1 = $sthcount->fetchrow_hashref ) {
2473
    if ( my $data1 = $sthcount->fetchrow_hashref ) {
2474
        
2474
        
2475
        if ( ( $data1->{renewalsallowed} && $data1->{renewalsallowed} > $data1->{renewals} ) || $override_limit ) {
2475
        if ( ( $data1->{renewalsallowed} && $data1->{renewalsallowed} > $data1->{renewals} ) || $override_limit ) {
2476
            $renewokay = 1;
2476
            $renewokay =  ($overduesblockrenew eq 'blockitem' and $overdue and !$override_limit)? 0 : 1;
2477
            $error = 'overdue' unless $renewokay;
2477
        }
2478
        }
2478
        else {
2479
        else {
2479
			$error="too_many";
2480
			$error="too_many";
(-)a/C4/ILSDI/Services.pm (-3 / +11 lines)
Lines 561-570 sub RenewLoan { Link Here
561
    return { code => 'RecordNotFound' } unless $$item{itemnumber};
561
    return { code => 'RecordNotFound' } unless $$item{itemnumber};
562
562
563
    # Add renewal if possible
563
    # Add renewal if possible
564
    my @renewal = CanBookBeRenewed( $borrowernumber, $itemnumber );
565
    if ( $renewal[0] ) { AddRenewal( $borrowernumber, $itemnumber ); }
566
567
    my $issue = GetItemIssue($itemnumber);
564
    my $issue = GetItemIssue($itemnumber);
565
    # Check for overdue on this document
566
    my $dt_due = $issue->{'date_due'}->truncate( to => 'day' );
567
    my $dt_today = DateTime->now( time_zone => C4::Context->tz() );
568
    my $overdue = 1 if DateTime->compare( $dt_today,$dt_due  ) == 1;
569
    # Check if any of patron's documents is late
570
    my ($memberblocked) = C4::Members::IsMemberBlocked($borrowernumber);
571
    # Is renewal allowed on overdue documents ?
572
    my $overduesblockrenew = C4::Context->preference('OverduesBlockRenew');
573
    my $norenewal = 1 if ( $overduesblockrenew eq 'blockall' and $memberblocked == -1 ) or ( $overduesblockrenew eq 'blockitem' and $overdue ) or $memberblocked == 1;# last to check if patron is restricted
574
    my @renewal = $norenewal ? (0,'overdue') : CanBookBeRenewed( $borrowernumber, $itemnumber );
575
    if ( $renewal[0] ) { AddRenewal( $borrowernumber, $itemnumber ); }
568
576
569
    # Hashref building
577
    # Hashref building
570
    my $out;
578
    my $out;
(-)a/circ/circulation.pl (-3 / +9 lines)
Lines 429-434 my $displayrelissues; Link Here
429
429
430
our $totalprice = 0;
430
our $totalprice = 0;
431
431
432
# Check if any of patron's documents is late
433
my ($memberblocked) = C4::Members::IsMemberBlocked($borrowernumber);
434
# and if renewal is allowed in that case
435
my $overduesblockrenew = C4::Context->preference('OverduesBlockRenew');
436
my $norenewal = 1 if $overduesblockrenew eq 'blockall' and $memberblocked == -1;
437
$template->param( norenewal => 1 ) if $overduesblockrenew ne 'allow';
438
432
sub build_issue_data {
439
sub build_issue_data {
433
    my $issueslist = shift;
440
    my $issueslist = shift;
434
    my $relatives = shift;
441
    my $relatives = shift;
Lines 444-452 sub build_issue_data { Link Here
444
            $it->{'itemnumber'}, $it->{'borrowernumber'}
451
            $it->{'itemnumber'}, $it->{'borrowernumber'}
445
        );
452
        );
446
        $it->{'charge'} = sprintf("%.2f", $it->{'charge'});
453
        $it->{'charge'} = sprintf("%.2f", $it->{'charge'});
447
        my ($can_renew, $can_renew_error) = CanBookBeRenewed( 
454
        my ($can_renew, $can_renew_error) = $norenewal ? (0, 'overdue') : CanBookBeRenewed(
448
            $it->{'borrowernumber'},$it->{'itemnumber'}
455
        $it->{'borrowernumber'},$it->{'itemnumber'}, undef, $it->{'overdue'}, $overduesblockrenew);
449
        );
450
        $it->{"renew_error_${can_renew_error}"} = 1 if defined $can_renew_error;
456
        $it->{"renew_error_${can_renew_error}"} = 1 if defined $can_renew_error;
451
        my ( $restype, $reserves, undef ) = CheckReserves( $it->{'itemnumber'} );
457
        my ( $restype, $reserves, undef ) = CheckReserves( $it->{'itemnumber'} );
452
        $it->{'can_renew'} = $can_renew;
458
        $it->{'can_renew'} = $can_renew;
(-)a/installer/data/mysql/sysprefs.sql (+1 lines)
Lines 406-408 INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES(' Link Here
406
INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('NotesBlacklist','','List of notes fields that should not appear in the title notes/description separator of details',NULL,'free');
406
INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('NotesBlacklist','','List of notes fields that should not appear in the title notes/description separator of details',NULL,'free');
407
INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('SCOUserCSS', '', NULL, 'Add CSS to be included in the SCO module in an embedded <style> tag.', 'free');
407
INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('SCOUserCSS', '', NULL, 'Add CSS to be included in the SCO module in an embedded <style> tag.', 'free');
408
INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('SCOUserJS', '', NULL, 'Define custom javascript for inclusion in the SCO module', 'free');
408
INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('SCOUserJS', '', NULL, 'Define custom javascript for inclusion in the SCO module', 'free');
409
INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES ('OverduesBlockRenew','allow','If any of patron\'s checked out documents is late, should renewal be allowed, blocked only on overdue items or blocked on whatever checked out document','allow|blockitem|blockall','Choice');
(-)a/installer/data/mysql/updatedatabase.pl (+7 lines)
Lines 6339-6344 if ( CheckVersion($DBversion) ) { Link Here
6339
   SetVersion ($DBversion);
6339
   SetVersion ($DBversion);
6340
}
6340
}
6341
6341
6342
$DBversion = "3.11.00.XXX";
6343
if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) {
6344
    $dbh->do("INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES ('OverduesBlockRenew','allow','If any of a patron\\'s checked out documents is late, should renewal be allowed, blocked only on overdue items or blocked on whatever checked out document','allow|blockitem|blockall','Choice')");
6345
    print "Upgrade to $DBversion done (Add system preference BlockRenewal ))\n";
6346
    SetVersion($DBversion);
6347
}
6348
6342
=head1 FUNCTIONS
6349
=head1 FUNCTIONS
6343
6350
6344
=head2 TableExists($table)
6351
=head2 TableExists($table)
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref (+8 lines)
Lines 286-291 Circulation: Link Here
286
                  confirmation: Ask for confirmation
286
                  confirmation: Ask for confirmation
287
            - when checking out to a borrower that has overdues outstanding	    
287
            - when checking out to a borrower that has overdues outstanding	    
288
        -
288
        -
289
            - If any of a patron's checked out items are late,
290
            - pref: OverduesBlockRenew
291
              type: choice
292
              choices:
293
                  allow: allow renewing of items.
294
                  blockall: block renewing for all items.
295
                  blockitem: block renewing only for overdue items.
296
        -
289
            - On checkin route the returned item to
297
            - On checkin route the returned item to
290
            - pref: HomeOrHoldingBranchReturn
298
            - pref: HomeOrHoldingBranchReturn
291
              type: choice
299
              type: choice
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt (-2 / +10 lines)
Lines 812-818 No patron matched <span class="ex">[% message %]</span> Link Here
812
        <td><span style="padding: 0 1em;">[% IF ( todayissue.renewals ) %][% todayissue.renewals %][% ELSE %]0[% END %]</span>
812
        <td><span style="padding: 0 1em;">[% IF ( todayissue.renewals ) %][% todayissue.renewals %][% ELSE %]0[% END %]</span>
813
        [% IF ( todayissue.can_renew ) %]
813
        [% IF ( todayissue.can_renew ) %]
814
        <input type="checkbox" name="all_items[]" value="[% todayissue.itemnumber %]" checked="checked" style="display: none;" />
814
        <input type="checkbox" name="all_items[]" value="[% todayissue.itemnumber %]" checked="checked" style="display: none;" />
815
        [% IF ( todayissue.od ) %]
815
        [% IF ( todayissue.od and !previssue.renew_error_overdue ) %]
816
            <input type="checkbox" class="radio" name="items[]" value="[% todayissue.itemnumber %]" checked="checked" />
816
            <input type="checkbox" class="radio" name="items[]" value="[% todayissue.itemnumber %]" checked="checked" />
817
        [% ELSE %]
817
        [% ELSE %]
818
            <input type="checkbox" class="radio" name="items[]" value="[% todayissue.itemnumber %]" />
818
            <input type="checkbox" class="radio" name="items[]" value="[% todayissue.itemnumber %]" />
Lines 834-839 No patron matched <span class="ex">[% message %]</span> Link Here
834
                [% IF ( todayissue.renew_error_too_many ) %]
834
                [% IF ( todayissue.renew_error_too_many ) %]
835
            Not renewable
835
            Not renewable
836
                [% END %]
836
                [% END %]
837
                [% IF ( todayissue.renew_error_overdue ) %]
838
                        Renewal not allowed (overdue)
839
                [% END %]
837
            [% IF ( todayissue.can_confirm ) %]
840
            [% IF ( todayissue.can_confirm ) %]
838
                </span>
841
                </span>
839
            [% END %]
842
            [% END %]
Lines 893-899 No patron matched <span class="ex">[% message %]</span> Link Here
893
        <td><span style="padding: 0 1em;">[% IF ( previssue.renewals ) %][% previssue.renewals %][% ELSE %]0[% END %]</span>
896
        <td><span style="padding: 0 1em;">[% IF ( previssue.renewals ) %][% previssue.renewals %][% ELSE %]0[% END %]</span>
894
        [% IF ( previssue.can_renew ) %]
897
        [% IF ( previssue.can_renew ) %]
895
        <input type="checkbox" name="all_items[]" value="[% previssue.itemnumber %]" checked="checked" style="display: none;" />
898
        <input type="checkbox" name="all_items[]" value="[% previssue.itemnumber %]" checked="checked" style="display: none;" />
896
        [% IF ( previssue.od ) %]
899
        [% IF ( previssue.od and !previssue.renew_error_overdue ) %]
897
            <input type="checkbox" class="radio" name="items[]" value="[% previssue.itemnumber %]" checked="checked" />
900
            <input type="checkbox" class="radio" name="items[]" value="[% previssue.itemnumber %]" checked="checked" />
898
        [% ELSE %]
901
        [% ELSE %]
899
            <input type="checkbox" class="radio" name="items[]" value="[% previssue.itemnumber %]" />
902
            <input type="checkbox" class="radio" name="items[]" value="[% previssue.itemnumber %]" />
Lines 915-920 No patron matched <span class="ex">[% message %]</span> Link Here
915
                [% IF ( previssue.renew_error_too_many ) %]
918
                [% IF ( previssue.renew_error_too_many ) %]
916
            Not renewable
919
            Not renewable
917
                [% END %]
920
                [% END %]
921
                [% IF ( previssue.renew_error_overdue ) %]
922
                        Renewal not allowed (overdue)
923
                [% END %]
918
            [% IF ( previssue.can_confirm ) %]
924
            [% IF ( previssue.can_confirm ) %]
919
                </span>
925
                </span>
920
            [% END %]
926
            [% END %]
Lines 954-960 No patron matched <span class="ex">[% message %]</span> Link Here
954
        [% END %]
960
        [% END %]
955
        [% END %]
961
        [% END %]
956
        <input type="submit" name="renew_checked" value="Renew or Return checked items" />
962
        <input type="submit" name="renew_checked" value="Renew or Return checked items" />
963
        [% UNLESS ( norenewal ) %]
957
        <input type="submit" id="renew_all" name="renew_all" value="Renew all" />
964
        <input type="submit" id="renew_all" name="renew_all" value="Renew all" />
965
        [% END %]
958
        [% IF export_remove_fields OR export_with_csv_profile %]
966
        [% IF export_remove_fields OR export_with_csv_profile %]
959
            <br/><br/>
967
            <br/><br/>
960
            Don't export fields : <input type="text" id="export_remove_fields" name="export_remove_fields" value="[% export_remove_fields %]" title="Use for iso2709 exports" />
968
            Don't export fields : <input type="text" id="export_remove_fields" name="export_remove_fields" value="[% export_remove_fields %]" title="Use for iso2709 exports" />
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt (-1 / +6 lines)
Lines 475-481 function validate1(date) { Link Here
475
            [% IF ( issueloo.norenew ) %]
475
            [% IF ( issueloo.norenew ) %]
476
                [% IF ( issueloo.can_confirm ) %]<span class="renewals-allowed" style="display: none">
476
                [% IF ( issueloo.can_confirm ) %]<span class="renewals-allowed" style="display: none">
477
                    <input type="checkbox" name="all_items[]" value="[% issueloo.itemnumber %]" checked="checked" style="display: none;" />
477
                    <input type="checkbox" name="all_items[]" value="[% issueloo.itemnumber %]" checked="checked" style="display: none;" />
478
                    [% IF ( issueloo.od ) %]
478
                    [% IF ( issueloo.od  and !issueloo.norenew_reason_overdue ) %]
479
                        <input type="checkbox" name="items[]" value="[% issueloo.itemnumber %]" checked="checked" />
479
                        <input type="checkbox" name="items[]" value="[% issueloo.itemnumber %]" checked="checked" />
480
                    [% ELSE %]
480
                    [% ELSE %]
481
                        <input type="checkbox" name="items[]" value="[% issueloo.itemnumber %]" />
481
                        <input type="checkbox" name="items[]" value="[% issueloo.itemnumber %]" />
Lines 483-488 function validate1(date) { Link Here
483
                    </span>
483
                    </span>
484
                    <span class="renewals-disabled">
484
                    <span class="renewals-disabled">
485
                [% END %]
485
                [% END %]
486
                [% IF ( issueloo.norenew_reason_overdue ) %]
487
                    Renewal not allowed (overdue)
488
                [% END %]
486
                [% IF ( issueloo.norenew_reason_on_reserve ) %]
489
                [% IF ( issueloo.norenew_reason_on_reserve ) %]
487
                <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% issueloo.biblionumber %]">On Hold</a>
490
                <a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% issueloo.biblionumber %]">On Hold</a>
488
                [% END %]
491
                [% END %]
Lines 525-531 function validate1(date) { Link Here
525
        [% END %]
528
        [% END %]
526
        [% END %]
529
        [% END %]
527
        <input type="submit" name="renew_checked" value="Renew or return checked items" />
530
        <input type="submit" name="renew_checked" value="Renew or return checked items" />
531
        [% UNLESS ( norenewal ) %]
528
        <input type="submit" id="renew_all" name="renew_all" value="Renew all" />
532
        <input type="submit" id="renew_all" name="renew_all" value="Renew all" />
533
        [% END %]
529
        </fieldset>
534
        </fieldset>
530
    [% ELSE %]<p>Patron has nothing checked out.</p>
535
    [% ELSE %]<p>Patron has nothing checked out.</p>
531
[% END %]
536
[% END %]
(-)a/koha-tmpl/opac-tmpl/prog/en/modules/opac-user.tt (-1 / +5 lines)
Lines 186-193 $.tablesorter.addParser({ Link Here
186
                [% IF ( OpacRenewalAllowed ) %]
186
                [% IF ( OpacRenewalAllowed ) %]
187
                    <td class="renew">[% IF ( ISSUE.status ) %][% IF ( canrenew ) %]<input type="checkbox" name="item" value="[% ISSUE.itemnumber %]"/> <a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&amp;item=[% ISSUE.itemnumber %]&amp;borrowernumber=[% ISSUE.borrowernumber %]">Renew</a>[% END %] <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span>
187
                    <td class="renew">[% IF ( ISSUE.status ) %][% IF ( canrenew ) %]<input type="checkbox" name="item" value="[% ISSUE.itemnumber %]"/> <a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&amp;item=[% ISSUE.itemnumber %]&amp;borrowernumber=[% ISSUE.borrowernumber %]">Renew</a>[% END %] <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span>
188
                        [% ELSE %]
188
                        [% ELSE %]
189
                        [% IF ( ISSUE.norenew_overdue ) %]
190
                           Renewal not allowed <span class="renewals">(overdue on a document)</span>
191
                        [% ELSE %]
189
                        Not renewable[% IF ( ISSUE.too_many ) %] <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span>[% ELSE %][% IF ( ISSUE.on_reserve ) %] <span class="renewals">(On hold)</span>[% END %][% END %] 
192
                        Not renewable[% IF ( ISSUE.too_many ) %] <span class="renewals">([% ISSUE.renewsleft %] of [% ISSUE.renewsallowed %] renewals remaining)</span>[% ELSE %][% IF ( ISSUE.on_reserve ) %] <span class="renewals">(On hold)</span>[% END %][% END %] 
190
                        [% END %]</td>
193
                        [% END %][% END %]</td>
191
                [% END %]
194
                [% END %]
192
[% IF ( OPACFinesTab ) %]
195
[% IF ( OPACFinesTab ) %]
193
                <td class="fines">[% IF ( ISSUE.charges ) %]Yes[% ELSE %]No[% END %]</td>
196
                <td class="fines">[% IF ( ISSUE.charges ) %]Yes[% ELSE %]No[% END %]</td>
Lines 330-335 $.tablesorter.addParser({ Link Here
330
[% ELSIF ( OVERDUE.status ) %]
333
[% ELSIF ( OVERDUE.status ) %]
331
[% IF ( canrenew ) %]<a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&amp;item=[% OVERDUE.itemnumber %]&amp;bornum=[% OVERDUE.borrowernumber %]">Renew</a>[% END %] <span class="renewals">([% OVERDUE.renewsleft %] of [% OVERDUE.renewsallowed %] renewals remaining)</span>
334
[% IF ( canrenew ) %]<a href="/cgi-bin/koha/opac-renew.pl?from=opac_user&amp;item=[% OVERDUE.itemnumber %]&amp;bornum=[% OVERDUE.borrowernumber %]">Renew</a>[% END %] <span class="renewals">([% OVERDUE.renewsleft %] of [% OVERDUE.renewsallowed %] renewals remaining)</span>
332
[% ELSIF ( OVERDUE.onreserve ) %]On hold
335
[% ELSIF ( OVERDUE.onreserve ) %]On hold
336
[% ELSIF ( OVERDUE.norenew_overdue ) %]Renewal not allowed<span class="renewals">(overdue on a document)</span>
333
[% ELSE %]No renewals left
337
[% ELSE %]No renewals left
334
[% END %]
338
[% END %]
335
</td>[% END %]
339
</td>[% END %]
(-)a/members/moremember.pl (-3 / +8 lines)
Lines 258-263 my @borrowers_with_issues; Link Here
258
my $overdues_exist = 0;
258
my $overdues_exist = 0;
259
my $totalprice = 0;
259
my $totalprice = 0;
260
260
261
# Check if any of patron's documents is late
262
my ($memberblocked)=C4::Members::IsMemberBlocked($borrowernumber);
263
# and if renewal is allowed in that case
264
my $overduesblockrenew = C4::Context->preference('OverduesBlockRenew');
265
my $norenewal = 1 if $overduesblockrenew eq 'blockall' and $memberblocked == -1;
266
$template->param( norenewal => 1 ) if $overduesblockrenew ne 'allow';
267
261
my @issuedata = build_issue_data($issue);
268
my @issuedata = build_issue_data($issue);
262
my @relissuedata = build_issue_data($relissue);
269
my @relissuedata = build_issue_data($relissue);
263
270
Lines 513-521 sub build_issue_data { Link Here
513
520
514
        $row{'charge'} = sprintf( "%.2f", $charge );
521
        $row{'charge'} = sprintf( "%.2f", $charge );
515
522
516
        my ( $renewokay, $renewerror ) =
523
        my ( $renewokay, $renewerror ) = $norenewal ? (0, 'overdue') : CanBookBeRenewed( $borrowernumber, $issue->{itemnumber}, $override_limit, $issue->{overdue}, $overduesblockrenew );
517
          CanBookBeRenewed( $borrowernumber, $issue->{itemnumber},
518
            $override_limit );
519
        $row{'norenew'} = !$renewokay;
524
        $row{'norenew'} = !$renewokay;
520
        $row{'can_confirm'} = ( !$renewokay && $renewerror ne 'on_reserve' );
525
        $row{'can_confirm'} = ( !$renewokay && $renewerror ne 'on_reserve' );
521
        $row{"norenew_reason_$renewerror"} = 1 if $renewerror;
526
        $row{"norenew_reason_$renewerror"} = 1 if $renewerror;
(-)a/opac/opac-user.pl (-2 / +9 lines)
Lines 153-158 my @overdues; Link Here
153
my @issuedat;
153
my @issuedat;
154
my $itemtypes = GetItemTypes();
154
my $itemtypes = GetItemTypes();
155
my $issues = GetPendingIssues($borrowernumber);
155
my $issues = GetPendingIssues($borrowernumber);
156
157
# Check if any of patron's documents is late
158
my ($memberblocked) = C4::Members::IsMemberBlocked($borrowernumber);
159
# and if renewal is allowed in that case
160
my $overduesblockrenew = C4::Context->preference('OverduesBlockRenew');
161
my $norenewal = 1 if $overduesblockrenew eq 'blockall' and $memberblocked == -1;
162
156
if ($issues){
163
if ($issues){
157
    foreach my $issue ( sort { $b->{date_due}->datetime() cmp $a->{date_due}->datetime() } @{$issues} ) {
164
    foreach my $issue ( sort { $b->{date_due}->datetime() cmp $a->{date_due}->datetime() } @{$issues} ) {
158
        # check for reserves
165
        # check for reserves
Lines 179-189 if ($issues){ Link Here
179
        $issue->{$publictype} = 1;
186
        $issue->{$publictype} = 1;
180
187
181
        # check if item is renewable
188
        # check if item is renewable
182
        my ($status,$renewerror) = CanBookBeRenewed( $borrowernumber, $issue->{'itemnumber'} );
189
        my ($status,$renewerror) = $norenewal ? (0, 'overdue') : CanBookBeRenewed( $borrowernumber, $issue->{'itemnumber'}, undef, $issue->{'overdue'}, $overduesblockrenew);
183
        ($issue->{'renewcount'},$issue->{'renewsallowed'},$issue->{'renewsleft'}) = GetRenewCount($borrowernumber, $issue->{'itemnumber'});
190
        ($issue->{'renewcount'},$issue->{'renewsallowed'},$issue->{'renewsleft'}) = GetRenewCount($borrowernumber, $issue->{'itemnumber'});
184
        if($status && C4::Context->preference("OpacRenewalAllowed")){
191
        if($status && C4::Context->preference("OpacRenewalAllowed")){
185
            $issue->{'status'} = $status;
192
            $issue->{'status'} = $status;
186
        }
193
        }
194
        $issue->{'norenew_overdue'} = 1 if $renewerror and $renewerror eq 'overdue';
187
        $issue->{'too_many'} = 1 if $renewerror and $renewerror eq 'too_many';
195
        $issue->{'too_many'} = 1 if $renewerror and $renewerror eq 'too_many';
188
        $issue->{'on_reserve'} = 1 if $renewerror and $renewerror eq 'on_reserve';
196
        $issue->{'on_reserve'} = 1 if $renewerror and $renewerror eq 'on_reserve';
189
197
190
- 

Return to bug 8236