From c7ad52698c1692e6762ee01ba2dab80ecf1a46b2 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. Signed-off-by: Julian Maurice --- 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 795ca8e..9bdf37d 100755 --- a/reports/borrowers_stats.pl +++ b/reports/borrowers_stats.pl @@ -407,7 +407,7 @@ sub calculate { } } $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 "; -- 2.1.4