Bug 21008: Use Koha::Patron->is_child

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Edit: I removed the category parameter as it is not really used.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This commit is contained in:
Jonathan Druart 2018-06-28 11:00:57 -03:00 committed by Nick Clemens
parent 14f818edf6
commit 5995275b74
2 changed files with 5 additions and 5 deletions

View File

@ -234,7 +234,7 @@ sub borrower_add_additional_fields {
# some borrower info is not returned in the standard call despite being assumed
# in a number of templates. It should not be the business of this script but in lieu of
# a revised api here it is ...
if ( $patron->category->category_type eq 'C' ) {
if ( $patron->is_child ) {
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->categorycode ) if $patron_categories->count == 1;

View File

@ -180,7 +180,7 @@ if ( $total_paid and $total_paid ne '0.00' ) {
$total_paid = '0.00'; #TODO not right with pay_individual
}
borrower_add_additional_fields($borrower, $template, $category);
borrower_add_additional_fields($patron, $template);
$template->param(%$borrower);
@ -201,19 +201,19 @@ $template->param(
output_html_with_http_headers $input, $cookie, $template->output;
sub borrower_add_additional_fields {
my ( $b_ref, $template, $category ) = @_;
my ( $patron, $template ) = @_;
# some borrower info is not returned in the standard call despite being assumed
# in a number of templates. It should not be the business of this script but in lieu of
# a revised api here it is ...
if ( $category->category_type eq 'C' ) {
if ( $patron->is_child ) {
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->categorycode ) if $patron_categories->count == 1;
}
if (C4::Context->preference('ExtendedPatronAttributes')) {
my $extendedattributes = GetBorrowerAttributes($b_ref->{borrowernumber});
my $extendedattributes = GetBorrowerAttributes($patron->borrowernumber);
$template->param( extendedattributes => $extendedattributes );
}