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

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt (-1 / +1 lines)
Lines 327-333 Link Here
327
                                                            <input type="hidden" name="barcode" value="[% ISSUE.barcode | html %]" />
327
                                                            <input type="hidden" name="barcode" value="[% ISSUE.barcode | html %]" />
328
                                                            <input type="hidden" name="newissues" value="[% newissues | html %]" />
328
                                                            <input type="hidden" name="newissues" value="[% newissues | html %]" />
329
                                                            [% IF ISSUE.can_be_renewed %]
329
                                                            [% IF ISSUE.can_be_renewed %]
330
                                                                <input type="hidden" name="op" value="checkout" />
330
                                                                <input type="hidden" name="op" value="renew" />
331
                                                                <input type="hidden" name="confirmed" value="1" />
331
                                                                <input type="hidden" name="confirmed" value="1" />
332
                                                                [% UNLESS ( ISSUE.renew ) %]
332
                                                                [% UNLESS ( ISSUE.renew ) %]
333
                                                                    <input type="submit" value="Renew item"  name="confirm  " class="btn renew" />
333
                                                                    <input type="submit" value="Renew item"  name="confirm  " class="btn renew" />
(-)a/opac/sco/sco-main.pl (-13 / +14 lines)
Lines 138-144 elsif ( $op eq "returnbook" && $allowselfcheckreturns ) { Link Here
138
    my ($doreturn) = AddReturn( $barcode, $branch );
138
    my ($doreturn) = AddReturn( $barcode, $branch );
139
    $template->param( returned => $doreturn );
139
    $template->param( returned => $doreturn );
140
}
140
}
141
elsif ( $patron && ( $op eq 'checkout' || $op eq 'renew' ) ) {
141
elsif ( $patron && ( $op eq 'checkout' ) ) {
142
    my $impossible  = {};
142
    my $impossible  = {};
143
    my $needconfirm = {};
143
    my $needconfirm = {};
144
    ( $impossible, $needconfirm ) = CanBookBeIssued(
144
    ( $impossible, $needconfirm ) = CanBookBeIssued(
Lines 182-203 elsif ( $patron && ( $op eq 'checkout' || $op eq 'renew' ) ) { Link Here
182
                barcode    => $barcode,
182
                barcode    => $barcode,
183
            );
183
            );
184
        }
184
        }
185
    } elsif ( $needconfirm->{RENEW_ISSUE} || $op eq 'renew' ) {
185
    } elsif ( $needconfirm->{RENEW_ISSUE} ){
186
        if ($confirmed) {
186
        $template->param(
187
            #warn "renewing";
188
            AddRenewal( $borrower->{borrowernumber}, $item->itemnumber );
189
            push @newissueslist, $barcode;
190
            $template->param( renewed => 1 );
191
        } else {
192
            #warn "renew confirmation";
193
            $template->param(
194
                renew               => 1,
187
                renew               => 1,
195
                barcode             => $barcode,
188
                barcode             => $barcode,
196
                confirm             => 1,
189
                confirm             => 1,
197
                confirm_renew_issue => 1,
190
                confirm_renew_issue => 1,
198
                hide_main           => 1,
191
                hide_main           => 1,
199
            );
192
        );
200
        }
201
    } elsif ( $confirm_required && !$confirmed ) {
193
    } elsif ( $confirm_required && !$confirmed ) {
202
        #warn "failed confirmation";
194
        #warn "failed confirmation";
203
        $template->param(
195
        $template->param(
Lines 260-265 elsif ( $patron && ( $op eq 'checkout' || $op eq 'renew' ) ) { Link Here
260
    }
252
    }
261
} # $op
253
} # $op
262
254
255
if ( $patron && ( $op eq 'renew' ) ) {
256
    my ($status,$renewerror) = CanBookBeRenewed( $borrower->{borrowernumber}, $item->itemnumber );
257
    if ($status) {
258
        #warn "renewing";
259
        AddRenewal( $borrower->{borrowernumber}, $item->itemnumber );
260
        push @newissueslist, $barcode;
261
        $template->param( renewed => 1 );
262
    }
263
}
264
263
if ($borrower) {
265
if ($borrower) {
264
#   warn "issuer's  branchcode: " .   $issuer->{branchcode};
266
#   warn "issuer's  branchcode: " .   $issuer->{branchcode};
265
#   warn   "user's  branchcode: " . $borrower->{branchcode};
267
#   warn   "user's  branchcode: " . $borrower->{branchcode};
266
- 

Return to bug 26301