From 302da68ce5c0268edb9a64db15e875fbbc76c7bf Mon Sep 17 00:00:00 2001
From: Ryan Higgins <rch@liblime.com>
Date: Sun, 25 May 2008 02:54:18 -0500
Subject: [PATCH] Fill statistics table properly for renewals; and
 issue->checkout, return->checkin

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 C4/Circulation.pm                                                 | 4 ++--
 koha-tmpl/intranet-tmpl/prog/en/modules/reports/issues_stats.tmpl | 5 +++--
 reports/issues_stats.pl                                           | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/C4/Circulation.pm b/C4/Circulation.pm
index 5712c0e4c1..0880c99f0d 100644
--- a/C4/Circulation.pm
+++ b/C4/Circulation.pm
@@ -1034,7 +1034,7 @@ sub AddIssue {
             C4::Context->userenv->{'branch'},
             'issue',                        $charge,
             '',                             $item->{'itemnumber'},
-            $item->{'itemtype'}, $borrower->{'borrowernumber'}
+            $item->{'itype'}, $borrower->{'borrowernumber'}
         );
     }
     
@@ -1879,7 +1879,7 @@ sub AddRenewal {
         $sth->finish;
     }
     # Log the renewal
-    UpdateStats( $branch, 'renew', $charge, '', $itemnumber );
+    UpdateStats( $branch, 'renew', $charge, '', $itemnumber, $item->{itype}, $borrowernumber);
 }
 
 sub GetRenewCount {
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/issues_stats.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/issues_stats.tmpl
index 7089ce4f1c..a37003acba 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/issues_stats.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/issues_stats.tmpl
@@ -145,8 +145,9 @@
                 </tr>
                 <tr class="hilighted"> 
                 <td><label for="PeriodTypeSel">Type:</label><select name="PeriodTypeSel" id="PeriodTypeSel">
-                    <option value ="issue">Issue</option>
-                    <option value ="return">Return</option>
+                    <option value ="issue">Checkout</option>
+                    <option value ="renew">Renewal</option>
+                    <option value ="return">Checkin</option>
                     <option value ="localuse">Local Use</option>
                     </select></td>
                 <td colspan="2"><label for="DisplayBy">group by</label> <select name="DisplayBy" id="DisplayBy">
diff --git a/reports/issues_stats.pl b/reports/issues_stats.pl
index 61b9ce76fe..8bce853876 100755
--- a/reports/issues_stats.pl
+++ b/reports/issues_stats.pl
@@ -291,7 +291,7 @@ sub calculate {
                         push @loopfilter, \%cell;
                 }
         }
-        push @loopfilter,{crit=>"Issue|Return ",filter=>$type};
+        push @loopfilter,{crit=>"Event",filter=>$type};
         push @loopfilter,{crit=>"Display by ",filter=>$dsp} if ($dsp);
         push @loopfilter,{crit=>"Select Day ",filter=>$daysel} if ($daysel);
         push @loopfilter,{crit=>"Select Month ",filter=>$monthsel} if ($monthsel);
-- 
2.11.0