2 # Script to perform searching
3 # For documentation try 'perldoc /path/to/search'
5 # Copyright 2006 LibLime
7 # This file is part of Koha
9 # Koha is free software; you can redistribute it and/or modify it under the
10 # terms of the GNU General Public License as published by the Free Software
11 # Foundation; either version 2 of the License, or (at your option) any later
14 # Koha is distributed in the hope that it will be useful, but WITHOUT ANY
15 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
16 # A PARTICULAR PURPOSE. See the GNU General Public License for more details.
18 # You should have received a copy of the GNU General Public License along
19 # with Koha; if not, write to the Free Software Foundation, Inc.,
20 # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
24 search - a search script for finding records in a Koha system (Version 3)
28 This script utilizes a new search API for Koha 3. It is designed to be
29 simple to use and configure, yet capable of performing feats like stemming,
30 field weighting, relevance ranking, support for multiple query language
31 formats (CCL, CQL, PQF), full support for the bib1 attribute set, extended
32 attribute sets defined in Zebra profiles, access to the full range of Z39.50
33 and SRU query options, federated searches on Z39.50/SRU targets, etc.
35 The API as represented in this script is mostly sound, even if the individual
36 functions in Search.pm and Koha.pm need to be cleaned up. Of course, you are
39 I will attempt to describe what is happening at each part of this script.
40 -- Joshua Ferraro <jmf AT liblime DOT com>
44 This script performs two functions:
48 =item 1. interacts with Koha to retrieve and display the results of a search
50 =item 2. loads the advanced search page
54 These two functions share many of the same variables and modules, so the first
55 task is to load what they have in common and determine which template to use.
56 Once determined, proceed to only load the variables and procedures necessary
59 =head2 LOADING ADVANCED SEARCH PAGE
61 This is fairly straightforward, and I won't go into detail ;-)
63 =head2 PERFORMING A SEARCH
65 If we're performing a search, this script performs three primary
70 =item 1. builds query strings (yes, plural)
72 =item 2. perform the search and return the results array
74 =item 3. build the HTML for output to the template
78 There are several additional secondary functions performed that I will
81 =head3 1. Building Query Strings
83 There are several types of queries needed in the process of search and retrieve:
87 =item 1 $query - the fully-built query passed to zebra
89 This is the most complex query that needs to be built. The original design goal
90 was to use a custom CCL2PQF query parser to translate an incoming CCL query into
91 a multi-leaf query to pass to Zebra. It needs to be multi-leaf to allow field
92 weighting, koha-specific relevance ranking, and stemming. When I have a chance
93 I'll try to flesh out this section to better explain.
95 This query incorporates query profiles that aren't compatible with most non-Zebra
96 Z39.50 targets to acomplish the field weighting and relevance ranking.
98 =item 2 $simple_query - a simple query that doesn't contain the field weighting,
99 stemming, etc., suitable to pass off to other search targets
101 This query is just the user's query expressed in CCL CQL, or PQF for passing to a
102 non-zebra Z39.50 target (one that doesn't support the extended profile that Zebra does).
104 =item 3 $query_cgi - passed to the template / saved for future refinements of
107 This is a simple string that completely expresses the query as a CGI string that
108 can be used for future refinements of the query or as a part of a history feature.
110 =item 4 $query_desc - Human search description - what the user sees in search
113 This is a simple string that is human readable. It will contain '=', ',', etc.
117 =head3 2. Perform the Search
119 This section takes the query strings and performs searches on the named servers,
120 including the Koha Zebra server, stores the results in a deeply nested object,
121 builds 'faceted results', and returns these objects.
125 The final major section of this script takes the objects collected thusfar and
126 builds the HTML for output to the template and user.
128 =head3 Additional Notes
134 use strict
; # always use
135 #use warnings; FIXME - Bug 2505
137 ## STEP 1. Load things that are used in both search page and
138 # results page and decide which template to load, operations
144 use C4
::Auth
qw(:DEFAULT get_session);
146 use C4
::Languages
qw(getAllLanguages);
148 use C4
::Members
qw(GetMember);
149 use C4
::VirtualShelves
qw(GetRecentShelves);
150 use POSIX
qw(ceil floor);
151 use C4
::Branch
; # GetBranches
153 my $DisplayMultiPlaceHold = C4
::Context
->preference("DisplayMultiPlaceHold");
154 # create a new CGI object
155 # FIXME: no_undef_params needs to be tested
156 use CGI
qw('-no_undef_params');
159 my ($template,$borrowernumber,$cookie);
161 # decide which template to use
164 my @params = $cgi->param("limit");
165 if ((@params>=1) || ($cgi->param("q")) || ($cgi->param('multibranchlimit')) || ($cgi->param('limit-yr')) ) {
166 $template_name = 'catalogue/results.tmpl';
169 $template_name = 'catalogue/advsearch.tmpl';
170 $template_type = 'advsearch';
173 ($template, $borrowernumber, $cookie) = get_template_and_user
({
174 template_name
=> $template_name,
177 authnotrequired
=> 0,
178 flagsrequired
=> { catalogue
=> 1 },
181 if (C4
::Context
->preference("marcflavour") eq "UNIMARC" ) {
182 $template->param('UNIMARC' => 1);
185 if($cgi->cookie("holdfor")){
186 my $holdfor_patron = GetMember
('borrowernumber' => $cgi->cookie("holdfor"));
188 holdfor
=> $cgi->cookie("holdfor"),
189 holdfor_surname
=> $holdfor_patron->{'surname'},
190 holdfor_firstname
=> $holdfor_patron->{'firstname'},
191 holdfor_cardnumber
=> $holdfor_patron->{'cardnumber'},
196 # Deprecated, but preserved because it's interesting :-)
197 # The same thing can be accomplished with mod_rewrite in
201 #my $uri = $cgi->url(-base => 1);
202 #my $relative_url = $cgi->url(-relative=>1);
203 #$uri.="/".$relative_url."?";
205 #my @cgi_params_list = $cgi->param();
206 #my $url_params = $cgi->Vars;
208 #for my $each_param_set (@cgi_params_list) {
209 # $uri.= join "", map "\&$each_param_set=".$_, split("\0",$url_params->{$each_param_set}) if $url_params->{$each_param_set};
211 #warn "New URI:$uri";
212 # Only re-write a URI if there are params or if it already hasn't been re-written
213 #unless (($cgi->param('r')) || (!$cgi->param()) ) {
214 # print $cgi->redirect( -uri=>$uri."&r=1",
215 # -cookie => $cookie);
220 my $branches = GetBranches
();
223 # we need to know the borrower branch code to set a default branch
224 my $borrowerbranchcode = C4
::Context
->userenv->{'branch'};
226 for my $branch_hash (sort { $branches->{$a}->{branchname
} cmp $branches->{$b}->{branchname
} } keys %$branches) {
227 # if independantbranches is activated, set the default branch to the borrower branch
228 my $selected = (C4
::Context
->preference("independantbranches") and ($borrowerbranchcode eq $branch_hash)) ?
1 : undef;
229 push @branch_loop, {value
=> "$branch_hash" , branchname
=> $branches->{$branch_hash}->{'branchname'}, selected
=> $selected};
232 my $categories = GetBranchCategories
(undef,'searchdomain');
234 $template->param(branchloop
=> \
@branch_loop, searchdomainloop
=> $categories);
236 # load the Type stuff
237 # load the Type stuff
238 my $itemtypes = GetItemTypes
;
239 # the index parameter is different for item-level itemtypes
240 my $itype_or_itemtype = (C4
::Context
->preference("item-level_itypes"))?
'itype':'itemtype';
244 my $advanced_search_types = C4
::Context
->preference("AdvancedSearchTypes");
246 if (!$advanced_search_types or $advanced_search_types eq 'itemtypes') { foreach my $thisitemtype ( sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'} } keys %$itemtypes ) {
247 my %row =( number
=>$cnt++,
248 ccl
=> $itype_or_itemtype,
249 code
=> $thisitemtype,
250 selected
=> $selected,
251 description
=> $itemtypes->{$thisitemtype}->{'description'},
253 imageurl
=> getitemtypeimagelocation
( 'intranet', $itemtypes->{$thisitemtype}->{'imageurl'} ),
255 $selected = 0 if ($selected) ;
256 push @itemtypesloop, \
%row;
258 $template->param(itemtypeloop
=> \
@itemtypesloop);
260 my $advsearchtypes = GetAuthorisedValues
($advanced_search_types);
261 for my $thisitemtype (sort {$a->{'lib'} cmp $b->{'lib'}} @
$advsearchtypes) {
264 ccl
=> $advanced_search_types,
265 code
=> $thisitemtype->{authorised_value
},
266 selected
=> $selected,
267 description
=> $thisitemtype->{'lib'},
269 imageurl
=> getitemtypeimagelocation
( 'intranet', $thisitemtype->{'imageurl'} ),
271 push @itemtypesloop, \
%row;
273 $template->param(itemtypeloop
=> \
@itemtypesloop);
276 # The following should only be loaded if we're bringing up the advanced search template
277 if ( $template_type eq 'advsearch' ) {
279 # load the servers (used for searching -- to do federated searching, etc.)
280 my $primary_servers_loop;# = displayPrimaryServers();
281 $template->param(outer_servers_loop
=> $primary_servers_loop,);
283 my $secondary_servers_loop;
284 $template->param(outer_sup_servers_loop
=> $secondary_servers_loop,);
286 # set the default sorting
287 my $default_sort_by = C4
::Context
->preference('defaultSortField')."_".C4
::Context
->preference('defaultSortOrder')
288 if (C4
::Context
->preference('OPACdefaultSortField') && C4
::Context
->preference('OPACdefaultSortOrder'));
289 $template->param($default_sort_by => 1);
291 # determine what to display next to the search boxes (ie, boolean option
292 # shouldn't appear on the first one, scan indexes should, adding a new
293 # box should only appear on the last, etc.
294 my @search_boxes_array;
295 my $search_boxes_count = C4
::Context
->preference("OPACAdvSearchInputCount") || 3; # FIXME: using OPAC sysprefs?
296 # FIXME: all this junk can be done in TMPL using __first__ and __last__
297 for (my $i=1;$i<=$search_boxes_count;$i++) {
298 # if it's the first one, don't display boolean option, but show scan indexes
300 push @search_boxes_array, {scan_index
=> 1};
302 # if it's the last one, show the 'add field' box
303 elsif ($i==$search_boxes_count) {
304 push @search_boxes_array,
311 push @search_boxes_array,
318 $template->param(uc(C4
::Context
->preference("marcflavour")) => 1,
319 search_boxes_loop
=> \
@search_boxes_array);
321 # load the language limits (for search)
322 my $languages_limit_loop = getAllLanguages
();
323 $template->param(search_languages_loop
=> $languages_limit_loop,);
325 # use the global setting by default
326 if ( C4
::Context
->preference("expandedSearchOption") == 1) {
327 $template->param( expanded_options
=> C4
::Context
->preference("expandedSearchOption") );
329 # but let the user override it
330 if ( ($cgi->param('expanded_options') == 0) || ($cgi->param('expanded_options') == 1 ) ) {
331 $template->param( expanded_options
=> $cgi->param('expanded_options'));
334 $template->param(virtualshelves
=> C4
::Context
->preference("virtualshelves"));
336 output_html_with_http_headers
$cgi, $cookie, $template->output;
340 ### OK, if we're this far, we're performing a search, not just loading the advanced search page
342 # Fetch the paramater list as a hash in scalar context:
343 # * returns paramater list as tied hash ref
344 # * we can edit the values by changing the key
345 # * multivalued CGI paramaters are returned as a packaged string separated by "\0" (null)
346 my $params = $cgi->Vars;
348 # Params that can have more than one value
349 # sort by is used to sort the query
350 # in theory can have more than one but generally there's just one
352 my $default_sort_by = C4
::Context
->preference('defaultSortField')."_".C4
::Context
->preference('defaultSortOrder')
353 if (C4
::Context
->preference('defaultSortField') && C4
::Context
->preference('defaultSortOrder'));
355 @sort_by = split("\0",$params->{'sort_by'}) if $params->{'sort_by'};
356 $sort_by[0] = $default_sort_by unless $sort_by[0];
357 foreach my $sort (@sort_by) {
358 $template->param($sort => 1);
360 $template->param('sort_by' => $sort_by[0]);
362 # Use the servers defined, or just search our local catalog(default)
364 @servers = split("\0",$params->{'server'}) if $params->{'server'};
366 #FIXME: this should be handled using Context.pm
367 @servers = ("biblioserver");
368 # @servers = C4::Context->config("biblioserver");
370 # operators include boolean and proximity operators and are used
371 # to evaluate multiple operands
373 @operators = split("\0",$params->{'op'}) if $params->{'op'};
375 # indexes are query qualifiers, like 'title', 'author', etc. They
376 # can be single or multiple parameters separated by comma: kw,right-Truncation
378 @indexes = split("\0",$params->{'idx'});
380 # if a simple index (only one) display the index used in the top search box
381 if ($indexes[0] && !$indexes[1]) {
382 $template->param("ms_".$indexes[0] => 1);}
385 # an operand can be a single term, a phrase, or a complete ccl query
387 @operands = split("\0",$params->{'q'}) if $params->{'q'};
389 # limits are use to limit to results to a pre-defined category such as branch or language
391 @limits = split("\0",$params->{'limit'}) if $params->{'limit'};
393 if($params->{'multibranchlimit'}) {
394 push @limits, join(" or ", map { "branch: $_ "} @
{GetBranchesInCategory
($params->{'multibranchlimit'})}) ;
398 foreach my $limit(@limits) {
399 if ($limit =~/available/) {
403 $template->param(available
=> $available);
405 # append year limits if they exist
408 if ($params->{'limit-yr'}) {
409 if ($params->{'limit-yr'} =~ /\d{4}-\d{4}/) {
410 my ($yr1,$yr2) = split(/-/, $params->{'limit-yr'});
411 $limit_yr = "yr,st-numeric,ge=$yr1 and yr,st-numeric,le=$yr2";
412 $limit_yr_value = "$yr1-$yr2";
414 elsif ($params->{'limit-yr'} =~ /\d{4}/) {
415 $limit_yr = "yr,st-numeric=$params->{'limit-yr'}";
416 $limit_yr_value = $params->{'limit-yr'};
418 push @limits,$limit_yr;
419 #FIXME: Should return a error to the user, incorect date format specified
422 # convert indexes and operands to corresponding parameter names for the z3950 search
423 # $ %z3950p will be a hash ref if the indexes are present (advacned search), otherwise undef
425 my $indexes2z3950 = {
426 kw
=>'title', au
=>'author', 'au,phr'=>'author', nb
=>'isbn', ns
=>'issn',
427 'lcn,phr'=>'dewey', su
=>'subject', 'su,phr'=>'subject',
428 ti
=>'title', 'ti,phr'=>'title', se
=>'title'
430 for (my $ii = 0; $ii < @operands; ++$ii)
432 my $name = $indexes2z3950->{$indexes[$ii]};
433 if (defined $name && defined $operands[$ii])
436 $z3950par->{$name} = $operands[$ii] if !exists $z3950par->{$name};
441 # Params that can only have one value
442 my $scan = $params->{'scan'};
443 my $count = C4
::Context
->preference('numSearchResults') || 20;
444 my $results_per_page = $params->{'count'} || $count;
445 my $offset = $params->{'offset'} || 0;
446 my $page = $cgi->param('page') || 1;
447 #my $offset = ($page-1)*$results_per_page;
449 my $expanded_facet = $params->{'expand'};
451 # Define some global variables
452 my ( $error,$query,$simple_query,$query_cgi,$query_desc,$limit,$limit_cgi,$limit_desc,$stopwords_removed,$query_type);
456 ## I. BUILD THE QUERY
457 my $lang = C4
::Output
::getlanguagecookie
($cgi);
458 ( $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,$lang);
460 ## parse the query_cgi string and put it into a form suitable for <input>s
462 my $scan_index_to_use;
464 for my $this_cgi ( split('&',$query_cgi) ) {
465 next unless $this_cgi;
466 $this_cgi =~ m/(.*=)(.*)/;
468 my $input_value = $2;
469 $input_name =~ s/=$//;
470 push @query_inputs, { input_name
=> $input_name, input_value
=> $input_value };
471 if ($input_name eq 'idx') {
472 $scan_index_to_use = $input_value; # unless $scan_index_to_use;
475 $template->param ( QUERY_INPUTS
=> \
@query_inputs,
476 scan_index_to_use
=> $scan_index_to_use );
478 ## parse the limit_cgi string and put it into a form suitable for <input>s
480 for my $this_cgi ( split('&',$limit_cgi) ) {
481 next unless $this_cgi;
482 # handle special case limit-yr
483 if ($this_cgi =~ /yr,st-numeric/) {
484 push @limit_inputs, { input_name
=> 'limit-yr', input_value
=> $limit_yr_value };
487 $this_cgi =~ m/(.*=)(.*)/;
489 my $input_value = $2;
490 $input_name =~ s/=$//;
491 push @limit_inputs, { input_name
=> $input_name, input_value
=> $input_value };
493 $template->param ( LIMIT_INPUTS
=> \
@limit_inputs );
495 ## II. DO THE SEARCH AND GET THE RESULTS
496 my $total; # the total results for the whole set
497 my $facets; # this object stores the faceted results that display on the left-hand of the results page
501 if (C4
::Context
->preference('NoZebra')) {
502 $query=~s/yr(:|=)\s*([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
503 $simple_query=~s/yr\s*(:|=)([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
506 ($error, $results_hashref, $facets) = NZgetRecords
($query,$simple_query,\
@sort_by,\
@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
510 ($error, $results_hashref, $facets) = getRecords
($query,$simple_query,\
@sort_by,\
@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
513 # This sorts the facets into alphabetical order
515 foreach my $f (@
$facets) {
516 $f->{facets
} = [ sort { uc($a->{facet_title_value
}) cmp uc($b->{facet_title_value
}) } @
{ $f->{facets
} } ];
520 $template->param(query_error
=> $error.$@
);
521 output_html_with_http_headers
$cgi, $cookie, $template->output;
525 # At this point, each server has given us a result set
526 # now we build that set for template display
527 my @sup_results_array;
528 for (my $i=0;$i<@servers;$i++) {
529 my $server = $servers[$i];
530 if ($server =~/biblioserver/) { # this is the local bibliographic server
531 $hits = $results_hashref->{$server}->{"hits"};
532 my $page = $cgi->param('page') || 0;
533 my @newresults = searchResults
('intranet', $query_desc, $hits, $results_per_page, $offset, $scan,
534 @
{$results_hashref->{$server}->{"RECORDS"}});
535 $total = $total + $results_hashref->{$server}->{"hits"};
536 ## If there's just one result, redirect to the detail page
538 my $biblionumber = $newresults[0]->{biblionumber
};
539 my $defaultview = C4
::Context
->preference('IntranetBiblioDefaultView');
540 my $views = { C4
::Search
::enabled_staff_search_views
};
541 if ($defaultview eq 'isbd' && $views->{can_view_ISBD
}) {
542 print $cgi->redirect("/cgi-bin/koha/catalogue/ISBDdetail.pl?biblionumber=$biblionumber");
543 } elsif ($defaultview eq 'marc' && $views->{can_view_MARC
}) {
544 print $cgi->redirect("/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=$biblionumber");
545 } elsif ($defaultview eq 'labeled_marc' && $views->{can_view_labeledMARC
}) {
546 print $cgi->redirect("/cgi-bin/koha/catalogue/labeledMARCdetail.pl?biblionumber=$biblionumber");
548 print $cgi->redirect("/cgi-bin/koha/catalogue/detail.pl?biblionumber=$biblionumber");
555 $template->param(total
=> $hits);
556 my $limit_cgi_not_availablity = $limit_cgi;
557 $limit_cgi_not_availablity =~ s/&limit=available//g;
558 $template->param(limit_cgi_not_availablity
=> $limit_cgi_not_availablity);
559 $template->param(limit_cgi
=> $limit_cgi);
560 $template->param(query_cgi
=> $query_cgi);
561 $template->param(query_desc
=> $query_desc);
562 $template->param(limit_desc
=> $limit_desc);
563 $template->param(offset
=> $offset);
564 $template->param(DisplayMultiPlaceHold
=> $DisplayMultiPlaceHold);
565 $template->param (z3950_search_params
=> C4
::Search
::z3950_search_args
($query_desc));
566 if ($query_desc || $limit_desc) {
567 $template->param(searchdesc
=> 1);
569 $template->param(stopwords_removed
=> "@$stopwords_removed") if $stopwords_removed;
570 $template->param(results_per_page
=> $results_per_page);
571 $template->param(SEARCH_RESULTS
=> \
@newresults);
573 ## FIXME: add a global function for this, it's better than the current global one
574 ## Build the page numbers on the bottom of the page
576 # total number of pages there will be
577 my $pages = ceil
($hits / $results_per_page);
578 # default page number
579 my $current_page_number = 1;
580 $current_page_number = ($offset / $results_per_page + 1) if $offset;
581 my $previous_page_offset = $offset - $results_per_page unless ($offset - $results_per_page <0);
582 my $next_page_offset = $offset + $results_per_page;
583 # If we're within the first 10 pages, keep it simple
584 #warn "current page:".$current_page_number;
585 if ($current_page_number < 10) {
586 # just show the first 10 pages
587 # Loop through the pages
588 my $pages_to_show = 10;
589 $pages_to_show = $pages if $pages<10;
590 for (my $i=1; $i<=$pages_to_show;$i++) {
591 # the offset for this page
592 my $this_offset = (($i*$results_per_page)-$results_per_page);
593 # the page number for this page
594 my $this_page_number = $i;
595 # it should only be highlighted if it's the current page
596 my $highlight = 1 if ($this_page_number == $current_page_number);
597 # put it in the array
598 push @page_numbers, { offset
=> $this_offset, pg
=> $this_page_number, highlight
=> $highlight, sort_by
=> join " ",@sort_by };
604 # now, show twenty pages, with the current one smack in the middle
606 for (my $i=$current_page_number; $i<=($current_page_number + 20 );$i++) {
607 my $this_offset = ((($i-9)*$results_per_page)-$results_per_page);
608 my $this_page_number = $i-9;
609 my $highlight = 1 if ($this_page_number == $current_page_number);
610 if ($this_page_number <= $pages) {
611 push @page_numbers, { offset
=> $this_offset, pg
=> $this_page_number, highlight
=> $highlight, sort_by
=> join " ",@sort_by };
615 # FIXME: no previous_page_offset when pages < 2
616 $template->param( PAGE_NUMBERS
=> \
@page_numbers,
617 previous_page_offset
=> $previous_page_offset) unless $pages < 2;
618 $template->param( next_page_offset
=> $next_page_offset) unless $pages eq $current_page_number;
624 $template->param(searchdesc
=> 1,query_desc
=> $query_desc,limit_desc
=> $limit_desc);
625 $template->param (z3950_search_params
=> C4
::Search
::z3950_search_args
($z3950par || $query_desc));
628 } # end of the if local
630 # asynchronously search the authority server
631 elsif ($server =~/authorityserver/) { # this is the local authority server
632 my @inner_sup_results_array;
633 for my $sup_record ( @
{$results_hashref->{$server}->{"RECORDS"}} ) {
634 my $marc_record_object = MARC
::Record
->new_from_usmarc($sup_record);
635 # warn "Authority Found: ".$marc_record_object->as_formatted();
636 push @inner_sup_results_array, {
637 'title' => $marc_record_object->field(100)->subfield('a'),
638 'link' => "&idx=an&q=".$marc_record_object->field('001')->as_string(),
641 push @sup_results_array, { servername
=> $server,
642 inner_sup_results_loop
=> \
@inner_sup_results_array} if @inner_sup_results_array;
644 # FIXME: can add support for other targets as needed here
645 $template->param( outer_sup_results_loop
=> \
@sup_results_array);
646 } #/end of the for loop
647 #$template->param(FEDERATED_RESULTS => \@results_array);
651 #classlist => $classlist,
654 facets_loop
=> $facets,
656 search_error
=> $error,
659 if ($query_desc || $limit_desc) {
660 $template->param(searchdesc
=> 1);
663 # VI. BUILD THE TEMPLATE
665 # Build drop-down list for 'Add To:' menu...
667 my $row_count = 10; # FIXME:This probably should be a syspref
668 my ($pubshelves, $total) = GetRecentShelves
(2, $row_count, undef);
669 my ($barshelves, $total) = GetRecentShelves
(1, $row_count, $borrowernumber);
671 my @pubshelves = @
{$pubshelves};
672 my @barshelves = @
{$barshelves};
675 $template->param( addpubshelves
=> scalar (@pubshelves));
676 $template->param( addpubshelvesloop
=> @pubshelves);
680 $template->param( addbarshelves
=> scalar (@barshelves));
681 $template->param( addbarshelvesloop
=> @barshelves);
686 output_html_with_http_headers
$cgi, $cookie, $template->output;