# Script to perform searching
# For documentation try 'perldoc /path/to/search'
#
-# $Header$
-#
# Copyright 2006 LibLime
#
# This file is part of Koha
=head1 NAME
-search - a search script for finding records in a Koha system (Version 2.4)
+search - a search script for finding records in a Koha system (Version 3)
=head1 OVERVIEW
-This script contains a demonstration of a new search API for Koha 2.4. It is
-designed to be simple to use and configure, yet capable of performing feats
-like stemming, field weighting, relevance ranking, support for multiple
-query language formats (CCL, CQL, PQF), full or nearly full support for the
-bib1 attribute set, extended attribute sets defined in Zebra profiles, access
-to the full range of Z39.50 query options, federated searches on Z39.50
-targets, etc.
+This script utilizes a new search API for Koha 3. It is designed to be
+simple to use and configure, yet capable of performing feats like stemming,
+field weighting, relevance ranking, support for multiple query language
+formats (CCL, CQL, PQF), full support for the bib1 attribute set, extended
+attribute sets defined in Zebra profiles, access to the full range of Z39.50
+and SRU query options, federated searches on Z39.50/SRU targets, etc.
-I believe the API as represented in this script is mostly sound, even if the
-individual functions in Search.pm and Koha.pm need to be cleaned up. Of course,
-you are free to disagree :-)
+The API as represented in this script is mostly sound, even if the individual
+functions in Search.pm and Koha.pm need to be cleaned up. Of course, you are
+free to disagree :-)
I will attempt to describe what is happening at each part of this script.
--- JF
+-- Joshua Ferraro <jmf AT liblime DOT com>
=head2 INTRO
Once determined, proceed to only load the variables and procedures necessary
for that function.
-=head2 THE ADVANCED SEARCH PAGE
-
-If we're loading the advanced search page this script will call a number of
-display* routines which populate objects that are sent to the template for
-display of things like search indexes, languages, search limits, branches,
-etc. These are not stored in the template for two reasons:
-
-=over
-
-=item 1. Efficiency - we have more control over objects inside the script, and it's possible to not duplicate things like indexes (if the search indexes were stored in the template they would need to be repeated)
-
-=item 2. Customization - if these elements were moved to the sql database it would allow a simple librarian to determine which fields to display on the page without editing any html (also how the fields should behave when being searched).
+=head2 LOADING ADVANCED SEARCH PAGE
-=back
-
-However, they create one problem : the strings aren't translated. I have an idea
-for how to do this that I will purusue soon.
+This is fairly straightforward, and I won't go into detail ;-)
=head2 PERFORMING A SEARCH
=over
-=item 1 Koha query - the query that is passed to Zebra
-
-This is the most complex query that needs to be built.The original design goal was to use a custom CCL2PQF query parser to translate an incoming CCL query into a multi-leaf query to pass to Zebra. It needs to be multi-leaf to allow field weighting, koha-specific relevance ranking, and stemming. When I have a chance I'll try to flesh out this section to better explain.
+=item 1 $query - the fully-built query passed to zebra
-This query incorporates query profiles that aren't compatible with non-Zebra Z39.50 targets to acomplish the field weighting and relevance ranking.
+This is the most complex query that needs to be built. The original design goal
+was to use a custom CCL2PQF query parser to translate an incoming CCL query into
+a multi-leaf query to pass to Zebra. It needs to be multi-leaf to allow field
+weighting, koha-specific relevance ranking, and stemming. When I have a chance
+I'll try to flesh out this section to better explain.
-=item 2 Federated query - the query that is passed to other Z39.50 targets
+This query incorporates query profiles that aren't compatible with most non-Zebra
+Z39.50 targets to acomplish the field weighting and relevance ranking.
-This query is just the user's query expressed in CCL CQL, or PQF for passing to a non-zebra Z39.50 target (one that doesn't support the extended profile that Zebra does).
+=item 2 $simple_query - a simple query that doesn't contain the field weighting,
+stemming, etc., suitable to pass off to other search targets
-=item 3 Search description - passed to the template / saved for future refinements of the query (by user)
+This query is just the user's query expressed in CCL CQL, or PQF for passing to a
+non-zebra Z39.50 target (one that doesn't support the extended profile that Zebra does).
-This is a simple string that completely expresses the query in a way that can be parsed by Koha for future refinements of the query or as a part of a history feature. It differs from the human search description in several ways:
+=item 3 $query_cgi - passed to the template / saved for future refinements of
+the query (by user)
-1. it does not contain commas or = signs
-2.
+This is a simple string that completely expresses the query as a CGI string that
+can be used for future refinements of the query or as a part of a history feature.
-=item 4 Human search description - what the user sees in the search_desc area
+=item 4 $query_desc - Human search description - what the user sees in search
+feedback area
-This is a simple string nearly identical to the Search description, but more human readable. It will contain = signs or commas, etc.
+This is a simple string that is human readable. It will contain '=', ',', etc.
=back
=head3 2. Perform the Search
-This section takes the query strings and performs searches on the named servers, including the Koha Zebra server, stores the results in a deeply nested object, builds 'faceted results', and returns these objects.
+This section takes the query strings and performs searches on the named servers,
+including the Koha Zebra server, stores the results in a deeply nested object,
+builds 'faceted results', and returns these objects.
=head3 3. Build HTML
-The final major section of this script takes the objects collected thusfar and builds the HTML for output to the template and user.
+The final major section of this script takes the objects collected thusfar and
+builds the HTML for output to the template and user.
=head3 Additional Notes
## STEP 1. Load things that are used in both search page and
# results page and decide which template to load, operations
# to perform, etc.
+
## load Koha modules
use C4::Context;
use C4::Output;
use C4::Auth;
use C4::Search;
-use C4::Languages; # getAllLanguages
+use C4::Languages qw(getAllLanguages);
use C4::Koha;
use POSIX qw(ceil floor);
use C4::Branch; # GetBranches
+
# create a new CGI object
-# not sure undef_params option is working, need to test
+# FIXME: no_undef_params needs to be tested
use CGI qw('-no_undef_params');
my $cgi = new CGI;
# decide which template to use
my $template_name;
+my $template_type;
my @params = $cgi->param("limit");
-if ((@params>1) || ($cgi->param("q")) ) {
+if ((@params>=1) || ($cgi->param("q")) || ($cgi->param('multibranchlimit')) || ($cgi->param('limit-yr')) ) {
$template_name = 'catalogue/results.tmpl';
}
else {
$template_name = 'catalogue/advsearch.tmpl';
+ $template_type = 'advsearch';
}
-
# load the template
($template, $borrowernumber, $cookie) = get_template_and_user({
template_name => $template_name,
flagsrequired => { catalogue => 1 },
}
);
-
-=head1 BUGS and FIXMEs
-
-There are many, most are documented in the code. The one that
-isn't fully documented, but referred to is the need for a full
-query parser.
-
-=cut
+if (C4::Context->preference("marcflavour") eq "UNIMARC" ) {
+ $template->param('UNIMARC' => 1);
+}
## URI Re-Writing
-# FIXME: URI re-writing should be tested more carefully and may better
-# handled by mod_rewrite or something else. The code below almost works,
-# but doesn't quite handle limits correctly when they are the only
-# param passed -- I'll work on this soon -- JF
+# Deprecated, but preserved because it's interesting :-)
+# The same thing can be accomplished with mod_rewrite in
+# a more elegant way
+#
#my $rewrite_flag;
#my $uri = $cgi->url(-base => 1);
#my $relative_url = $cgi->url(-relative=>1);
#warn "URI:$uri";
#my @cgi_params_list = $cgi->param();
#my $url_params = $cgi->Vars;
-
+#
#for my $each_param_set (@cgi_params_list) {
# $uri.= join "", map "\&$each_param_set=".$_, split("\0",$url_params->{$each_param_set}) if $url_params->{$each_param_set};
#}
# load the branches
my $branches = GetBranches();
my @branch_loop;
-push @branch_loop, {value => "", branchname => "All Branches", };
-for my $branch_hash (keys %$branches) {
- push @branch_loop, {value => "branch: $branch_hash", branchname => $branches->{$branch_hash}->{'branchname'}, };
+
+# we need to know the borrower branch code to set a default branch
+my $borrowerbranchcode = C4::Context->userenv->{'branch'};
+
+for my $branch_hash (sort { $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname} } keys %$branches) {
+ # if independantbranches is activated, set the default branch to the borrower branch
+ my $selected = (C4::Context->preference("independantbranches") and ($borrowerbranchcode eq $branch_hash)) ? 1 : undef;
+ push @branch_loop, {value => "$branch_hash" , branchname => $branches->{$branch_hash}->{'branchname'}, selected => $selected};
}
-$template->param(branchloop => \@branch_loop,);
-# load the itemtypes (Called Collection Codes in the template -- used for circ rules )
+my $categories = GetBranchCategories(undef,'searchdomain');
+
+$template->param(branchloop => \@branch_loop, searchdomainloop => $categories);
+
+# load the Type stuff
+# load the Type stuff
my $itemtypes = GetItemTypes;
+# the index parameter is different for item-level itemtypes
+my $itype_or_itemtype = (C4::Context->preference("item-level_itypes"))?'itype':'itemtype';
my @itemtypesloop;
my $selected=1;
my $cnt;
-my $imgdir = getitemtypeimagesrc();
-foreach my $thisitemtype ( sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'} } keys %$itemtypes ) {
+my $advanced_search_types = C4::Context->preference("AdvancedSearchTypes");
+
+if (!$advanced_search_types or $advanced_search_types eq 'itemtypes') { foreach my $thisitemtype ( sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'} } keys %$itemtypes ) {
my %row =( number=>$cnt++,
- imageurl=> $itemtypes->{$thisitemtype}->{'imageurl'}?($imgdir."/".$itemtypes->{$thisitemtype}->{'imageurl'}):"",
+ ccl => qq($itype_or_itemtype,phr),
code => $thisitemtype,
selected => $selected,
description => $itemtypes->{$thisitemtype}->{'description'},
count5 => $cnt % 4,
+ imageurl=> getitemtypeimagelocation( 'intranet', $itemtypes->{$thisitemtype}->{'imageurl'} ),
+ );
+ $selected = 0 if ($selected) ;
+ push @itemtypesloop, \%row;
+ }
+ $template->param(itemtypeloop => \@itemtypesloop);
+} else {
+ my $advsearchtypes = GetAuthorisedValues($advanced_search_types);
+ for my $thisitemtype (sort {$a->{'lib'} cmp $b->{'lib'}} @$advsearchtypes) {
+ my %row =(
+ number=>$cnt++,
+ ccl => $advanced_search_types,
+ code => $thisitemtype->{authorised_value},
+ selected => $selected,
+ description => $thisitemtype->{'lib'},
+ count5 => $cnt % 4,
+ imageurl=> getitemtypeimagelocation( 'intranet', $thisitemtype->{'imageurl'} ),
);
- $selected = 0 if ($selected) ;
- push @itemtypesloop, \%row;
+ push @itemtypesloop, \%row;
+ }
+ $template->param(itemtypeloop => \@itemtypesloop);
}
-$template->param(itemtypeloop => \@itemtypesloop);
-
-# # load the itypes (Called item types in the template -- just authorized values for searching)
-# my ($itypecount,@itype_loop) = GetCcodes();
-# $template->param(itypeloop=>\@itype_loop,);
-
-# load the languages ( for switching from one template to another )
-# my @languages_options = displayLanguages($cgi);
-# my $languages_count = @languages_options;
-# if($languages_count > 1){
-# $template->param(languages => \@languages_options);
-# }
# The following should only be loaded if we're bringing up the advanced search template
-if ( $template_name eq "catalogue/advsearch.tmpl" ) {
+if ( $template_type eq 'advsearch' ) {
+
# load the servers (used for searching -- to do federated searching, etc.)
my $primary_servers_loop;# = displayPrimaryServers();
$template->param(outer_servers_loop => $primary_servers_loop,);
my $secondary_servers_loop;# = displaySecondaryServers();
$template->param(outer_sup_servers_loop => $secondary_servers_loop,);
-
+
+ # set the default sorting
+ my $default_sort_by = C4::Context->preference('defaultSortField')."_".C4::Context->preference('defaultSortOrder')
+ if (C4::Context->preference('OPACdefaultSortField') && C4::Context->preference('OPACdefaultSortOrder'));
+ $template->param($default_sort_by => 1);
+
# determine what to display next to the search boxes (ie, boolean option
# shouldn't appear on the first one, scan indexes should, adding a new
# box should only appear on the last, etc.
- # FIXME: this stuff should be cleaned up a bit and the html should be turned
- # into flags for the template -- I'll work on that soon -- JF
my @search_boxes_array;
- my $search_boxes_count = 1; # should be a syspref
- for (my $i=0;$i<=$search_boxes_count;$i++) {
- if ($i==0) {
+ my $search_boxes_count = C4::Context->preference("OPACAdvSearchInputCount") || 3; # FIXME: using OPAC sysprefs?
+ # FIXME: all this junk can be done in TMPL using __first__ and __last__
+ for (my $i=1;$i<=$search_boxes_count;$i++) {
+ # if it's the first one, don't display boolean option, but show scan indexes
+ if ($i==1) {
+ push @search_boxes_array, {scan_index => 1};
+ }
+ # if it's the last one, show the 'add field' box
+ elsif ($i==$search_boxes_count) {
push @search_boxes_array,
{
- search_boxes_label => 1,
- scan_index => 1,
+ boolean => 1,
+ add_field => 1,
};
-
}
- elsif ($i==$search_boxes_count) {
+ else {
push @search_boxes_array,
{
- add_field => "1",};
+ boolean => 1,
+ };
}
}
# load the language limits (for search)
my $languages_limit_loop = getAllLanguages();
$template->param(search_languages_loop => $languages_limit_loop,);
-
- my $expanded_options;
- if (not defined $cgi->param('expanded_options')){
- $expanded_options = C4::Context->preference("expandedSearchOption");
+
+ # use the global setting by default
+ if ( C4::Context->preference("expandedSearchOption") == 1) {
+ $template->param( expanded_options => C4::Context->preference("expandedSearchOption") );
}
- else {
- $expanded_options = $cgi->param('expanded_options');
+ # but let the user override it
+ if ( ($cgi->param('expanded_options') == 0) || ($cgi->param('expanded_options') == 1 ) ) {
+ $template->param( expanded_options => $cgi->param('expanded_options'));
}
- $template->param(expanded_options => $expanded_options);
-
- # load the sort_by options for the template
- my $sort_by = $cgi->param('sort_by');
output_html_with_http_headers $cgi, $cookie, $template->output;
exit;
}
-### OK, if we're this far, we're performing an actual search
+### OK, if we're this far, we're performing a search, not just loading the advanced search page
# Fetch the paramater list as a hash in scalar context:
# * returns paramater list as tied hash ref
# Params that can have more than one value
# sort by is used to sort the query
+# in theory can have more than one but generally there's just one
my @sort_by;
+my $default_sort_by = C4::Context->preference('defaultSortField')."_".C4::Context->preference('defaultSortOrder')
+ if (C4::Context->preference('defaultSortField') && C4::Context->preference('defaultSortOrder'));
+
@sort_by = split("\0",$params->{'sort_by'}) if $params->{'sort_by'};
-#
+$sort_by[0] = $default_sort_by unless $sort_by[0];
+foreach my $sort (@sort_by) {
+ $template->param($sort => 1);
+}
+$template->param('sort_by' => $sort_by[0]);
+
# Use the servers defined, or just search our local catalog(default)
my @servers;
@servers = split("\0",$params->{'server'}) if $params->{'server'};
@servers = ("biblioserver");
# @servers = C4::Context->config("biblioserver");
}
-
# operators include boolean and proximity operators and are used
# to evaluate multiple operands
my @operators;
@operators = split("\0",$params->{'op'}) if $params->{'op'};
# indexes are query qualifiers, like 'title', 'author', etc. They
-# can be simple or complex
+# can be single or multiple parameters separated by comma: kw,right-Truncation
my @indexes;
-@indexes = split("\0",$params->{'idx'}) if $params->{'idx'};
+@indexes = split("\0",$params->{'idx'});
+
+# if a simple index (only one) display the index used in the top search box
+if ($indexes[0] && !$indexes[1]) {
+ $template->param("ms_".$indexes[0] => 1);}
+
# an operand can be a single term, a phrase, or a complete ccl query
my @operands;
my @limits;
@limits = split("\0",$params->{'limit'}) if $params->{'limit'};
+if($params->{'multibranchlimit'}) {
+push @limits, join(" or ", map { "branch: $_ "} @{GetBranchesInCategory($params->{'multibranchlimit'})}) ;
+}
+
my $available;
foreach my $limit(@limits) {
if ($limit =~/available/) {
}
}
$template->param(available => $available);
-push @limits, map "yr:".$_, split("\0",$params->{'limit-yr'}) if $params->{'limit-yr'};
+
+# append year limits if they exist
+my $limit_yr;
+my $limit_yr_value;
+if ($params->{'limit-yr'}) {
+ if ($params->{'limit-yr'} =~ /\d{4}-\d{4}/) {
+ my ($yr1,$yr2) = split(/-/, $params->{'limit-yr'});
+ $limit_yr = "yr,st-numeric,ge=$yr1 and yr,st-numeric,le=$yr2";
+ $limit_yr_value = "$yr1-$yr2";
+ }
+ elsif ($params->{'limit-yr'} =~ /\d{4}/) {
+ $limit_yr = "yr,st-numeric=$params->{'limit-yr'}";
+ $limit_yr_value = $params->{'limit-yr'};
+ }
+ push @limits,$limit_yr;
+ #FIXME: Should return a error to the user, incorect date format specified
+}
+
+# convert indexes and operands to corresponding parameter names for the z3950 search
+# $ %z3950p will be a hash ref if the indexes are present (advacned search), otherwise undef
+my $z3950par;
+my $indexes2z3950 = {
+ kw=>'title', au=>'author', 'au,phr'=>'author', nb=>'isbn', ns=>'issn',
+ 'lcn,phr'=>'dewey', su=>'subject', 'su,phr'=>'subject',
+ ti=>'title', 'ti,phr'=>'title', se=>'title'
+};
+for (my $ii = 0; $ii < @operands; ++$ii)
+{
+ my $name = $indexes2z3950->{$indexes[$ii]};
+ if (defined $name && defined $operands[$ii])
+ {
+ $z3950par ||= {};
+ $z3950par->{$name} = $operands[$ii] if !exists $z3950par->{$name};
+ }
+}
+
# Params that can only have one value
-my $query = $params->{'q'};
my $scan = $params->{'scan'};
-my $results_per_page = $params->{'count'} || 20;
+my $count = C4::Context->preference('numSearchResults') || 20;
+my $results_per_page = $params->{'count'} || $count;
+my $offset = $params->{'offset'} || 0;
my $page = $cgi->param('page') || 1;
-my $offset = ($page-1)*$results_per_page;
+#my $offset = ($page-1)*$results_per_page;
my $hits;
my $expanded_facet = $params->{'expand'};
# Define some global variables
-my $error; # used for error handling
-my $search_desc; # the query expressed in terms that humans understand
-my $koha_query; # the query expressed in terms that zoom understands with field weighting and stemming
-my $federated_query;
-my $query_type; # usually not needed, but can be used to trigger ccl, cql, or pqf queries if set
+my ( $error,$query,$simple_query,$query_cgi,$query_desc,$limit,$limit_cgi,$limit_desc,$stopwords_removed,$query_type);
+
my @results;
+
## I. BUILD THE QUERY
-($error,$search_desc,$koha_query,$federated_query,$query_type) = buildQuery($query,\@operators,\@operands,\@indexes,\@limits);
+( $error,$query,$simple_query,$query_cgi,$query_desc,$limit,$limit_cgi,$limit_desc,$stopwords_removed,$query_type) = buildQuery(\@operators,\@operands,\@indexes,\@limits,\@sort_by,$scan);
+
+## parse the query_cgi string and put it into a form suitable for <input>s
+my @query_inputs;
+my $scan_index_to_use;
+
+for my $this_cgi ( split('&',$query_cgi) ) {
+ next unless $this_cgi;
+ $this_cgi =~ m/(.*=)(.*)/;
+ my $input_name = $1;
+ my $input_value = $2;
+ $input_name =~ s/=$//;
+ push @query_inputs, { input_name => $input_name, input_value => $input_value };
+ if ($input_name eq 'idx') {
+ $scan_index_to_use = $input_value; # unless $scan_index_to_use;
+ }
+}
+$template->param ( QUERY_INPUTS => \@query_inputs,
+ scan_index_to_use => $scan_index_to_use );
+
+## parse the limit_cgi string and put it into a form suitable for <input>s
+my @limit_inputs;
+for my $this_cgi ( split('&',$limit_cgi) ) {
+ next unless $this_cgi;
+ # handle special case limit-yr
+ if ($this_cgi =~ /yr,st-numeric/) {
+ push @limit_inputs, { input_name => 'limit-yr', input_value => $limit_yr_value };
+ next;
+ }
+ $this_cgi =~ m/(.*=)(.*)/;
+ my $input_name = $1;
+ my $input_value = $2;
+ $input_name =~ s/=$//;
+ push @limit_inputs, { input_name => $input_name, input_value => $input_value };
+}
+$template->param ( LIMIT_INPUTS => \@limit_inputs );
## II. DO THE SEARCH AND GET THE RESULTS
my $total; # the total results for the whole set
my $results_hashref;
if (C4::Context->preference('NoZebra')) {
+ $query=~s/yr(:|=)\s*([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
+ $simple_query=~s/yr\s*(:|=)([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
+ # warn $query;
eval {
- ($error, $results_hashref, $facets) = NZgetRecords($koha_query,$federated_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
+ ($error, $results_hashref, $facets) = NZgetRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
};
} else {
eval {
- ($error, $results_hashref, $facets) = getRecords($koha_query,$federated_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
+ ($error, $results_hashref, $facets) = getRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
};
}
if ($@ || $error) {
$template->param(query_error => $error.$@);
-
output_html_with_http_headers $cgi, $cookie, $template->output;
exit;
}
-my $op=$cgi->param("operation");
-if ($op eq "bulkedit"){
- my ($countchanged,$listunchanged)=
- EditBiblios($results_hashref->{'biblioserver'}->{"RECORDS"},
- $params->{"tagsubfield"},
- $params->{"inputvalue"},
- $params->{"targetvalue"},
- $params->{"test"}
- );
- $template->param(bulkeditresults=>1,
- tagsubfield=>$params->{"tagsubfield"},
- inputvalue=>$params->{"inputvalue"},
- targetvalue=>$params->{"targetvalue"},
- countchanged=>$countchanged,
- countunchanged=>scalar(@$listunchanged),
- listunchanged=>$listunchanged);
-}
-if (C4::Context->userenv->{'flags'}==1 ||(C4::Context->userenv->{'flags'} & ( 2**9 ) )){
-#Edit Catalogue Permissions
- $template->param(bulkedit => 1);
- $template->param(tagsubfields=>GetManagedTagSubfields());
-}
# At this point, each server has given us a result set
# now we build that set for template display
my @sup_results_array;
-for (my $i=0;$i<=@servers;$i++) {
+for (my $i=0;$i<@servers;$i++) {
my $server = $servers[$i];
if ($server =~/biblioserver/) { # this is the local bibliographic server
$hits = $results_hashref->{$server}->{"hits"};
my $page = $cgi->param('page') || 0;
- my @newresults = searchResults( $search_desc,$hits,$results_per_page,$offset,@{$results_hashref->{$server}->{"RECORDS"}});
+ my @newresults = searchResults( $query_desc,$hits,$results_per_page,$offset,$scan,@{$results_hashref->{$server}->{"RECORDS"}});
$total = $total + $results_hashref->{$server}->{"hits"};
+ ## If there's just one result, redirect to the detail page
+ if ($total == 1) {
+ my $biblionumber = $newresults[0]->{biblionumber};
+ my $defaultview = C4::Context->preference('IntranetBiblioDefaultView');
+ my $views = { C4::Search::enabled_staff_search_views };
+ if ($defaultview eq 'isbd' && $views->{can_view_ISBD}) {
+ print $cgi->redirect("/cgi-bin/koha/catalogue/ISBDdetail.pl?biblionumber=$biblionumber");
+ } elsif ($defaultview eq 'marc' && $views->{can_view_MARC}) {
+ print $cgi->redirect("/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=$biblionumber");
+ } elsif ($defaultview eq 'labeled_marc' && $views->{can_view_labeledMARC}) {
+ print $cgi->redirect("/cgi-bin/koha/catalogue/labeledMARCdetail.pl?biblionumber=$biblionumber");
+ } else {
+ print $cgi->redirect("/cgi-bin/koha/catalogue/detail.pl?biblionumber=$biblionumber");
+ }
+ exit;
+ }
+
+
if ($hits) {
$template->param(total => $hits);
- $template->param(searchdesc => ($query_type?"$query_type=":"")."$search_desc" );
+ my $limit_cgi_not_availablity = $limit_cgi;
+ $limit_cgi_not_availablity =~ s/&limit=available//g;
+ $template->param(limit_cgi_not_availablity => $limit_cgi_not_availablity);
+ $template->param(limit_cgi => $limit_cgi);
+ $template->param(query_cgi => $query_cgi);
+ $template->param(query_desc => $query_desc);
+ $template->param(limit_desc => $limit_desc);
+ $template->param (z3950_search_params => C4::Search::z3950_search_args($query_desc));
+ if ($query_desc || $limit_desc) {
+ $template->param(searchdesc => 1);
+ }
+ $template->param(stopwords_removed => "@$stopwords_removed") if $stopwords_removed;
$template->param(results_per_page => $results_per_page);
$template->param(SEARCH_RESULTS => \@newresults);
-# my @page_numbers;
-# my $pages = ceil($hits / $results_per_page);
-# my $previous_page_offset = $offset - $results_per_page unless ($offset - $results_per_page <0);
-# my $next_page_offset = $offset + $results_per_page;
-# for (my $j=1; $j<=$pages;$j++) {
-# my $this_offset = (($j*$results_per_page)-$results_per_page);
-# my $this_page_number = $j;
-# my $highlight = 1 if ($this_page_number == $current_page_number);
-# if ($this_page_number <= $pages) {
-# push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight, sort_by => join " ",@sort_by };
-# }
-# }
-# $template->param(PAGE_NUMBERS => \@page_numbers,
-# previous_page_offset => $previous_page_offset,
-# next_page_offset => $next_page_offset) unless $pages < 2;
- my $link="/cgi-bin/koha/catalogue/search.pl?q=$search_desc&";
- foreach my $sort (@sort_by){
- $link.="&sort_by=".$sort."&";
- }
- $template->param(
- pagination_bar => pagination_bar(
- $link,
- getnbpages($hits, $results_per_page),
- $page,
- 'page'
- ),
- );
- }
- } # end of the if local
- else {
- # check if it's a z3950 or opensearch source
- my $zed3950 = 0; # FIXME :: Hardcoded value.
- if ($zed3950) {
- my @inner_sup_results_array;
- for my $sup_record ( @{$results_hashref->{$server}->{"RECORDS"}} ) {
- my $marc_record_object = MARC::Record->new_from_usmarc($sup_record);
- my $control_number = $marc_record_object->field('010')->subfield('a') if $marc_record_object->field('010');
- $control_number =~ s/^ //g;
- my $link = "http://catalog.loc.gov/cgi-bin/Pwebrecon.cgi?SAB1=".$control_number."&BOOL1=all+of+these&FLD1=LC+Control+Number+LCCN+%28K010%29+%28K010%29&GRP1=AND+with+next+set&SAB2=&BOOL2=all+of+these&FLD2=Keyword+Anywhere+%28GKEY%29+%28GKEY%29&PID=6211&SEQ=20060816121838&CNT=25&HIST=1";
- my $title = $marc_record_object->title();
- push @inner_sup_results_array, {
- 'title' => $title,
- 'link' => $link,
- };
+ ## FIXME: add a global function for this, it's better than the current global one
+ ## Build the page numbers on the bottom of the page
+ my @page_numbers;
+ # total number of pages there will be
+ my $pages = ceil($hits / $results_per_page);
+ # default page number
+ my $current_page_number = 1;
+ $current_page_number = ($offset / $results_per_page + 1) if $offset;
+ my $previous_page_offset = $offset - $results_per_page unless ($offset - $results_per_page <0);
+ my $next_page_offset = $offset + $results_per_page;
+ # If we're within the first 10 pages, keep it simple
+ #warn "current page:".$current_page_number;
+ if ($current_page_number < 10) {
+ # just show the first 10 pages
+ # Loop through the pages
+ my $pages_to_show = 10;
+ $pages_to_show = $pages if $pages<10;
+ for (my $i=1; $i<=$pages_to_show;$i++) {
+ # the offset for this page
+ my $this_offset = (($i*$results_per_page)-$results_per_page);
+ # the page number for this page
+ my $this_page_number = $i;
+ # it should only be highlighted if it's the current page
+ my $highlight = 1 if ($this_page_number == $current_page_number);
+ # put it in the array
+ push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight, sort_by => join " ",@sort_by };
+
+ }
+
+ }
+
+ # now, show twenty pages, with the current one smack in the middle
+ else {
+ for (my $i=$current_page_number; $i<=($current_page_number + 20 );$i++) {
+ my $this_offset = ((($i-9)*$results_per_page)-$results_per_page);
+ my $this_page_number = $i-9;
+ my $highlight = 1 if ($this_page_number == $current_page_number);
+ if ($this_page_number <= $pages) {
+ push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight, sort_by => join " ",@sort_by };
+ }
+ }
}
- my $servername = $server;
- push @sup_results_array, { servername => $servername, inner_sup_results_loop => \@inner_sup_results_array};
- $template->param(outer_sup_results_loop => \@sup_results_array);
+ # FIXME: no previous_page_offset when pages < 2
+ $template->param( PAGE_NUMBERS => \@page_numbers,
+ previous_page_offset => $previous_page_offset) unless $pages < 2;
+ $template->param( next_page_offset => $next_page_offset) unless $pages eq $current_page_number;
}
- }
+
+ # no hits
+ else {
+ $template->param(searchdesc => 1,query_desc => $query_desc,limit_desc => $limit_desc);
+ $template->param (z3950_search_params => C4::Search::z3950_search_args($z3950par || $query_desc));
+ }
+
+ } # end of the if local
+
+ # asynchronously search the authority server
+ elsif ($server =~/authorityserver/) { # this is the local authority server
+ my @inner_sup_results_array;
+ for my $sup_record ( @{$results_hashref->{$server}->{"RECORDS"}} ) {
+ my $marc_record_object = MARC::Record->new_from_usmarc($sup_record);
+ # warn "Authority Found: ".$marc_record_object->as_formatted();
+ push @inner_sup_results_array, {
+ 'title' => $marc_record_object->field(100)->subfield('a'),
+ 'link' => "&idx=an&q=".$marc_record_object->field('001')->as_string(),
+ };
+ }
+ push @sup_results_array, { servername => $server,
+ inner_sup_results_loop => \@inner_sup_results_array} if @inner_sup_results_array;
+ }
+ # FIXME: can add support for other targets as needed here
+ $template->param( outer_sup_results_loop => \@sup_results_array);
} #/end of the for loop
#$template->param(FEDERATED_RESULTS => \@results_array);
$template->param(
#classlist => $classlist,
total => $total,
- searchdesc => ($query_type?"$query_type=":"")."$search_desc",
opacfacets => 1,
facets_loop => $facets,
- scan_use => $scan,
+ scan => $scan,
search_error => $error,
);
-## Now let's find out if we have any supplemental data to show the user
-# and in the meantime, save the current query for statistical purposes, etc.
-my $koha_spsuggest; # a flag to tell if we've got suggestions coming from Koha
-my @koha_spsuggest; # place we store the suggestions to be returned to the template as LOOP
-my $phrases = $search_desc;
-my $ipaddress;
-
-if ( C4::Context->preference("kohaspsuggest") ) {
- my ($suggest_host, $suggest_dbname, $suggest_user, $suggest_pwd) = split(':', C4::Context->preference("kohaspsuggest"));
- eval {
- my $koha_spsuggest_dbh;
- # FIXME: this needs to be moved to Context.pm
- eval {
- $koha_spsuggest_dbh=DBI->connect("DBI:mysql:$suggest_dbname:$suggest_host","$suggest_user","$suggest_pwd");
- };
- if ($@) {
- warn "can't connect to spsuggest db";
- }
- else {
- my $koha_spsuggest_insert = "INSERT INTO phrase_log(phr_phrase,phr_resultcount,phr_ip) VALUES(?,?,?)";
- my $koha_spsuggest_query = "SELECT display FROM distincts WHERE strcmp(soundex(suggestion), soundex(?)) = 0 order by soundex(suggestion) limit 0,5";
- my $koha_spsuggest_sth = $koha_spsuggest_dbh->prepare($koha_spsuggest_query);
- $koha_spsuggest_sth->execute($phrases);
- while (my $spsuggestion = $koha_spsuggest_sth->fetchrow_array) {
- $spsuggestion =~ s/(:|\/)//g;
- my %line;
- $line{spsuggestion} = $spsuggestion;
- push @koha_spsuggest,\%line;
- $koha_spsuggest = 1;
- }
-
- # Now save the current query
- $koha_spsuggest_sth=$koha_spsuggest_dbh->prepare($koha_spsuggest_insert);
- #$koha_spsuggest_sth->execute($phrases,$results_per_page,$ipaddress);
- $koha_spsuggest_sth->finish;
- $template->param( koha_spsuggest => $koha_spsuggest ) unless $hits;
- $template->param( SPELL_SUGGEST => \@koha_spsuggest,
- );
- }
- };
- if ($@) {
- warn "Kohaspsuggest failure:".$@;
- }
+if ($query_desc || $limit_desc) {
+ $template->param(searchdesc => 1);
}
# VI. BUILD THE TEMPLATE
projects / koha.git / blobdiff