Bug 14262: change the build-git-snapshot version to 3.21

[srvgit] / C4 / Stats.pm

diff --git a/C4/Stats.pm b/C4/Stats.pm
index 078595d..c0c3514 100644 (file)
--- a/C4/Stats.pm
+++ b/C4/Stats.pm
@@ -5,18 +5,18 @@ package C4::Stats;
 #
 # This file is part of Koha.
 #
-# Koha is free software; you can redistribute it and/or modify it under the
-# terms of the GNU General Public License as published by the Free Software
-# Foundation; either version 2 of the License, or (at your option) any later
-# version.
+# Koha is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
 #
-# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
-# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
-# A PARTICULAR PURPOSE.  See the GNU General Public License for more details.
+# Koha is distributed in the hope that it will be useful, but
+# WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
 #
-# You should have received a copy of the GNU General Public License along
-# with Koha; if not, write to the Free Software Foundation, Inc.,
-# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
 
 use strict;
 use warnings;
@@ -87,14 +87,14 @@ sub UpdateStats {
     return () if ! defined $params;
 # change these arrays if new types of transaction or new parameters are allowed
     my @allowed_keys = qw (type branch amount other itemnumber itemtype borrowernumber accountno ccode);
-    my @allowed_circulation_types = qw (renew issue localuse return);
+    my @allowed_circulation_types = qw (renew issue localuse return onsite_checkout);
     my @allowed_accounts_types = qw (writeoff payment);
     my @circulation_mandatory_keys = qw (type branch borrowernumber itemnumber ccode itemtype);
     my @accounts_mandatory_keys = qw (type branch borrowernumber amount);
 
     my @mandatory_keys = ();
     if (! exists $params->{type} or ! defined $params->{type}) {
-        croak ("UpdateStats does not received type param");
+        croak ("UpdateStats did not received type param");
     }
     if (grep ($_ eq $params->{type}, @allowed_circulation_types  )) {
         @mandatory_keys = @circulation_mandatory_keys;
@@ -105,13 +105,13 @@ sub UpdateStats {
     }
     my @missing_params = ();
     for my $mykey (@mandatory_keys ) {
-        push @missing_params, $mykey if !grep (/^$mykey/, keys $params);
+        push @missing_params, $mykey if !grep (/^$mykey/, keys %$params);
     }
     if (scalar @missing_params > 0 ) {
-        croak ("UpdateStats does not received mandatory param(s): ".join (", ",@missing_params ));
+        croak ("UpdateStats did not received mandatory param(s): ".join (", ",@missing_params ));
     }
     my @invalid_params = ();
-    for my $myparam (keys $params ) {
+    for my $myparam (keys %$params ) {
         push @invalid_params, $myparam unless grep (/^$myparam$/, @allowed_keys);
     }
     if (scalar @invalid_params > 0 ) {