2 # Script to perform searching
3 # For documentation try 'perldoc /path/to/search'
5 # Copyright 2006 LibLime
6 # Copyright 2010 BibLibre
8 # This file is part of Koha
10 # Koha is free software; you can redistribute it and/or modify it under the
11 # terms of the GNU General Public License as published by the Free Software
12 # Foundation; either version 2 of the License, or (at your option) any later
15 # Koha is distributed in the hope that it will be useful, but WITHOUT ANY
16 # WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
17 # A PARTICULAR PURPOSE. See the GNU General Public License for more details.
19 # You should have received a copy of the GNU General Public License along
20 # with Koha; if not, write to the Free Software Foundation, Inc.,
21 # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
25 search - a search script for finding records in a Koha system (Version 3)
29 This script utilizes a new search API for Koha 3. It is designed to be
30 simple to use and configure, yet capable of performing feats like stemming,
31 field weighting, relevance ranking, support for multiple query language
32 formats (CCL, CQL, PQF), full support for the bib1 attribute set, extended
33 attribute sets defined in Zebra profiles, access to the full range of Z39.50
34 and SRU query options, federated searches on Z39.50/SRU targets, etc.
36 The API as represented in this script is mostly sound, even if the individual
37 functions in Search.pm and Koha.pm need to be cleaned up. Of course, you are
40 I will attempt to describe what is happening at each part of this script.
41 -- Joshua Ferraro <jmf AT liblime DOT com>
45 This script performs two functions:
49 =item 1. interacts with Koha to retrieve and display the results of a search
51 =item 2. loads the advanced search page
55 These two functions share many of the same variables and modules, so the first
56 task is to load what they have in common and determine which template to use.
57 Once determined, proceed to only load the variables and procedures necessary
60 =head2 LOADING ADVANCED SEARCH PAGE
62 This is fairly straightforward, and I won't go into detail ;-)
64 =head2 PERFORMING A SEARCH
66 If we're performing a search, this script performs three primary
71 =item 1. builds query strings (yes, plural)
73 =item 2. perform the search and return the results array
75 =item 3. build the HTML for output to the template
79 There are several additional secondary functions performed that I will
82 =head3 1. Building Query Strings
84 There are several types of queries needed in the process of search and retrieve:
88 =item 1 $query - the fully-built query passed to zebra
90 This is the most complex query that needs to be built. The original design goal
91 was to use a custom CCL2PQF query parser to translate an incoming CCL query into
92 a multi-leaf query to pass to Zebra. It needs to be multi-leaf to allow field
93 weighting, koha-specific relevance ranking, and stemming. When I have a chance
94 I'll try to flesh out this section to better explain.
96 This query incorporates query profiles that aren't compatible with most non-Zebra
97 Z39.50 targets to acomplish the field weighting and relevance ranking.
99 =item 2 $simple_query - a simple query that doesn't contain the field weighting,
100 stemming, etc., suitable to pass off to other search targets
102 This query is just the user's query expressed in CCL CQL, or PQF for passing to a
103 non-zebra Z39.50 target (one that doesn't support the extended profile that Zebra does).
105 =item 3 $query_cgi - passed to the template / saved for future refinements of
108 This is a simple string that completely expresses the query as a CGI string that
109 can be used for future refinements of the query or as a part of a history feature.
111 =item 4 $query_desc - Human search description - what the user sees in search
114 This is a simple string that is human readable. It will contain '=', ',', etc.
118 =head3 2. Perform the Search
120 This section takes the query strings and performs searches on the named servers,
121 including the Koha Zebra server, stores the results in a deeply nested object,
122 builds 'faceted results', and returns these objects.
126 The final major section of this script takes the objects collected thusfar and
127 builds the HTML for output to the template and user.
129 =head3 Additional Notes
135 use strict; # always use
136 #use warnings; FIXME - Bug 2505
138 ## STEP 1. Load things that are used in both search page and
139 # results page and decide which template to load, operations
145 use C4::Auth qw(:DEFAULT get_session);
147 use C4::Languages qw(getAllLanguages);
149 use C4::Members qw(GetMember);
150 use C4::VirtualShelves qw(GetRecentShelves);
151 use POSIX qw(ceil floor);
152 use C4::Branch; # GetBranches
154 my $DisplayMultiPlaceHold = C4::Context->preference("DisplayMultiPlaceHold");
155 # create a new CGI object
156 # FIXME: no_undef_params needs to be tested
157 use CGI qw('-no_undef_params');
160 my ($template,$borrowernumber,$cookie);
162 # decide which template to use
165 my @params = $cgi->param("limit");
166 if ((@params>=1) || ($cgi->param("q")) || ($cgi->param('multibranchlimit')) || ($cgi->param('limit-yr')) ) {
167 $template_name = 'catalogue/results.tmpl';
170 $template_name = 'catalogue/advsearch.tmpl';
171 $template_type = 'advsearch';
174 ($template, $borrowernumber, $cookie) = get_template_and_user({
175 template_name => $template_name,
178 authnotrequired => 0,
179 flagsrequired => { catalogue => 1 },
182 if (C4::Context->preference("marcflavour") eq "UNIMARC" ) {
183 $template->param('UNIMARC' => 1);
186 if($cgi->cookie("holdfor")){
187 my $holdfor_patron = GetMember('borrowernumber' => $cgi->cookie("holdfor"));
189 holdfor => $cgi->cookie("holdfor"),
190 holdfor_surname => $holdfor_patron->{'surname'},
191 holdfor_firstname => $holdfor_patron->{'firstname'},
192 holdfor_cardnumber => $holdfor_patron->{'cardnumber'},
197 # Deprecated, but preserved because it's interesting :-)
198 # The same thing can be accomplished with mod_rewrite in
202 #my $uri = $cgi->url(-base => 1);
203 #my $relative_url = $cgi->url(-relative=>1);
204 #$uri.="/".$relative_url."?";
206 #my @cgi_params_list = $cgi->param();
207 #my $url_params = $cgi->Vars;
209 #for my $each_param_set (@cgi_params_list) {
210 # $uri.= join "", map "\&$each_param_set=".$_, split("\0",$url_params->{$each_param_set}) if $url_params->{$each_param_set};
212 #warn "New URI:$uri";
213 # Only re-write a URI if there are params or if it already hasn't been re-written
214 #unless (($cgi->param('r')) || (!$cgi->param()) ) {
215 # print $cgi->redirect( -uri=>$uri."&r=1",
216 # -cookie => $cookie);
221 my $branches = GetBranches();
223 # Populate branch_loop with all branches sorted by their name. If
224 # independantbranches is activated, set the default branch to the borrower
225 # branch, except for superlibrarian who need to search all libraries.
226 my $user = C4::Context->userenv;
227 my @branch_loop = map {
230 branchname => $branches->{$_}->{branchname},
231 selected => $user->{branch} eq $_ && C4::Branch::onlymine(),
234 $branches->{$a}->{branchname} cmp $branches->{$b}->{branchname}
237 my $categories = GetBranchCategories(undef,'searchdomain');
239 $template->param(branchloop => \@branch_loop, searchdomainloop => $categories);
241 # load the Type stuff
242 # load the Type stuff
243 my $itemtypes = GetItemTypes;
244 # the index parameter is different for item-level itemtypes
245 my $itype_or_itemtype = (C4::Context->preference("item-level_itypes"))?'itype':'itemtype';
249 my $advanced_search_types = C4::Context->preference("AdvancedSearchTypes");
251 if (!$advanced_search_types or $advanced_search_types eq 'itemtypes') { foreach my $thisitemtype ( sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'} } keys %$itemtypes ) {
252 my %row =( number=>$cnt++,
253 ccl => qq($itype_or_itemtype,phr),
254 code => $thisitemtype,
255 selected => $selected,
256 description => $itemtypes->{$thisitemtype}->{'description'},
258 imageurl=> getitemtypeimagelocation( 'intranet', $itemtypes->{$thisitemtype}->{'imageurl'} ),
260 $selected = 0 if ($selected) ;
261 push @itemtypesloop, \%row;
263 $template->param(itemtypeloop => \@itemtypesloop);
265 my $advsearchtypes = GetAuthorisedValues($advanced_search_types);
266 for my $thisitemtype (sort {$a->{'lib'} cmp $b->{'lib'}} @$advsearchtypes) {
269 ccl => $advanced_search_types,
270 code => $thisitemtype->{authorised_value},
271 selected => $selected,
272 description => $thisitemtype->{'lib'},
274 imageurl=> getitemtypeimagelocation( 'intranet', $thisitemtype->{'imageurl'} ),
276 push @itemtypesloop, \%row;
278 $template->param(itemtypeloop => \@itemtypesloop);
281 # The following should only be loaded if we're bringing up the advanced search template
282 if ( $template_type eq 'advsearch' ) {
284 # load the servers (used for searching -- to do federated searching, etc.)
285 my $primary_servers_loop;# = displayPrimaryServers();
286 $template->param(outer_servers_loop => $primary_servers_loop,);
288 my $secondary_servers_loop;
289 $template->param(outer_sup_servers_loop => $secondary_servers_loop,);
291 # set the default sorting
292 my $default_sort_by = C4::Context->preference('defaultSortField')."_".C4::Context->preference('defaultSortOrder')
293 if (C4::Context->preference('OPACdefaultSortField') && C4::Context->preference('OPACdefaultSortOrder'));
294 $template->param($default_sort_by => 1);
296 # determine what to display next to the search boxes (ie, boolean option
297 # shouldn't appear on the first one, scan indexes should, adding a new
298 # box should only appear on the last, etc.
299 my @search_boxes_array;
300 my $search_boxes_count = 3; # begin whith 3 boxes
301 # FIXME: all this junk can be done in TMPL using __first__ and __last__
302 for (my $i=1;$i<=$search_boxes_count;$i++) {
303 # if it's the first one, don't display boolean option, but show scan indexes
305 push @search_boxes_array, {scan_index => 1};
307 # if it's the last one, show the 'add field' box
308 elsif ($i==$search_boxes_count) {
309 push @search_boxes_array,
316 push @search_boxes_array,
323 $template->param(uc(C4::Context->preference("marcflavour")) => 1,
324 search_boxes_loop => \@search_boxes_array);
326 # load the language limits (for search)
327 my $languages_limit_loop = getAllLanguages();
328 $template->param(search_languages_loop => $languages_limit_loop,);
330 # use the global setting by default
331 if ( C4::Context->preference("expandedSearchOption") == 1) {
332 $template->param( expanded_options => C4::Context->preference("expandedSearchOption") );
334 # but let the user override it
335 if ( ($cgi->param('expanded_options') == 0) || ($cgi->param('expanded_options') == 1 ) ) {
336 $template->param( expanded_options => $cgi->param('expanded_options'));
339 $template->param(virtualshelves => C4::Context->preference("virtualshelves"));
341 output_html_with_http_headers $cgi, $cookie, $template->output;
345 ### OK, if we're this far, we're performing a search, not just loading the advanced search page
347 # Fetch the paramater list as a hash in scalar context:
348 # * returns paramater list as tied hash ref
349 # * we can edit the values by changing the key
350 # * multivalued CGI paramaters are returned as a packaged string separated by "\0" (null)
351 my $params = $cgi->Vars;
353 # Params that can have more than one value
354 # sort by is used to sort the query
355 # in theory can have more than one but generally there's just one
357 my $default_sort_by = C4::Context->preference('defaultSortField')."_".C4::Context->preference('defaultSortOrder')
358 if (C4::Context->preference('defaultSortField') && C4::Context->preference('defaultSortOrder'));
360 @sort_by = split("\0",$params->{'sort_by'}) if $params->{'sort_by'};
361 $sort_by[0] = $default_sort_by unless $sort_by[0];
362 foreach my $sort (@sort_by) {
363 $template->param($sort => 1);
365 $template->param('sort_by' => $sort_by[0]);
367 # Use the servers defined, or just search our local catalog(default)
369 @servers = split("\0",$params->{'server'}) if $params->{'server'};
371 #FIXME: this should be handled using Context.pm
372 @servers = ("biblioserver");
373 # @servers = C4::Context->config("biblioserver");
375 # operators include boolean and proximity operators and are used
376 # to evaluate multiple operands
378 @operators = split("\0",$params->{'op'}) if $params->{'op'};
380 # indexes are query qualifiers, like 'title', 'author', etc. They
381 # can be single or multiple parameters separated by comma: kw,right-Truncation
383 @indexes = split("\0",$params->{'idx'});
385 # if a simple index (only one) display the index used in the top search box
386 if ($indexes[0] && (!$indexes[1] || $params->{'scan'})) {
387 my $idx = "ms_".$indexes[0];
388 $idx =~ s/\,/comma/g; # template toolkit doesnt like variables with a , in it
389 $template->param($idx => 1);
393 # an operand can be a single term, a phrase, or a complete ccl query
395 @operands = split("\0",$params->{'q'}) if $params->{'q'};
397 # limits are use to limit to results to a pre-defined category such as branch or language
399 @limits = split("\0",$params->{'limit'}) if $params->{'limit'};
401 if($params->{'multibranchlimit'}) {
402 push @limits, '('.join( " or ", map { "branch: $_ " } @{ GetBranchesInCategory( $params->{'multibranchlimit'} ) } ).')';
406 foreach my $limit(@limits) {
407 if ($limit =~/available/) {
411 $template->param(available => $available);
413 # append year limits if they exist
416 if ($params->{'limit-yr'}) {
417 if ($params->{'limit-yr'} =~ /\d{4}-\d{4}/) {
418 my ($yr1,$yr2) = split(/-/, $params->{'limit-yr'});
419 $limit_yr = "yr,st-numeric,ge=$yr1 and yr,st-numeric,le=$yr2";
420 $limit_yr_value = "$yr1-$yr2";
422 elsif ($params->{'limit-yr'} =~ /\d{4}/) {
423 $limit_yr = "yr,st-numeric=$params->{'limit-yr'}";
424 $limit_yr_value = $params->{'limit-yr'};
426 push @limits,$limit_yr;
427 #FIXME: Should return a error to the user, incorect date format specified
430 # convert indexes and operands to corresponding parameter names for the z3950 search
431 # $ %z3950p will be a hash ref if the indexes are present (advacned search), otherwise undef
433 my $indexes2z3950 = {
434 kw=>'title', au=>'author', 'au,phr'=>'author', nb=>'isbn', ns=>'issn',
435 'lcn,phr'=>'dewey', su=>'subject', 'su,phr'=>'subject',
436 ti=>'title', 'ti,phr'=>'title', se=>'title'
438 for (my $ii = 0; $ii < @operands; ++$ii)
440 my $name = $indexes2z3950->{$indexes[$ii]};
441 if (defined $name && defined $operands[$ii])
444 $z3950par->{$name} = $operands[$ii] if !exists $z3950par->{$name};
449 # Params that can only have one value
450 my $scan = $params->{'scan'};
451 my $count = C4::Context->preference('numSearchResults') || 20;
452 my $results_per_page = $params->{'count'} || $count;
453 my $offset = $params->{'offset'} || 0;
454 my $page = $cgi->param('page') || 1;
455 #my $offset = ($page-1)*$results_per_page;
457 my $expanded_facet = $params->{'expand'};
459 # Define some global variables
460 my ( $error,$query,$simple_query,$query_cgi,$query_desc,$limit,$limit_cgi,$limit_desc,$stopwords_removed,$query_type);
464 ## I. BUILD THE QUERY
465 my $lang = C4::Output::getlanguagecookie($cgi);
466 ( $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);
468 ## parse the query_cgi string and put it into a form suitable for <input>s
470 my $scan_index_to_use;
472 for my $this_cgi ( split('&',$query_cgi) ) {
473 next unless $this_cgi;
474 $this_cgi =~ m/(.?)=(.*)/;
476 my $input_value = $2;
477 push @query_inputs, { input_name => $input_name, input_value => $input_value };
478 if ($input_name eq 'idx') {
479 $scan_index_to_use = $input_value; # unless $scan_index_to_use;
482 $template->param ( QUERY_INPUTS => \@query_inputs,
483 scan_index_to_use => $scan_index_to_use );
485 ## parse the limit_cgi string and put it into a form suitable for <input>s
487 for my $this_cgi ( split('&',$limit_cgi) ) {
488 next unless $this_cgi;
489 # handle special case limit-yr
490 if ($this_cgi =~ /yr,st-numeric/) {
491 push @limit_inputs, { input_name => 'limit-yr', input_value => $limit_yr_value };
494 $this_cgi =~ m/(.*=)(.*)/;
496 my $input_value = $2;
497 $input_name =~ s/=$//;
498 push @limit_inputs, { input_name => $input_name, input_value => $input_value };
500 $template->param ( LIMIT_INPUTS => \@limit_inputs );
502 ## II. DO THE SEARCH AND GET THE RESULTS
503 my $total; # the total results for the whole set
504 my $facets; # this object stores the faceted results that display on the left-hand of the results page
508 if (C4::Context->preference('NoZebra')) {
509 $query=~s/yr(:|=)\s*([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
510 $simple_query=~s/yr\s*(:|=)([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
513 ($error, $results_hashref, $facets) = NZgetRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
517 ($error, $results_hashref, $facets) = getRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
520 # This sorts the facets into alphabetical order
522 foreach my $f (@$facets) {
523 $f->{facets} = [ sort { uc($a->{facet_title_value}) cmp uc($b->{facet_title_value}) } @{ $f->{facets} } ];
527 $template->param(query_error => $error.$@);
528 output_html_with_http_headers $cgi, $cookie, $template->output;
532 # At this point, each server has given us a result set
533 # now we build that set for template display
534 my @sup_results_array;
535 for (my $i=0;$i<@servers;$i++) {
536 my $server = $servers[$i];
537 if ($server =~/biblioserver/) { # this is the local bibliographic server
538 $hits = $results_hashref->{$server}->{"hits"};
539 my $page = $cgi->param('page') || 0;
540 my @newresults = searchResults('intranet', $query_desc, $hits, $results_per_page, $offset, $scan,
541 @{$results_hashref->{$server}->{"RECORDS"}});
542 $total = $total + $results_hashref->{$server}->{"hits"};
543 ## If there's just one result, redirect to the detail page
545 my $biblionumber = $newresults[0]->{biblionumber};
546 my $defaultview = C4::Context->preference('IntranetBiblioDefaultView');
547 my $views = { C4::Search::enabled_staff_search_views };
548 if ($defaultview eq 'isbd' && $views->{can_view_ISBD}) {
549 print $cgi->redirect("/cgi-bin/koha/catalogue/ISBDdetail.pl?biblionumber=$biblionumber");
550 } elsif ($defaultview eq 'marc' && $views->{can_view_MARC}) {
551 print $cgi->redirect("/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=$biblionumber");
552 } elsif ($defaultview eq 'labeled_marc' && $views->{can_view_labeledMARC}) {
553 print $cgi->redirect("/cgi-bin/koha/catalogue/labeledMARCdetail.pl?biblionumber=$biblionumber");
555 print $cgi->redirect("/cgi-bin/koha/catalogue/detail.pl?biblionumber=$biblionumber");
562 $template->param(total => $hits);
563 my $limit_cgi_not_availablity = $limit_cgi;
564 $limit_cgi_not_availablity =~ s/&limit=available//g;
565 $template->param(limit_cgi_not_availablity => $limit_cgi_not_availablity);
566 $template->param(limit_cgi => $limit_cgi);
567 $template->param(query_cgi => $query_cgi);
568 $template->param(query_desc => $query_desc);
569 $template->param(limit_desc => $limit_desc);
570 $template->param(offset => $offset);
571 $template->param(DisplayMultiPlaceHold => $DisplayMultiPlaceHold);
572 $template->param (z3950_search_params => C4::Search::z3950_search_args($query_desc));
573 if ($query_desc || $limit_desc) {
574 $template->param(searchdesc => 1);
576 $template->param(stopwords_removed => "@$stopwords_removed") if $stopwords_removed;
577 $template->param(results_per_page => $results_per_page);
578 $template->param(SEARCH_RESULTS => \@newresults);
580 ## FIXME: add a global function for this, it's better than the current global one
581 ## Build the page numbers on the bottom of the page
583 # total number of pages there will be
584 my $pages = ceil($hits / $results_per_page);
585 # default page number
586 my $current_page_number = 1;
587 $current_page_number = ($offset / $results_per_page + 1) if $offset;
588 my $previous_page_offset = $offset - $results_per_page unless ($offset - $results_per_page <0);
589 my $next_page_offset = $offset + $results_per_page;
590 # If we're within the first 10 pages, keep it simple
591 #warn "current page:".$current_page_number;
592 if ($current_page_number < 10) {
593 # just show the first 10 pages
594 # Loop through the pages
595 my $pages_to_show = 10;
596 $pages_to_show = $pages if $pages<10;
597 for (my $i=1; $i<=$pages_to_show;$i++) {
598 # the offset for this page
599 my $this_offset = (($i*$results_per_page)-$results_per_page);
600 # the page number for this page
601 my $this_page_number = $i;
602 # it should only be highlighted if it's the current page
603 my $highlight = 1 if ($this_page_number == $current_page_number);
604 # put it in the array
605 push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight, sort_by => join " ",@sort_by };
611 # now, show twenty pages, with the current one smack in the middle
613 for (my $i=$current_page_number; $i<=($current_page_number + 20 );$i++) {
614 my $this_offset = ((($i-9)*$results_per_page)-$results_per_page);
615 my $this_page_number = $i-9;
616 my $highlight = 1 if ($this_page_number == $current_page_number);
617 if ($this_page_number <= $pages) {
618 push @page_numbers, { offset => $this_offset, pg => $this_page_number, highlight => $highlight, sort_by => join " ",@sort_by };
622 # FIXME: no previous_page_offset when pages < 2
623 $template->param( PAGE_NUMBERS => \@page_numbers,
624 previous_page_offset => $previous_page_offset) unless $pages < 2;
625 $template->param( next_page_offset => $next_page_offset) unless $pages eq $current_page_number;
631 $template->param(searchdesc => 1,query_desc => $query_desc,limit_desc => $limit_desc);
632 $template->param (z3950_search_params => C4::Search::z3950_search_args($z3950par || $query_desc));
635 } # end of the if local
637 # asynchronously search the authority server
638 elsif ($server =~/authorityserver/) { # this is the local authority server
639 my @inner_sup_results_array;
640 for my $sup_record ( @{$results_hashref->{$server}->{"RECORDS"}} ) {
641 my $marc_record_object = MARC::Record->new_from_usmarc($sup_record);
642 # warn "Authority Found: ".$marc_record_object->as_formatted();
643 push @inner_sup_results_array, {
644 'title' => $marc_record_object->field(100)->subfield('a'),
645 'link' => "&idx=an&q=".$marc_record_object->field('001')->as_string(),
648 push @sup_results_array, { servername => $server,
649 inner_sup_results_loop => \@inner_sup_results_array} if @inner_sup_results_array;
651 # FIXME: can add support for other targets as needed here
652 $template->param( outer_sup_results_loop => \@sup_results_array);
653 } #/end of the for loop
654 #$template->param(FEDERATED_RESULTS => \@results_array);
658 #classlist => $classlist,
661 facets_loop => $facets,
662 displayFacetCount=> C4::Context->preference('displayFacetCount')||0,
664 search_error => $error,
667 if ($query_desc || $limit_desc) {
668 $template->param(searchdesc => 1);
671 # VI. BUILD THE TEMPLATE
673 # Build drop-down list for 'Add To:' menu...
675 my $row_count = 10; # FIXME:This probably should be a syspref
676 my ($pubshelves, $total) = GetRecentShelves(2, $row_count, undef);
677 my ($barshelves, $total) = GetRecentShelves(1, $row_count, $borrowernumber);
679 if (@{$pubshelves}) {
680 $template->param( addpubshelves => scalar @{$pubshelves});
681 $template->param( addpubshelvesloop => $pubshelves);
684 if (@{$barshelves}) {
685 $template->param( addbarshelves => scalar @{$barshelves});
686 $template->param( addbarshelvesloop => $barshelves);
691 output_html_with_http_headers $cgi, $cookie, $template->output;