From 208e72a16b8ee79f1da0f24fe0af4e070acf73af 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: Nick Clemens Signed-off-by: Mason James --- 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 607f9d1588..61502cfe33 100755 --- a/reports/borrowers_stats.pl +++ b/reports/borrowers_stats.pl @@ -422,7 +422,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.39.5