Attachment 128026 Details for Bug 21982 – Bug 21982: Circulation statistics wizard does not count deleted items

[patch] Bug 21982: Circulation statistics wizard does not count deleted items

Bug-21982-Circulation-statistics-wizard-does-not-c.patch (text/plain), 1.52 KB, created by Jérémy Breuillard on 2021-11-26 08:42:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jérémy Breuillard

Created: 2021-11-26 08:42:01 UTC

Size: 1.52 KB

patch

obsolete

>From 1759da1d4347a8b4f9cb80b1f63817786777573b Mon Sep 17 00:00:00 2001
>From: jeremy breuillard <jeremy.breuillard@biblibre.com>
>Date: Wed, 24 Nov 2021 16:56:09 +0100
>Subject: [PATCH] Bug 21982: Circulation statistics wizard does not count
> deleted items
>
>By default statistic assistant filter is set on the
>existing items and doesn't mind about deleted items.
>The patch corrects this. Items belong to the same library.
>
>Test plan:
>1)Make sure to have at least 1 or more item(s) in deleteditems database table
>2)Home > Reports > Circulation statistics
>3)Fill up 'Issuing library', 'Home library' and 'Holding library'
>4)Submit
>5)Notice that only statistics for active items are shown
>6)Apply patch and repeat from 2) to 5)
>7)Now active items and deleted items are shown
>---
> reports/issues_stats.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/reports/issues_stats.pl b/reports/issues_stats.pl
>index 3eb27fb3e2..995604d633 100755
>--- a/reports/issues_stats.pl
>+++ b/reports/issues_stats.pl
>@@ -504,7 +504,7 @@ sub calculate {
>             $strcalc .= " LEFT JOIN borrower_attributes AS attribute_$_ ON (statistics.borrowernumber = attribute_$_.borrowernumber AND attribute_$_.code = '$_') ";
>         }
>     }
>-    $strcalc .= "LEFT JOIN items ON statistics.itemnumber=items.itemnumber "
>+    $strcalc .= "LEFT JOIN (SELECT * FROM items UNION SELECT * FROM deleteditems) items ON statistics.itemnumber=items.itemnumber "
>       if ( $linefield =~ /^items\./
>         or $colfield =~ /^items\./
>         or $process == 5
>-- 
>2.17.1

Actions: View | Diff | Splinter Review

Attachments on bug 21982: 128011 | 128012 | 128013 | 128026 | 138075 | 138106