use Modern::Perl;
use CGI qw ( -utf8 );
-use Digest::MD5 qw(md5_base64);
use C4::Auth qw(get_template_and_user checkpw);
use C4::Koha;
use C4::Items;
use Koha::DateUtils qw( dt_from_string );
use Koha::Acquisition::Currencies;
+use Koha::Patrons;
use Koha::Patron::Images;
use Koha::Patron::Messages;
+use Koha::Token;
my $query = new CGI;
$query->param(-name=>'koha_login_context',-values=>['sco']);
}
$query->param(-name=>'sco_user_login',-values=>[1]);
-my ($template, $loggedinuser, $cookie) = get_template_and_user({
- template_name => "sco/sco-main.tt",
- authnotrequired => 0,
- flagsrequired => { circulate => "self_checkout" },
- query => $query,
- type => "opac",
- debug => 1,
-});
-if (C4::Context->preference('SelfCheckoutByLogin'))
-{
- $template->param(authbylogin => 1);
-}
+my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+ {
+ template_name => "sco/sco-main.tt",
+ authnotrequired => 0,
+ flagsrequired => { self_check => "self_checkout_module" },
+ query => $query,
+ type => "opac",
+ debug => 1,
+ }
+);
# Get the self checkout timeout preference, or use 120 seconds as a default
my $selfchecktimeout = 120000;
if (C4::Context->preference('SelfCheckTimeout')) {
$selfchecktimeout = C4::Context->preference('SelfCheckTimeout') * 1000;
}
-$template->param(SelfCheckTimeout => $selfchecktimeout);
+$template->param( SelfCheckTimeout => $selfchecktimeout );
# Checks policy laid out by AllowSelfCheckReturns, defaults to 'on' if preference is undefined
my $allowselfcheckreturns = 1;
if (defined C4::Context->preference('AllowSelfCheckReturns')) {
$allowselfcheckreturns = C4::Context->preference('AllowSelfCheckReturns');
}
-$template->param(AllowSelfCheckReturns => $allowselfcheckreturns);
-
my $issuerid = $loggedinuser;
-my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed) = (
+my ($op, $patronid, $patronlogin, $patronpw, $barcode, $confirmed, $newissues) = (
$query->param("op") || '',
$query->param("patronid") || '',
$query->param("patronlogin")|| '',
$query->param("patronpw") || '',
$query->param("barcode") || '',
$query->param("confirmed") || '',
+ $query->param("newissues") || '',
);
-
+my @newissueslist = split /,/, $newissues;
my $issuenoconfirm = 1; #don't need to confirm on issue.
-#warn "issuerid: " . $issuerid;
-my $issuer = GetMember( borrowernumber => $issuerid );
+my $issuer = Koha::Patrons->find( $issuerid )->unblessed;
my $item = GetItem(undef,$barcode);
if (C4::Context->preference('SelfCheckoutByLogin') && !$patronid) {
my $dbh = C4::Context->dbh;
my $resval;
($resval, $patronid) = checkpw($dbh, $patronlogin, $patronpw);
}
-my $borrower = GetMember( cardnumber => $patronid );
-my $currencySymbol = "";
-if ( my $active_currency = Koha::Acquisition::Currencies->get_active ) {
- $currencySymbol = $active_currency->symbol;
+my ( $borrower, $patron );
+if ( $patronid ) {
+ $patron = Koha::Patrons->find( { cardnumber => $patronid } );
+ $borrower = $patron->unblessed if $patron;
}
my $branch = $issuer->{branchcode};
#warn "issuer cardnumber: " . $issuer->{cardnumber};
#warn "patron cardnumber: " . $borrower->{cardnumber};
if ($op eq "logout") {
+ $template->param( loggedout => 1 );
$query->param( patronid => undef, patronlogin => undef, patronpw => undef );
}
elsif ( $op eq "returnbook" && $allowselfcheckreturns ) {
my ($doreturn) = AddReturn( $barcode, $branch );
- #warn "returnbook: " . $doreturn;
- $borrower = GetMember( cardnumber => $patronid );
+ $template->param( returned => $doreturn );
}
-elsif ( $op eq "checkout" ) {
+elsif ( $patron && ( $op eq 'checkout' || $op eq 'renew' ) ) {
my $impossible = {};
my $needconfirm = {};
- if ( !$confirmed ) {
- ( $impossible, $needconfirm ) = CanBookBeIssued(
- $borrower,
- $barcode,
- undef,
- 0,
- C4::Context->preference("AllowItemsOnHoldCheckoutSCO")
- );
+ ( $impossible, $needconfirm ) = CanBookBeIssued(
+ $patron,
+ $barcode,
+ undef,
+ 0,
+ C4::Context->preference("AllowItemsOnHoldCheckoutSCO")
+ );
+ my $issue_error;
+ if ( $confirm_required = scalar keys %$needconfirm ) {
+ for my $error ( qw( UNKNOWN_BARCODE max_loans_allowed ISSUED_TO_ANOTHER NO_MORE_RENEWALS NOT_FOR_LOAN DEBT WTHDRAWN RESTRICTED RESERVED ITEMNOTSAMEBRANCH EXPIRED DEBARRED CARD_LOST GNA INVALID_DATE UNKNOWN_BARCODE TOO_MANY DEBT_GUARANTEES USERBLOCKEDOVERDUE PATRON_CANT PREVISSUE NOT_FOR_LOAN_FORCING ITEM_LOST) ) {
+ if ( $needconfirm->{$error} ) {
+ $issue_error = $error;
+ $confirmed = 0;
+ last;
+ }
+ }
}
- $confirm_required = scalar keys %$needconfirm;
#warn "confirm_required: " . $confirm_required ;
if (scalar keys %$impossible) {
- # warn "impossible: numkeys: " . scalar (keys(%$impossible));
- #warn join " ", keys %$impossible;
- my $issue_error = (keys %$impossible)[0];
+ my $issue_error = (keys %$impossible)[0]; # FIXME This is wrong, we assume only one error and keys are not ordered
- # FIXME we assume only one error.
$template->param(
impossible => $issue_error,
"circ_error_$issue_error" => 1,
hide_main => 1,
);
if ($issue_error eq 'DEBT') {
- $template->param(amount => $currencySymbol.$impossible->{DEBT});
+ $template->param(DEBT => $impossible->{DEBT});
}
#warn "issue_error: " . $issue_error ;
if ( $issue_error eq "NO_MORE_RENEWALS" ) {
barcode => $barcode,
);
}
- } elsif ( $needconfirm->{RENEW_ISSUE} ) {
+ } elsif ( $needconfirm->{RENEW_ISSUE} || $op eq 'renew' ) {
if ($confirmed) {
#warn "renewing";
- AddRenewal( $borrower, $item->{itemnumber} );
+ AddRenewal( $borrower->{borrowernumber}, $item->{itemnumber} );
+ push @newissueslist, $barcode;
+ $template->param( renewed => 1 );
} else {
#warn "renew confirmation";
$template->param(
}
} elsif ( $confirm_required && !$confirmed ) {
#warn "failed confirmation";
- my $issue_error = (keys %$needconfirm)[0];
$template->param(
- impossible => (keys %$needconfirm)[0],
+ impossible => 1,
"circ_error_$issue_error" => 1,
hide_main => 1,
);
if ($issue_error eq 'DEBT') {
- $template->param(amount => $currencySymbol.$needconfirm->{DEBT});
+ $template->param(DEBT => $needconfirm->{DEBT});
}
} else {
if ( $confirmed || $issuenoconfirm ) { # we'll want to call getpatroninfo again to get updated issues.
- my $hold_existed;
+ my ( $hold_existed, $item );
if ( C4::Context->preference('HoldFeeMode') eq 'any_time_is_collected' ) {
# There is no easy way to know if the patron has been charged for this item.
# So we check if a hold existed for this item before the check in
- my $item = Koha::Items->find({ barcode => $barcode });
- $hold_existed = Koha::Holds->search({ -or => { 'biblionumber' => $item->biblionumber, 'itemnumber' => $item->itemnumber}})->count;
+ $item = Koha::Items->find({ barcode => $barcode });
+ $hold_existed = Koha::Holds->search(
+ {
+ -and => {
+ borrowernumber => $borrower->{borrowernumber},
+ -or => {
+ biblionumber => $item->biblionumber,
+ itemnumber => $item->itemnumber
+ }
+ }
+ }
+ )->count;
}
+
AddIssue( $borrower, $barcode );
+ push @newissueslist, $barcode;
if ( $hold_existed ) {
my $dtf = Koha::Database->new->schema->storage->datetime_parser;
$template->param(
+ issued => 1,
# If the hold existed before the check in, let's confirm that the charge line exists
# Note that this should not be needed but since we do not have proper exception handling here we do it this way
- patron_has_hold_fee => Koha::Account::Lines->search({ borrowernumber => $borrower->{borrowernumber}, accounttype => 'Res', date => $dtf->format_date( dt_from_string ) })->count,
+ patron_has_hold_fee => Koha::Account::Lines->search(
+ {
+ borrowernumber => $borrower->{borrowernumber},
+ accounttype => 'Res',
+ description => 'Reserve Charge - ' . $item->biblio->title,
+ date => $dtf->format_date(dt_from_string)
+ }
+ )->count,
);
}
} else {
}
} # $op
-if ($borrower->{cardnumber}) {
+if ($borrower) {
# warn "issuer's branchcode: " . $issuer->{branchcode};
# warn "user's branchcode: " . $borrower->{branchcode};
my $borrowername = sprintf "%s %s", ($borrower->{firstname} || ''), ($borrower->{surname} || '');
- my @issues;
- my ($issueslist) = GetPendingIssues( $borrower->{'borrowernumber'} );
- foreach my $it (@$issueslist) {
+ my $pending_checkouts = $patron->pending_checkouts;
+ my @checkouts;
+ while ( my $c = $pending_checkouts->next ) {
+ my $checkout = $c->unblessed_all_relateds;
my ($can_be_renewed, $renew_error) = CanBookBeRenewed(
$borrower->{borrowernumber},
- $it->{itemnumber},
+ $checkout->{itemnumber},
);
- $it->{can_be_renewed} = $can_be_renewed;
- $it->{renew_error} = $renew_error;
- $it->{date_due} = $it->{date_due_sql};
- push @issues, $it;
+ $checkout->{can_be_renewed} = $can_be_renewed; # In the future this will be $checkout->can_be_renewed
+ $checkout->{renew_error} = $renew_error;
+ $checkout->{overdue} = $c->is_overdue;
+ push @checkouts, $checkout;
}
$template->param(
validuser => 1,
borrowername => $borrowername,
- issues_count => scalar(@issues),
- ISSUES => \@issues,
+ issues_count => scalar(@checkouts),
+ ISSUES => \@checkouts,
+ newissues => join(',',@newissueslist),
patronid => $patronid,
patronlogin => $patronlogin,
patronpw => $patronpw,
my $patron_image = Koha::Patron::Images->find($borrower->{borrowernumber});
$template->param(
display_patron_image => 1,
- cardnumber => $borrower->{cardnumber},
+ csrf_token => Koha::Token->new->generate_csrf( { session_id => scalar $query->cookie('CGISESSID') . $borrower->{cardnumber}, id => $borrower->{userid}} ),
) if $patron_image;
}
} else {
);
}
-$template->param(
- SCOUserJS => C4::Context->preference('SCOUserJS'),
- SCOUserCSS => C4::Context->preference('SCOUserCSS'),
-);
-
output_html_with_http_headers $query, $cookie, $template->output, undef, { force_no_caching => 1 };