Bug 17380: Do not use GuessAuthTypeCode in MetadataRecord::Authority
[koha.git] / authorities / auth_finder.pl
index 99ac2c1..b484f03 100755 (executable)
@@ -5,85 +5,75 @@
 #
 # 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;
 
-use CGI;
+use CGI qw ( -utf8 );
 use C4::Output;
 use C4::Auth;
 use C4::Context;
 use C4::AuthoritiesMarc;
 use C4::Acquisition;
-use C4::Koha;    # XXX subfield_is_koha_internal_p
+use C4::Koha;
+
+use Koha::Authority::Types;
 
 my $query        = new CGI;
-my $op           = $query->param('op');
-my $authtypecode = $query->param('authtypecode');
-my $index        = $query->param('index');
-my $tagid        = $query->param('tagid');
-my $resultstring = $query->param('result');
-my $dbh          = C4::Context->dbh;
-
-my $startfrom = $query->param('startfrom');
-$startfrom = 0 if ( !defined $startfrom );
-my ( $template, $loggedinuser, $cookie );
-my $resultsperpage;
-
-my $authtypes = getauthtypes;
-my @authtypesloop;
-foreach my $thisauthtype ( keys %$authtypes ) {
-    my %row = (
-        value        => $thisauthtype,
-        selected     => ($thisauthtype eq $authtypecode),
-        authtypetext => $authtypes->{$thisauthtype}{'authtypetext'},
-        index        => $index,
-    );
-    push @authtypesloop, \%row;
-}
+my $op           = $query->param('op') || '';
+my $authtypecode = $query->param('authtypecode') || '';
+my $index        = $query->param('index') || '';
+my $tagid        = $query->param('tagid') || '';
+my $source       = $query->param('source') || '';
+my $relationship = $query->param('relationship') || '';
+
+my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+    {
+        template_name => ( $op eq 'do_search' )
+        ? 'authorities/searchresultlist-auth.tt'
+        : 'authorities/auth_finder.tt',
+        query           => $query,
+        type            => 'intranet',
+        authnotrequired => 0,
+        flagsrequired   => { catalogue => 1 },
+    }
+);
 
-$op ||= q{};
-if ( $op eq "do_search" ) {
-    my @marclist  = $query->param('marclist');
-    my @and_or    = $query->param('and_or');
-    my @excluding = $query->param('excluding');
-    my @operator  = $query->param('operator');
-    my @value     = ($query->param('value_mainstr')||undef, $query->param('value_main')||undef, $query->param('value_any')||undef);
-    my $orderby   = $query->param('orderby');
+my $authority_types = Koha::Authority::Types->search( {}, { order_by => ['authtypetext'] } );
 
-    $resultsperpage = $query->param('resultsperpage');
-    $resultsperpage = 20 if ( !defined $resultsperpage );
+# If search form posted
+if ( $op eq "do_search" ) {
+    my @marclist  = $query->multi_param('marclist');
+    my @and_or    = $query->multi_param('and_or');
+    my @excluding = $query->multi_param('excluding');
+    my @operator  = $query->multi_param('operator');
+    my @value     = (
+        $query->param('value_mainstr') || undef,
+        $query->param('value_main')    || undef,
+        $query->param('value_any')     || undef,
+        $query->param('value_match')   || undef
+    );
+    my $orderby        = $query->param('orderby')        || '';
+    my $startfrom      = $query->param('startfrom')      || 0;
+    my $resultsperpage = $query->param('resultsperpage') || 20;
 
     my ( $results, $total ) =
       SearchAuthorities( \@marclist, \@and_or, \@excluding, \@operator, \@value,
         $startfrom * $resultsperpage,
-        $resultsperpage, $authtypecode, $orderby);
-
-    # If an authority heading is repeated, add an arrayref to those repetions
-    # First heading -- Second heading
-    for my $heading ( @$results ) {
-        my @repets = split / -- /, $heading->{summary};
-        if ( @repets > 1 ) {
-            my @repets_loop;
-            for (my $i = 0; $i < @repets; $i++) {
-                push @repets_loop,
-                    { index => $index, repet => $i+1, value => $repets[$i] };
-            }
-            $heading->{repets} = \@repets_loop;
-        }
-    }
+        $resultsperpage, $authtypecode, $orderby );
+
     # multi page display gestion
     my $displaynext = 0;
     my $displayprev = $startfrom;
@@ -93,9 +83,8 @@ if ( $op eq "do_search" ) {
 
     my @field_data = ();
 
-    my @marclist_ini =
-      $query->param('marclist')
-      ; # get marclist again, as the previous one has been modified by catalogsearch (mainentry replaced by field name
+# get marclist again, as the previous one has been modified by catalogsearch (mainentry replaced by field name)
+    my @marclist_ini = $query->multi_param('marclist');
     for ( my $i = 0 ; $i <= $#marclist ; $i++ ) {
         push @field_data, { term => "marclist",  val => $marclist_ini[$i] };
         push @field_data, { term => "and_or",    val => $and_or[$i] };
@@ -103,12 +92,16 @@ if ( $op eq "do_search" ) {
         push @field_data, { term => "operator",  val => $operator[$i] };
     }
 
-    push @field_data, { term => "value_mainstr", val => $query->param('value_mainstr') || "" };
-    push @field_data, { term => "value_main",    val => $query->param('value_main')    || "" };
-    push @field_data, { term => "value_any",     val => $query->param('value_any')     || ""};
+    push @field_data,
+      { term => "value_mainstr", val => scalar $query->param('value_mainstr') || "" };
+    push @field_data,
+      { term => "value_main", val => scalar $query->param('value_main') || "" };
+    push @field_data,
+      { term => "value_any", val => scalar $query->param('value_any') || "" };
+    push @field_data,
+      { term => "value_match", val => scalar $query->param('value_match') || "" };
 
     my @numbers = ();
-
     if ( $total > $resultsperpage ) {
         for ( my $i = 1 ; $i < $total / $resultsperpage + 1 ; $i++ ) {
             if ( $i < 16 ) {
@@ -127,69 +120,54 @@ if ( $op eq "do_search" ) {
 
     my $from = $startfrom * $resultsperpage + 1;
     my $to;
-
     if ( $total < ( ( $startfrom + 1 ) * $resultsperpage ) ) {
         $to = $total;
     }
     else {
         $to = ( ( $startfrom + 1 ) * $resultsperpage );
     }
-    ( $template, $loggedinuser, $cookie ) = get_template_and_user(
-        {
-            template_name   => "authorities/searchresultlist-auth.tmpl",
-            query           => $query,
-            type            => 'intranet',
-            authnotrequired => 0,
-            flagsrequired   => { catalogue => 1 },
-        }
-    );
 
     $template->param( result => $results ) if $results;
     $template->param(
-        orderby      => $orderby,
-        startfrom      => $startfrom,
-    displaynext    => $displaynext,
-    displayprev    => $displayprev,
-    resultsperpage => $resultsperpage,
-    startfromnext  => $startfrom + 1,
-    startfromprev  => $startfrom - 1,
-        searchdata     => \@field_data,
-        total          => $total,
-        from           => $from,
-        to             => $to,
-        numbers        => \@numbers,
-        authtypecode   => $authtypecode,
-        value_mainstr  => $query->param('value_mainstr') || "", 
-        value_main     => $query->param('value_main') || "",
-        value_any      => $query->param('value_any') || "",
-    );
-} else {
-    ( $template, $loggedinuser, $cookie ) = get_template_and_user(
-        {
-            template_name   => "authorities/auth_finder.tmpl",
-            query           => $query,
-            type            => 'intranet',
-            authnotrequired => 0,
-            flagsrequired   => { catalogue => 1 },
-        }
+        orderby          => $orderby,
+        startfrom        => $startfrom,
+        displaynext      => $displaynext,
+        displayprev      => $displayprev,
+        resultsperpage   => $resultsperpage,
+        startfromnext    => $startfrom + 1,
+        startfromprev    => $startfrom - 1,
+        searchdata       => \@field_data,
+        total            => $total,
+        from             => $from,
+        to               => $to,
+        numbers          => \@numbers,
+        operator_mainstr => ( @operator > 0 && $operator[0] )
+        ? $operator[0]
+        : '',
+        operator_main  => ( @operator > 1 && $operator[1] ) ? $operator[1] : '',
+        operator_any   => ( @operator > 2 && $operator[2] ) ? $operator[2] : '',
+        operator_match => ( @operator > 3 && $operator[3] ) ? $operator[3] : '',
     );
+}
+else {
 
-    $template->param(
-        resultstring => $resultstring,
-    );
+    # special case for UNIMARC field 210c builder
+    my $resultstring = $query->param('result') || '';
+    $template->param( resultstring => $resultstring, );
 }
 
 $template->param(
-    value_mainstr => $query->param('value_mainstr') || "", 
-    value_main    => $query->param('value_main') || "",
-    value_any     => $query->param('value_any') || "",
+    op            => $op,
+    value_mainstr => scalar $query->param('value_mainstr') || '',
+    value_main    => scalar $query->param('value_main') || '',
+    value_any     => scalar $query->param('value_any') || '',
+    value_match   => scalar $query->param('value_match') || '',
     tagid         => $tagid,
     index         => $index,
-    authtypesloop => \@authtypesloop,
+    authority_types  => $authority_types,
     authtypecode  => $authtypecode,
-    value_mainstr  => $query->param('value_mainstr') || "", 
-    value_main     => $query->param('value_main')    || "",
-    value_any      => $query->param('value_any')     || "",
+    source        => $source,
+    relationship  => $relationship,
 );
 
 # Print the page