Bug 10855: Additional marc fields are not inserted
[koha.git] / serials / subscription-bib-search.pl
index 78cbc4d..a74afb0 100755 (executable)
@@ -2,22 +2,22 @@
 # WARNING: 4-character tab stops here
 
 # Copyright 2000-2002 Katipo Communications
+# Parts Copyright 2010 Biblibre
 #
 # 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., 59 Temple Place,
-# Suite 330, Boston, MA  02111-1307 USA
-
+# You should have received a copy of the GNU General Public License
+# along with Koha; if not, see <http://www.gnu.org/licenses>.
 
 =head1 NAME
 
@@ -46,154 +46,189 @@ to multipage gestion.
 
 =cut
 
-
 use strict;
-require Exporter;
-use CGI;
+use warnings;
+
+use CGI qw ( -utf8 );
 use C4::Koha;
 use C4::Auth;
-use HTML::Template;
 use C4::Context;
 use C4::Output;
-use C4::Interface::CGI::Output;
-use C4::SearchMarc;
+use C4::Search;
+use C4::Biblio;
+use C4::Debug;
 
-my $query=new CGI;
-# my $type=$query->param('type');
-my $op = $query->param('op');
+my $input = new CGI;
+my $op = $input->param('op') || q{};
 my $dbh = C4::Context->dbh;
 
-my $startfrom=$query->param('startfrom');
-$startfrom=0 unless $startfrom;
-my ($template, $loggedinuser, $cookie);
+my $startfrom = $input->param('startfrom');
+$startfrom = 0 unless $startfrom;
+my ( $template, $loggedinuser, $cookie );
 my $resultsperpage;
 
-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');
-
-    $resultsperpage= $query->param('resultsperpage');
-    $resultsperpage = 19 if(!defined $resultsperpage);
-    my $orderby = $query->param('orderby');
-
-    # builds tag and subfield arrays
-    my @tags;
-
-    foreach my $marc (@marclist) {
-        if ($marc) {
-            my ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,$marc);
-            if ($tag) {
-                push @tags,$dbh->quote("$tag$subfield");
-            } else {
-                push @tags, $dbh->quote(substr($marc,0,4));
-            }
-        } else {
-            push @tags, "";
+my $itype_or_itemtype =
+  ( C4::Context->preference("item-level_itypes") ) ? 'itype' : 'itemtype';
+
+my $query = $input->param('q');
+
+# don't run the search if no search term !
+if ( $op eq "do_search" && $query ) {
+
+    ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+        {
+            template_name   => "serials/result.tt",
+            query           => $input,
+            type            => "intranet",
+            authnotrequired => 0,
+            flagsrequired   => { catalogue => 1, serials => '*' },
+            debug           => 1,
         }
-    }
-    my ($results,$total) = catalogsearch($dbh, \@tags,\@and_or,
-                               \@excluding, \@operator, \@value,
-                               $startfrom*$resultsperpage, $resultsperpage,$orderby);
-
-    ($template, $loggedinuser, $cookie)
-        = get_template_and_user({template_name => "serials/result.tmpl",
-                query => $query,
-                type => "intranet",
-                authnotrequired => 0,
-                flagsrequired => {borrowers => 1},
-                flagsrequired => {catalogue => 1},
-                debug => 1,
-                });
+    );
 
-    # multi page display gestion
-    my $displaynext=0;
-    my $displayprev=$startfrom;
-    if(($total - (($startfrom+1)*($resultsperpage))) > 0 ){
-        $displaynext = 1;
+    # add the limits if applicable
+    my $itemtypelimit = $input->param('itemtypelimit');
+    my $ccodelimit    = $input->param('ccodelimit');
+    my $op = C4::Context->preference('UseQueryParser') ? '&&' : 'and';
+    $query .= " $op $itype_or_itemtype:$itemtypelimit" if $itemtypelimit;
+    $query .= " $op ccode:$ccodelimit" if $ccodelimit;
+    $debug && warn $query;
+    $resultsperpage = $input->param('resultsperpage');
+    $resultsperpage = 20 if ( !defined $resultsperpage );
+
+    my ( $error, $marcrecords, $total_hits ) =
+      SimpleSearch( $query, $startfrom * $resultsperpage, $resultsperpage );
+    my $total = 0;
+    if ( defined $marcrecords ) {
+        $total = scalar @{$marcrecords};
     }
 
-    my @field_data = ();
-
+    if ( defined $error ) {
+        $template->param( query_error => $error );
+        warn "error: " . $error;
+        output_html_with_http_headers $input, $cookie, $template->output;
+        exit;
+    }
+    my @results;
+
+    for ( my $i = 0 ; $i < $total ; $i++ ) {
+        my %resultsloop;
+        my $marcrecord = C4::Search::new_record_from_zebra( 'biblioserver', $marcrecords->[$i] );
+        my $biblio = TransformMarcToKoha( C4::Context->dbh, $marcrecord, '' );
+
+        #build the hash for the template.
+        $resultsloop{highlight}       = ( $i % 2 ) ? (1) : (0);
+        $resultsloop{title}           = $biblio->{'title'};
+        $resultsloop{subtitle}        = $biblio->{'subtitle'};
+        $resultsloop{biblionumber}    = $biblio->{'biblionumber'};
+        $resultsloop{author}          = $biblio->{'author'};
+        $resultsloop{publishercode}   = $biblio->{'publishercode'};
+        $resultsloop{publicationyear} = $biblio->{'publicationyear'};
+        $resultsloop{issn}            = $biblio->{'issn'};
+
+        push @results, \%resultsloop;
+    }
 
-    for(my $i = 0 ; $i <= $#marclist ; $i++)
-    {
-        push @field_data, { term => "marclist", val=>$marclist[$i] };
-        push @field_data, { term => "and_or", val=>$and_or[$i] };
-        push @field_data, { term => "excluding", val=>$excluding[$i] };
-        push @field_data, { term => "operator", val=>$operator[$i] };
-        push @field_data, { term => "value", val=>$value[$i] };
+    # multi page display gestion
+    my $displaynext = 0;
+    my $displayprev = $startfrom;
+    if ( ( $total_hits - ( ( $startfrom + 1 ) * ($resultsperpage) ) ) > 0 ) {
+        $displaynext = 1;
     }
 
     my @numbers = ();
 
-    if ($total>$resultsperpage)
-    {
-        for (my $i=1; $i<$total/$resultsperpage+1; $i++)
-        {
-            if ($i<16)
-            {
-                my $highlight=0;
-                ($startfrom==($i-1)) && ($highlight=1);
-                push @numbers, { number => $i,
-                    highlight => $highlight ,
-                    searchdata=> \@field_data,
-                    startfrom => ($i-1)};
+    if ( $total_hits > $resultsperpage ) {
+        for ( my $i = 1 ; $i < $total / $resultsperpage + 1 ; $i++ ) {
+            if ( $i < 16 ) {
+                my $highlight = 0;
+                ( $startfrom == ( $i - 1 ) ) && ( $highlight = 1 );
+                push @numbers,
+                  {
+                    number     => $i,
+                    highlight  => $highlight,
+                    searchdata => \@results,
+                    startfrom  => ( $i - 1 )
+                  };
             }
         }
     }
 
-    my $from = $startfrom*$resultsperpage+1;
+    my $from = 0;
+    $from = $startfrom * $resultsperpage + 1 if ( $total_hits > 0 );
     my $to;
 
-    if($total < (($startfrom+1)*$resultsperpage))
-    {
+    if ( $total_hits < ( ( $startfrom + 1 ) * $resultsperpage ) ) {
         $to = $total;
-    } else {
-        $to = (($startfrom+1)*$resultsperpage);
     }
-    $template->param(result => $results,
-                            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,
-                            );
-} else {
-    ($template, $loggedinuser, $cookie)
-        = get_template_and_user({template_name => "serials/subscription-bib-search.tmpl",
-                query => $query,
-                type => "intranet",
-                authnotrequired => 0,
-                flagsrequired => {catalogue => 1},
-                debug => 1,
-                });
-
-    my  %itemtypes = GetItemTypes();
-    my @values = values %itemtypes;
-    my $CGIitemtype=CGI::scrolling_list(
-            -name     => 'value',
-            -values   => \@values,
-            -labels   => \%itemtypes,
-            -size     => 1,
-            -multiple => 0
+    else {
+        $to = ( ( $startfrom + 1 ) * $resultsperpage );
+    }
+    $template->param(
+        query          => $query,
+        resultsloop    => \@results,
+        startfrom      => $startfrom,
+        displaynext    => $displaynext,
+        displayprev    => $displayprev,
+        resultsperpage => $resultsperpage,
+        startfromnext  => $startfrom + 1,
+        startfromprev  => $startfrom - 1,
+        total          => $total_hits,
+        from           => $from,
+        to             => $to,
+        numbers        => \@numbers,
+    );
+}    # end of if ($op eq "do_search" & $query)
+else {
+    ( $template, $loggedinuser, $cookie ) = get_template_and_user(
+        {
+            template_name   => "serials/subscription-bib-search.tt",
+            query           => $input,
+            type            => "intranet",
+            authnotrequired => 0,
+            flagsrequired   => { catalogue => 1, serials => '*' },
+            debug           => 1,
+        }
     );
 
+    # load the itemtypes
+    my $itemtypes = GetItemTypes();
+    my @itemtypesloop;
+    foreach my $thisitemtype (
+        sort {
+            $itemtypes->{$a}->{'description'}
+              cmp $itemtypes->{$b}->{'description'}
+        } keys %$itemtypes
+      )
+    {
+        my %row = (
+            code        => $thisitemtype,
+            description => $itemtypes->{$thisitemtype}->{'description'},
+        );
+        push @itemtypesloop, \%row;
+    }
+
+    # load Collection Codes
+    my $authvalues = GetAuthorisedValues('CCODE');
+    my @ccodesloop;
+    for my $thisauthvalue ( sort { $a->{'lib'} cmp $b->{'lib'} } @$authvalues )
+    {
+        my %row = (
+            code        => $thisauthvalue->{'authorised_value'},
+            description => $thisauthvalue->{'lib'},
+        );
+        push @ccodesloop, \%row;
+    }
+
     $template->param(
-            CGIitemtype => $CGIitemtype,
+        itemtypeloop => \@itemtypesloop,
+        ccodeloop    => \@ccodesloop,
+        no_query     => $op eq "do_search" ? 1 : 0,
     );
 }
 
 # Print the page
-output_html_with_http_headers $query, $cookie, $template->output;
+output_html_with_http_headers $input, $cookie, $template->output;
 
 # Local Variables:
 # tab-width: 4