From f2dee024a7e503132fd0c82a4f47d8c5353613e4 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=utf8 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 Signed-off-by: Marcel de Rooy Signed-off-by: Kyle M Hall --- members/moremember.pl | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/members/moremember.pl b/members/moremember.pl index 420bc41125..0027099fe1 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.39.2