From 5c529ccd6f90597e20e3651e4e0442fa7a9fd464 Mon Sep 17 00:00:00 2001 From: Adrien Saurat Date: Thu, 13 Dec 2012 10:13:19 +0100 Subject: [PATCH] Bug 8306: Patrons statistics, fix for patron activity choice The "inactive" for "patron activity" choice is now effective. --- reports/borrowers_stats.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/reports/borrowers_stats.pl b/reports/borrowers_stats.pl index 9e00ea3..b51f871 100755 --- a/reports/borrowers_stats.pl +++ b/reports/borrowers_stats.pl @@ -322,7 +322,7 @@ sub calculate { @$filters[7]=~ s/\*/%/g if (@$filters[7]); $strcalc .= " AND sort2 like '" . @$filters[7] ."'" if ( @$filters[7] ); $strcalc .= " AND borrowernumber in (select distinct(borrowernumber) from old_issues where issuedate > '" . $newperioddate . "')" if ($activity eq 'active'); - $strcalc .= " AND borrowernumber not in (select distinct(borrowernumber) from old_issues where issuedate > '" . $newperioddate . "')" if ($activity eq 'nonactive'); + $strcalc .= " AND borrowernumber not in (select distinct(borrowernumber) from old_issues where issuedate > '" . $newperioddate . "' AND borrowernumber IS NOT NULL)" if ($activity eq 'nonactive'); $strcalc .= " AND $status='1' " if ($status); $strcalc .= " group by $linefield, $colfield"; push @loopfilter, {sql=>1, crit=>"Query", filter=>$strcalc}; -- 1.7.9.5