Bug 14782: Add possibility to search patrons using the username (userid)

[koha_ffzg] / svc / members / search

diff --git a/svc/members/search b/svc/members/search
index 1b3a02a..1c2d644 100755 (executable)
--- a/svc/members/search
+++ b/svc/members/search
@@ -4,23 +4,23 @@
 #
 # 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 Modern::Perl;
 use CGI;
 
-use C4::Auth qw( get_template_and_user );
+use C4::Auth qw( get_template_and_user haspermission get_user_subpermissions );
 use C4::Output qw( output_with_http_headers );
 use C4::Utils::DataTables qw( dt_get_params );
 use C4::Utils::DataTables::Members qw( search );
@@ -43,7 +43,9 @@ my $firstletter  = $input->param('firstletter');
 my $categorycode = $input->param('categorycode');
 my $branchcode = $input->param('branchcode');
 my $searchtype = $input->param('searchtype');
-my $searchfieldstype = $input->param('searchfieldstype');
+my $searchfieldstype = $input->param('searchfieldstype') || 'standard';
+my $has_permission = $input->param('has_permission');
+my $selection_type = $input->param('selection_type');
 
 if ( $searchfieldstype eq "dateofbirth" ) {
     $searchmember = output_pref({dt => dt_from_string($searchmember), dateformat => 'iso', dateonly => 1});
@@ -57,8 +59,24 @@ foreach (grep {$_ =~ /^mDataProp/} keys %dt_params) {
     $dt_params{$_} =~ s/^dt_//;
 }
 
+my $results;
+# If the user filled a term, maybe it's a cardnumber.
+# This cannot be the case if a first letter is given.
+if ( $searchmember
+    and not $firstletter
+    and $searchfieldstype
+    and $searchfieldstype eq 'standard' )
+{
+    my $member = C4::Members::GetMember( cardnumber => $searchmember );
+    $results = {
+        iTotalRecords        => 1,
+        iTotalDisplayRecords => 1,
+        patrons              => [ $member ],
+    } if $member;
+}
+
 # Perform the patrons search
-my $results = C4::Utils::DataTables::Members::search(
+$results = C4::Utils::DataTables::Members::search(
     {
         searchmember => $searchmember,
         firstletter => $firstletter,
@@ -67,15 +85,39 @@ my $results = C4::Utils::DataTables::Members::search(
         searchtype => $searchtype,
         searchfieldstype => $searchfieldstype,
         dt_params => \%dt_params,
-
     }
-);
+) unless $results;
+
+# It is not recommanded to use the has_permission param if you use the pagination
+# The filter is done AFTER requested the data
+if ($has_permission) {
+    my ( $permission, $subpermission ) = split /\./, $has_permission;
+    my @patrons_with_permission;
+    for my $patron ( @{ $results->{patrons} } ) {
+        my $perms = haspermission( $patron->{userid} );
+        if (   $perms->{superlibrarian} == 1
+            or $perms->{$permission} == 1 )
+        {
+            push @patrons_with_permission, $patron;
+            next;
+        }
+
+        if ($subpermission) {
+            my $subperms = get_user_subpermissions( $patron->{userid} );
+            push @patrons_with_permission, $patron
+              if $subperms->{$permission}->{$subpermission};
+        }
+    }
+    $results->{patrons} = \@patrons_with_permission;
+    $results->{iTotalDisplayRecords} = scalar( @patrons_with_permission );
+}
 
 $template->param(
     sEcho => $sEcho,
     iTotalRecords => $results->{iTotalRecords},
     iTotalDisplayRecords => $results->{iTotalDisplayRecords},
-    aaData => $results->{patrons}
+    aaData => $results->{patrons},
+    selection_type => $selection_type,
 );
 
 output_with_http_headers $input, $cookie, $template->output, 'json';
@@ -98,7 +140,7 @@ All following params are optional:
     firstletter => search patrons with surname begins with this pattern (currently only used for 1 letter)
     categorycode and branchcode => search patrons belong to a given categorycode or a branchcode
     searchtype: can be 'contain' or 'start_with'
-    searchfieldstype: Can be 'standard', 'email', 'borrowernumber', 'phone' or 'address'
+    searchfieldstype: Can be 'standard', 'email', 'borrowernumber', 'userid', 'phone' or 'address'
 
 =cut