From 2c4f5382afbabd85dadd066c9490c664c36ccb70 Mon Sep 17 00:00:00 2001 From: Katrin Fischer Date: Mon, 8 Jun 2015 04:17:53 +0200 Subject: [PATCH] Bug 10063: Correct documentation of C4::Members::IsMemberBlocked Rephrased documentation a bit, replacing fine days with the more general term restriction. As IsDebarred checks for existing active restrictions. TEST PLAN --------- 1) apply patch 2) git diff origin/master -- do the changes make sense 3) perldoc C4::Members -- look for the IsMemberBlocked. -- Does it reflect current state 4) koha qa test tools Signed-off-by: Mark Tompsett Signed-off-by: Jonathan Druart --- C4/Members.pm | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/C4/Members.pm b/C4/Members.pm index c5e5453..2843da4 100644 --- a/C4/Members.pm +++ b/C4/Members.pm @@ -515,25 +515,23 @@ sub GetMemberRelatives { my ($block_status, $count) = IsMemberBlocked( $borrowernumber ); -Returns whether a patron has overdue items that may result -in a block or whether the patron has active fine days -that would block circulation privileges. +Returns whether a patron is restricted or has overdue items that may result +in a block of circulation privileges. C<$block_status> can have the following values: -1 if the patron has outstanding fine days or a manual debarment, in which case +1 if the patron is currently restricted, in which case C<$count> is the expiration date (9999-12-31 for indefinite) -1 if the patron has overdue items, in which case C<$count> is the number of them 0 if the patron has no overdue items or outstanding fine days, in which case C<$count> is 0 -Outstanding fine days are checked before current overdue items -are. +Existing active restrictions are checked before current overdue items. FIXME: this needs to be split into two functions; a potential block based on the number of current overdue items could be orthogonal -to a block based on whether the patron has any fine days accrued. +to a block based on whether the patron has any restrictions. =cut -- 2.1.0