Attachment #112236 for bug #24083

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

Lines 133-138 Link Here

(-)a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt (+4 lines)
133	<input type="hidden" name="patronid" value="[% patronid \| html %]" />	133	<input type="hidden" name="patronid" value="[% patronid \| html %]" />
134	<input type="hidden" name="barcode" value="[% barcode \| html %]" />	134	<input type="hidden" name="barcode" value="[% barcode \| html %]" />
135	<input type="hidden" name="confirmed" value="1" />	135	<input type="hidden" name="confirmed" value="1" />
		136	<input type="hidden" name="seen" value="1" />
136	<input type="hidden" name="newissues" value="[% newissues \| html %]" />	137	<input type="hidden" name="newissues" value="[% newissues \| html %]" />
137	<button type="submit" name="confirm" class="btn btn-primary"><i class="fa fa-refresh" aria-hidden="true"></i> Renew item</button>	138	<button type="submit" name="confirm" class="btn btn-primary"><i class="fa fa-refresh" aria-hidden="true"></i> Renew item</button>
138	</form>	139	</form>
Lines 142-147 Link Here
142	<input type="hidden" name="patronid" value="[% patronid \| html %]" />	143	<input type="hidden" name="patronid" value="[% patronid \| html %]" />
143	<input type="hidden" name="barcode" value="[% barcode \| html %]" />	144	<input type="hidden" name="barcode" value="[% barcode \| html %]" />
144	<input type="hidden" name="confirmed" value="1" />	145	<input type="hidden" name="confirmed" value="1" />
		146	<input type="hidden" name="seen" value="1" />
145	<input type="hidden" name="newissues" value="[% newissues \| html %]" />	147	<input type="hidden" name="newissues" value="[% newissues \| html %]" />
146	<button type="submit" class="btn btn-primary"><i class="fa fa-refresh" aria-hidden="true"></i> Renew item</button>	148	<button type="submit" class="btn btn-primary"><i class="fa fa-refresh" aria-hidden="true"></i> Renew item</button>
147	</form>	149	</form>
Lines 341-346 Link Here
341	<span>This item has been scheduled for automatic renewal and cannot be renewed</span>	343	<span>This item has been scheduled for automatic renewal and cannot be renewed</span>
342	[% ELSIF ISSUE.renew_error == 'onsite_checkout' %]	344	[% ELSIF ISSUE.renew_error == 'onsite_checkout' %]
343	<span>This is a on-site checkout, it cannot be renewed.</span>	345	<span>This is a on-site checkout, it cannot be renewed.</span>
		346	[% ELSIF ISSUE.renew_error == 'too_unseen' %]
		347	<span>You have renewed this item the maximum number of consecutive times without it being seen by the library.</span>
344	[% ELSE %]	348	[% ELSE %]
345	<span>No renewals allowed</span>	349	<span>No renewals allowed</span>
346	[% END %]	350	[% END %]




}

my $issuerid = $loggedinuser;
my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $seen, $newissues) = (
    $query->param("op")         || '',
    $query->param("patronid")   || '',
    $query->param("patronlogin")|| '',
    $query->param("patronpw")   || '',
    $query->param("barcode")    || '',
    $query->param("confirmed")  || '',
    $query->param("seen")       || 0,
    $query->param("newissues")  || '',
);


    my ($status,$renewerror) = CanBookBeRenewed( $borrower->{borrowernumber}, $item->itemnumber );
    if ($status) {
        #warn "renewing";
        AddRenewal( $borrower->{borrowernumber}, $item->itemnumber, undef, undef, undef, undef, $seen );
        push @newissueslist, $barcode;
        $template->param( renewed => 1 );
    }
- 

Return to bug 24083

Lines 99-111 if (defined C4::Context->preference('SCOAllowCheckin')) { Link Here

(-)a/opac/sco/sco-main.pl (-3 / +3 lines)
99	}	99	}
100		100
101	my $issuerid = $loggedinuser;	101	my $issuerid = $loggedinuser;
102	my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $newissues) = (	102	my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $seen, $newissues) = (
103	$query->param("op") \|\| '',	103	$query->param("op") \|\| '',
104	$query->param("patronid") \|\| '',	104	$query->param("patronid") \|\| '',
105	$query->param("patronlogin")\|\| '',	105	$query->param("patronlogin")\|\| '',
106	$query->param("patronpw") \|\| '',	106	$query->param("patronpw") \|\| '',
107	$query->param("barcode") \|\| '',	107	$query->param("barcode") \|\| '',
108	$query->param("confirmed") \|\| '',	108	$query->param("confirmed") \|\| '',
		109	$query->param("seen") \|\| 0,
109	$query->param("newissues") \|\| '',	110	$query->param("newissues") \|\| '',
110	);	111	);
111		112
Lines 268-274 if ( $patron && ( $op eq 'renew' ) ) { Link Here
268	my ($status,$renewerror) = CanBookBeRenewed( $borrower->{borrowernumber}, $item->itemnumber );	269	my ($status,$renewerror) = CanBookBeRenewed( $borrower->{borrowernumber}, $item->itemnumber );
269	if ($status) {	270	if ($status) {
270	#warn "renewing";	271	#warn "renewing";
271	AddRenewal( $borrower->{borrowernumber}, $item->itemnumber );	272	AddRenewal( $borrower->{borrowernumber}, $item->itemnumber, undef, undef, undef, undef, $seen );
272	push @newissueslist, $barcode;	273	push @newissueslist, $barcode;
273	$template->param( renewed => 1 );	274	$template->param( renewed => 1 );
274	}	275	}
275	-