Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 13108 - Members are restricted even if the debarment is ended - opac-reserve

Browse source 
This is the same issue as bug 12134.

Test Plan:
1) Add a manual restriction to a patron with expiration date in the past
2) Go on the OPAC and connect
3) Try to add an hold on a record (opac-reserve.pl)
3) Note the warning message
    "Sorry, you cannot place holds because your account has been frozen ..."
4) Apply this patch
5) Repeat step 2
6) Note the warning message does not appear anymore

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Makes code cleaner, also works as described.

NOTE: If you set a debarment with date in the past in the GUI,
there will be no entry in borrowers.debarred and you won't be
able to see the problem. Set one with a date in the future and
then alter the date in borrower_debarments and borrowers.debarred.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
This commit is contained in:
Fridolin Somers 2014-10-17 18:37:56 +02:00 committed by Tomas Cohen Arazi
parent b60192397c
commit a317f7d62f
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
opac/opac-reserve.pl
View file

@ -35,6 +35,7 @@ use C4::Branch; # GetBranches
use C4::Overdues;
use C4::Debug;
use Koha::DateUtils;
use Koha::Borrower::Debarments qw(IsDebarred);
use Date::Calc qw/Today Date_to_Days/;
# use Data::Dumper;
@ -320,7 +321,7 @@ if ( $borr->{lost} && ($borr->{lost} == 1) ) {
lost => 1
);
}
if ( $borr->{'debarred'} ) {
if ( IsDebarred($borrowernumber) ) {
$noreserves = 1;
$template->param(
message => 1,