# FIXME There are too many calls to Koha::Patrons->find in this script
-use strict;
-use warnings;
+use Modern::Perl;
use CGI qw ( -utf8 );
use DateTime;
use DateTime::Duration;
+use Scalar::Util qw( looks_like_number );
use C4::Output;
use C4::Print;
use C4::Auth qw/:DEFAULT get_session haspermission/;
my $barcodes = [];
my $barcode = $query->param('barcode');
+my $findborrower;
+my $autoswitched;
+my $borrowernumber = $query->param('borrowernumber');
+
+if (C4::Context->preference("AutoSwitchPatron") && $barcode) {
+ if (Koha::Patrons->search( { cardnumber => $barcode} )->count() > 0) {
+ $findborrower = $barcode;
+ undef $barcode;
+ undef $borrowernumber;
+ $autoswitched = 1;
+ }
+}
+$findborrower ||= $query->param('findborrower') || q{};
+$findborrower =~ s|,| |g;
+
# Barcode given by user could be '0'
if ( $barcode || ( defined($barcode) && $barcode eq '0' ) ) {
$barcodes = [ $barcode ];
$barcodes = [ uniq @$barcodes ];
my $template_name = q|circ/circulation.tt|;
-my $borrowernumber = $query->param('borrowernumber');
my $patron = $borrowernumber ? Koha::Patrons->find( $borrowernumber ) : undef;
my $batch = $query->param('batch');
my $batch_allowed = 0;
flagsrequired => { circulate => 'circulate_remaining_permissions' },
}
);
+my $logged_in_user = Koha::Patrons->find( $loggedinuser ) or die "Not logged in";
my $force_allow_issue = $query->param('forceallow') || 0;
if (!C4::Auth::haspermission( C4::Context->userenv->{id} , { circulate => 'force_checkout' } )) {
my $searchtype = $query->param('searchtype') || q{contain};
-my $findborrower = $query->param('findborrower') || q{};
-$findborrower =~ s|,| |g;
-
my $branch = C4::Context->userenv->{'branch'};
if (C4::Context->preference("DisplayClearScreenButton")) {
}
# get the borrower information.....
-if ($borrowernumber) {
- $patron = Koha::Patrons->find( $borrowernumber );
+my $balance = 0;
+$patron ||= Koha::Patrons->find( $borrowernumber ) if $borrowernumber;
+if ($patron) {
+
+ $template->param( borrowernumber => $patron->borrowernumber );
+ output_and_exit_if_error( $query, $cookie, $template, { module => 'members', logged_in_user => $logged_in_user, current_patron => $patron } );
+
my $overdues = $patron->get_overdues;
my $issues = $patron->checkouts;
- my $balance = $patron->account->balance;
+ $balance = $patron->account->balance;
# if the expiry date is before today ie they have expired
my $template_params = { barcode => $barcode };
# always check for blockers on issuing
my ( $error, $question, $alerts, $messages ) = CanBookBeIssued(
- $patron->unblessed,
+ $patron,
$barcode, $datedue,
$inprocess,
undef,
if ( $item ) {
$biblio = $item->biblio;
- my $mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $biblio->frameworkcode, kohafield => 'items.notforloan', authorised_value => { not => undef } });
+ my $mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $biblio->frameworkcode, kohafield => 'items.notforloan', authorised_value => [ -and => {'!=' => undef }, {'!=' => ''}] });
$template_params->{authvalcode_notforloan} = $mss->count ? $mss->next->authorised_value : undef;
}
}
}
- unless( $onsite_checkout and C4::Context->preference("OnSiteCheckoutsForce") ) {
+ if ( $error->{UNKNOWN_BARCODE} or not $onsite_checkout or not C4::Context->preference("OnSiteCheckoutsForce") ) {
delete $question->{'DEBT'} if ($debt_confirmed);
foreach my $impossible ( keys %$error ) {
$template_params->{$impossible} = $$error{$impossible};
$blocker = 1;
}
}
- if( !$blocker || $force_allow_issue ){
+
+ if( $item and ( !$blocker or $force_allow_issue ) ){
my $confirm_required = 0;
unless($issueconfirmed){
# Get the item title for more information
##################################################################################
# BUILD HTML
# show all reserves of this borrower, and the position of the reservation ....
-if ($borrowernumber) {
- my $holds = Koha::Holds->search( { borrowernumber => $borrowernumber } );
+if ($patron) {
+ my $holds = Koha::Holds->search( { borrowernumber => $borrowernumber } ); # FIXME must be Koha::Patron->holds
my $waiting_holds = $holds->waiting;
$template->param(
holds_count => $holds->count(),
WaitingHolds => $waiting_holds,
);
-
- my $category_type = $patron->category->category_type;
- $template->param( adultborrower => 1 ) if ( $category_type eq 'A' || $category_type eq 'I' );
}
-#title
-my $flags = $patron ? C4::Members::patronflags( $patron->unblessed ) : {};
-foreach my $flag ( sort keys %$flags ) {
- $flags->{$flag}->{'message'} =~ s#\n#<br />#g;
- if ( $flags->{$flag}->{'noissues'} ) {
+if ( $patron ) {
+ my $noissues;
+ if ( $patron->gonenoaddress ) {
+ $template->param( gna => 1 );
+ $noissues = 1;
+ }
+ if ( $patron->lost ) {
+ $template->param( lost=> 1 );
+ $noissues = 1;
+ }
+ if ( $patron->is_debarred ) {
+ $template->param( dbarred=> 1 );
+ $noissues = 1;
+ }
+ my $account = $patron->account;
+ if( ( my $owing = $account->non_issues_charges ) > 0 ) {
+ my $noissuescharge = C4::Context->preference("noissuescharge") || 5; # FIXME If noissuescharge == 0 then 5, why??
+ $noissues ||= ( not C4::Context->preference("AllowFineOverride") and ( $owing > $noissuescharge ) );
$template->param(
- noissues => ($force_allow_issue) ? 0 : 'true',
- forceallow => $force_allow_issue,
+ charges => 1,
+ chargesamount => $owing,
+ )
+ } elsif ( $balance < 0 ) {
+ $template->param(
+ credits => 1,
+ creditsamount => -$balance,
);
- if ( $flag eq 'GNA' ) {
- $template->param( gna => 'true' );
- }
- elsif ( $flag eq 'LOST' ) {
- $template->param( lost => 'true' );
- }
- elsif ( $flag eq 'DBARRED' ) {
- $template->param( dbarred => 'true' );
- }
- elsif ( $flag eq 'CHARGES' ) {
- $template->param(
- charges => 'true',
- chargesmsg => $flags->{'CHARGES'}->{'message'},
- chargesamount => $flags->{'CHARGES'}->{'amount'},
- charges_is_blocker => 1
- );
- }
- elsif ( $flag eq 'CHARGES_GUARANTEES' ) {
- $template->param(
- charges_guarantees => 'true',
- chargesmsg_guarantees => $flags->{'CHARGES_GUARANTEES'}->{'message'},
- chargesamount_guarantees => $flags->{'CHARGES_GUARANTEES'}->{'amount'},
- charges_guarantees_is_blocker => 1
- );
- }
- elsif ( $flag eq 'CREDITS' ) {
- $template->param(
- credits => 'true',
- creditsmsg => $flags->{'CREDITS'}->{'message'},
- creditsamount => sprintf("%.02f", -($flags->{'CREDITS'}->{'amount'})), # from patron's pov
- );
- }
}
- else {
- if ( $flag eq 'CHARGES' ) {
- $template->param(
- charges => 'true',
- chargesmsg => $flags->{'CHARGES'}->{'message'},
- chargesamount => $flags->{'CHARGES'}->{'amount'},
- );
- }
- elsif ( $flag eq 'CHARGES_GUARANTEES' ) {
- $template->param(
- charges_guarantees => 'true',
- chargesmsg_guarantees => $flags->{'CHARGES_GUARANTEES'}->{'message'},
- chargesamount_guarantees => $flags->{'CHARGES_GUARANTEES'}->{'amount'},
- );
- }
- elsif ( $flag eq 'CREDITS' ) {
- $template->param(
- credits => 'true',
- creditsmsg => $flags->{'CREDITS'}->{'message'},
- creditsamount => sprintf("%.02f", -($flags->{'CREDITS'}->{'amount'})), # from patron's pov
- );
- }
- elsif ( $flag eq 'ODUES' ) {
- $template->param(
- odues => 'true',
- oduesmsg => $flags->{'ODUES'}->{'message'}
- );
- my $items = $flags->{$flag}->{'itemlist'};
- if ( ! $query->param('module') || $query->param('module') ne 'returns' ) {
- $template->param( nonreturns => 'true' );
- }
+ my $no_issues_charge_guarantees = C4::Context->preference("NoIssuesChargeGuarantees");
+ $no_issues_charge_guarantees = undef unless looks_like_number( $no_issues_charge_guarantees );
+ if ( defined $no_issues_charge_guarantees ) {
+ my $guarantees_non_issues_charges = 0;
+ my $guarantees = $patron->guarantees;
+ while ( my $g = $guarantees->next ) {
+ $guarantees_non_issues_charges += $g->account->non_issues_charges;
}
- elsif ( $flag eq 'NOTES' ) {
+ if ( $guarantees_non_issues_charges > $no_issues_charge_guarantees ) {
$template->param(
- notes => 'true',
- notesmsg => $flags->{'NOTES'}->{'message'}
+ charges_guarantees => 1,
+ chargesamount_guarantees => $guarantees_non_issues_charges,
);
+ $noissues = 1 unless C4::Context->preference("allowfineoverride");
}
}
-}
-my $amountold = $flags ? $flags->{'CHARGES'}->{'message'} || 0 : 0;
-$amountold =~ s/^.*\$//; # remove upto the $, if any
+ if ( $patron->has_overdues ) {
+ $template->param( odues => 1 );
+ }
-my ( $total, $accts, $numaccts) = GetMemberAccountRecords( $borrowernumber );
+ if ( $patron->borrowernotes ) {
+ my $borrowernotes = $patron->borrowernotes;
+ $borrowernotes =~ s#\n#<br />#g;
+ $template->param(
+ notes =>1,
+ notesmsg => $borrowernotes,
+ )
+ }
-if ( $patron && $patron->category->category_type eq 'C') {
- my $patron_categories = Koha::Patron::Categories->search_limited({ category_type => 'A' }, {order_by => ['categorycode']});
- $template->param( 'CATCODE_MULTI' => 1) if $patron_categories->count > 1;
- $template->param( 'catcode' => $patron_categories->next ) if $patron_categories->count == 1;
+ if ( $noissues ) {
+ $template->param(
+ noissues => ($force_allow_issue) ? 0 : 'true',
+ forceallow => $force_allow_issue,
+ );
+ }
}
my $messages = Koha::Patron::Messages->search(
my $av = Koha::AuthorisedValues->search({ category => 'ROADTYPE', authorised_value => $patron->streettype });
my $roadtype = $av->count ? $av->next->lib : '';
$template->param(
- %{ $patron->unblessed },
- borrower => $patron->unblessed,
roadtype => $roadtype,
patron => $patron,
categoryname => $patron->category->description,
expiry => $patron->dateexpiry,
- is_child => ( $patron->category->category_type eq 'C' ),
- picture => ( $patron->image ? 1 : 0 ),
);
}
borrowernumber => $borrowernumber,
branch => $branch,
was_renewed => scalar $query->param('was_renewed') ? 1 : 0,
- amountold => $amountold,
barcodes => $barcodes,
stickyduedate => $stickyduedate,
duedatespec => $duedatespec,
restoreduedatespec => $restoreduedatespec,
message => $message,
- totaldue => sprintf('%.2f', $total),
+ totaldue => sprintf('%.2f', $balance), # FIXME not used in template?
inprocess => $inprocess,
$view => 1,
batch_allowed => $batch_allowed,
AudioAlerts => C4::Context->preference("AudioAlerts"),
fast_cataloging => $fast_cataloging,
CircAutoPrintQuickSlip => C4::Context->preference("CircAutoPrintQuickSlip"),
- SuspendHoldsIntranet => C4::Context->preference('SuspendHoldsIntranet'),
- AutoResumeSuspendedHolds => C4::Context->preference('AutoResumeSuspendedHolds'),
RoutingSerials => C4::Context->preference('RoutingSerials'),
relatives_issues_count => $relatives_issues_count,
relatives_borrowernumbers => \@relatives,
debt_confirmed => $debt_confirmed,
SpecifyDueDate => $duedatespec_allow,
CircAutocompl => C4::Context->preference("CircAutocompl"),
- debarments => GetDebarments({ borrowernumber => $borrowernumber }),
+ debarments => scalar GetDebarments({ borrowernumber => $borrowernumber }),
todaysdate => output_pref( { dt => dt_from_string()->set(hour => 23)->set(minute => 59), dateformat => 'sql' } ),
has_modifications => $has_modifications,
override_high_holds => $override_high_holds,
nopermission => scalar $query->param('nopermission'),
+ autoswitched => $autoswitched,
);
output_html_with_http_headers $query, $cookie, $template->output;