From 367a0e95bbd024e28246ece665ca5077382e5106 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 1 Jun 2017 13:52:30 -0300 Subject: [PATCH] Bug 18789: Use Koha::Patron->is_child where needed Test plan: When you are on a page related to a child you should see the "Update child to adult patron" button --- circ/circulation.pl | 1 - koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc | 6 +++--- members/boraccount.pl | 1 - members/mancredit.pl | 1 - members/maninvoice.pl | 1 - members/member-flags.pl | 1 - members/member-password.pl | 1 - members/moremember.pl | 1 - members/printfeercpt.pl | 1 - members/printinvoice.pl | 1 - members/readingrec.pl | 1 - 11 files changed, 3 insertions(+), 13 deletions(-) diff --git a/circ/circulation.pl b/circ/circulation.pl index db329d46cb..d01c9c3001 100755 --- a/circ/circulation.pl +++ b/circ/circulation.pl @@ -606,7 +606,6 @@ if ( $patron ) { patron => $patron, categoryname => $patron->category->description, expiry => $patron->dateexpiry, - is_child => ( $patron->category->category_type eq 'C' ), ); } diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc index 3d5276986f..91c8d2a6a5 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc @@ -34,7 +34,7 @@ $(document).ready(function(){ $(".btn-group").removeClass("open"); return false; }); - [% IF ( is_child ) %]$("#updatechild").click(function(){ + [% IF ( patron.is_child ) %]$("#updatechild").click(function(){ update_child(); $(".btn-group").removeClass("open"); });[% END %] @@ -92,7 +92,7 @@ function confirm_both_deletion() { } } -[% IF ( is_child ) %]function confirm_updatechild() { +[% IF ( patron.is_child ) %]function confirm_updatechild() { var is_confirmed = window.confirm(_("Are you sure you want to update this child to an Adult category? This cannot be undone.")); if (is_confirmed) { window.location='/cgi-bin/koha/members/update-child.pl?op=update&borrowernumber=[% patron.borrowernumber %]&catcode=[% catcode %]&catcode_multi=[% CATCODE_MULTI %]'; @@ -197,7 +197,7 @@ function searchToHold(){ [% ELSE %]
  • Delete
  • [% END %] - [% IF ( is_child ) %] + [% IF ( patron.is_child ) %]
  • Update child to adult patron
  • [% ELSE %]
  • Update child to adult patron
  • diff --git a/members/boraccount.pl b/members/boraccount.pl index 3600ed22e7..c647940317 100755 --- a/members/boraccount.pl +++ b/members/boraccount.pl @@ -108,7 +108,6 @@ $template->param( finesview => 1, total => sprintf("%.2f",$total), totalcredit => $totalcredit, - is_child => ($patron->category->category_type eq 'C'), reverse_col => $reverse_col, accounts => $accts, RoutingSerials => C4::Context->preference('RoutingSerials'), diff --git a/members/mancredit.pl b/members/mancredit.pl index 4b0c9b167f..72acf52ba3 100755 --- a/members/mancredit.pl +++ b/members/mancredit.pl @@ -96,7 +96,6 @@ if ($add){ $template->param( finesview => 1, - is_child => ($patron->category->category_type eq 'C'), # FIXME is_child should be a Koha::Patron method RoutingSerials => C4::Context->preference('RoutingSerials'), ); output_html_with_http_headers $input, $cookie, $template->output; diff --git a/members/maninvoice.pl b/members/maninvoice.pl index f2872c8eba..7de26e7198 100755 --- a/members/maninvoice.pl +++ b/members/maninvoice.pl @@ -122,7 +122,6 @@ if ($add){ $template->param( patron => $patron, finesview => 1, - is_child => ($patron->category->category_type eq 'C'), RoutingSerials => C4::Context->preference('RoutingSerials'), ); output_html_with_http_headers $input, $cookie, $template->output; diff --git a/members/member-flags.pl b/members/member-flags.pl index 190c89c061..cf90f03ce8 100755 --- a/members/member-flags.pl +++ b/members/member-flags.pl @@ -196,7 +196,6 @@ if (C4::Context->preference('ExtendedPatronAttributes')) { $template->param( patron => $patron, loop => \@loop, - is_child => ($category_type eq 'C'), RoutingSerials => C4::Context->preference('RoutingSerials'), csrf_token => Koha::Token->new->generate_csrf( { session_id => scalar $input->cookie('CGISESSID'), } ), ); diff --git a/members/member-password.pl b/members/member-password.pl index 19201fcaf9..3565906995 100755 --- a/members/member-password.pl +++ b/members/member-password.pl @@ -120,7 +120,6 @@ if ( C4::Context->preference('ExtendedPatronAttributes') ) { $template->param( patron => $patron, destination => $destination, - is_child => ( $category_type eq 'C' ), minPasswordLength => $minpw, RoutingSerials => C4::Context->preference('RoutingSerials'), csrf_token => Koha::Token->new->generate_csrf({ session_id => scalar $input->cookie('CGISESSID'), }), diff --git a/members/moremember.pl b/members/moremember.pl index 08a017072a..33acf8ca28 100755 --- a/members/moremember.pl +++ b/members/moremember.pl @@ -342,7 +342,6 @@ $template->param( totaldue_raw => $total, overdues_exist => $overdues_exist, StaffMember => $category_type eq 'S', - is_child => $category_type eq 'C', samebranch => $samebranch, quickslip => $quickslip, housebound_role => scalar $patron->housebound_role, diff --git a/members/printfeercpt.pl b/members/printfeercpt.pl index 2e44ea9ed3..5c1ba72f43 100755 --- a/members/printfeercpt.pl +++ b/members/printfeercpt.pl @@ -122,7 +122,6 @@ $template->param( finesview => 1, total => sprintf("%.2f",$total), totalcredit => $totalcredit, - is_child => ($category->category_type eq 'C'), accounts => \@accountrows ); output_html_with_http_headers $input, $cookie, $template->output; diff --git a/members/printinvoice.pl b/members/printinvoice.pl index 22cba6d2f8..f9a2bdc2c5 100755 --- a/members/printinvoice.pl +++ b/members/printinvoice.pl @@ -122,7 +122,6 @@ $template->param( finesview => 1, total => sprintf( "%.2f", $total ), totalcredit => $totalcredit, - is_child => ( $category->category_type eq 'C' ), accounts => \@accountrows ); diff --git a/members/readingrec.pl b/members/readingrec.pl index eb6a1f0c7b..e495542b8c 100755 --- a/members/readingrec.pl +++ b/members/readingrec.pl @@ -116,7 +116,6 @@ if (C4::Context->preference('ExtendedPatronAttributes')) { $template->param( patron => $patron, readingrecordview => 1, - is_child => ( $category->category_type eq 'C' ), loop_reading => $issues, RoutingSerials => C4::Context->preference('RoutingSerials'), ); -- 2.11.0