From 27b9b50ac9bc6869fd8206e8173ab7db2e5a4f77 Mon Sep 17 00:00:00 2001 From: Aleisha Amohia Date: Thu, 18 May 2017 23:10:13 +0000 Subject: [PATCH] Bug 18552: Borrower debarments not showing on member detail page MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit To test: 1) Go to a borrowers details page and create a manual restriction 2) Notice the restriction shows at the top of the page but the restriction tab says the member is currently unrestricted 3) Apply patch and refresh page 4) Restrictions tab should now correctly show debarments and correct number is shown 5) Confirming deleting and adding restrictions still works as expected Sponsored-by: Catalyst IT Followed test plan, works as expected Signed-off-by: Marc VĂ©ron --- members/moremember.pl | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/members/moremember.pl b/members/moremember.pl index 420bc41..0027099 100755 --- a/members/moremember.pl +++ b/members/moremember.pl @@ -155,7 +155,11 @@ for (qw(gonenoaddress lost borrowernotes)) { } if ( $patron->is_debarred ) { - $template->param( 'userdebarred' => 1, 'flagged' => 1 ); + $template->param( + userdebarred => 1, + flagged => 1, + debarments => GetDebarments({ borrowernumber => $borrowernumber }), + ); my $debar = $data->{'debarred'}; if ( $debar ne "9999-12-31" ) { $template->param( 'userdebarreddate' => output_pref( { dt => dt_from_string( $debar ), dateonly => 1 } ) ); @@ -359,7 +363,6 @@ $template->param( AutoResumeSuspendedHolds => C4::Context->preference('AutoResumeSuspendedHolds'), SuspendHoldsIntranet => C4::Context->preference('SuspendHoldsIntranet'), RoutingSerials => C4::Context->preference('RoutingSerials'), - debarments => GetDebarments({ borrowernumber => $borrowernumber }), PatronsPerPage => C4::Context->preference("PatronsPerPage") || 20, relatives_issues_count => $relatives_issues_count, relatives_borrowernumbers => \@relatives, -- 2.1.4