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;
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);
161 my $lang = C4::Templates::getlanguage($cgi, 'intranet');
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 my $itemtypes = GetItemTypes;
243 # the index parameter is different for item-level itemtypes
244 my $itype_or_itemtype = (C4::Context->preference("item-level_itypes"))?'itype':'itemtype';
245 my @advancedsearchesloop;
247 my $advanced_search_types = C4::Context->preference("AdvancedSearchTypes") || "itemtypes";
248 my @advanced_search_types = split(/\|/, $advanced_search_types);
250 foreach my $advanced_srch_type (@advanced_search_types) {
251 if ($advanced_srch_type eq 'itemtypes') {
252 # itemtype is a special case, since it's not defined in authorized values
254 foreach my $thisitemtype ( sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'} } keys %$itemtypes ) {
255 my %row =( number=>$cnt++,
256 ccl => "$itype_or_itemtype,phr",
257 code => $thisitemtype,
258 description => $itemtypes->{$thisitemtype}->{'description'},
259 imageurl=> getitemtypeimagelocation( 'intranet', $itemtypes->{$thisitemtype}->{'imageurl'} ),
261 push @itypesloop, \%row;
263 my %search_code = ( advanced_search_type => $advanced_srch_type,
264 code_loop => \@itypesloop );
265 push @advancedsearchesloop, \%search_code;
267 # covers all the other cases: non-itemtype authorized values
268 my $advsearchtypes = GetAuthorisedValues($advanced_srch_type);
270 for my $thisitemtype (@$advsearchtypes) {
273 ccl => $advanced_srch_type,
274 code => $thisitemtype->{authorised_value},
275 description => $thisitemtype->{'lib'},
276 imageurl => getitemtypeimagelocation( 'intranet', $thisitemtype->{'imageurl'} ),
278 push @authvalueloop, \%row;
280 my %search_code = ( advanced_search_type => $advanced_srch_type,
281 code_loop => \@authvalueloop );
282 push @advancedsearchesloop, \%search_code;
285 $template->param(advancedsearchesloop => \@advancedsearchesloop);
287 # The following should only be loaded if we're bringing up the advanced search template
288 if ( $template_type eq 'advsearch' ) {
290 # load the servers (used for searching -- to do federated searching, etc.)
291 my $primary_servers_loop;# = displayPrimaryServers();
292 $template->param(outer_servers_loop => $primary_servers_loop,);
294 my $secondary_servers_loop;
295 $template->param(outer_sup_servers_loop => $secondary_servers_loop,);
297 # set the default sorting
298 if ( C4::Context->preference('OPACdefaultSortField')
299 && C4::Context->preference('OPACdefaultSortOrder') ) {
300 my $default_sort_by =
301 C4::Context->preference('defaultSortField') . '_'
302 . C4::Context->preference('defaultSortOrder');
303 $template->param( sort_by => $default_sort_by );
306 # determine what to display next to the search boxes (ie, boolean option
307 # shouldn't appear on the first one, scan indexes should, adding a new
308 # box should only appear on the last, etc.
309 my @search_boxes_array;
310 my $search_boxes_count = 3; # begin whith 3 boxes
311 # FIXME: all this junk can be done in TMPL using __first__ and __last__
312 for (my $i=1;$i<=$search_boxes_count;$i++) {
313 # if it's the first one, don't display boolean option, but show scan indexes
315 push @search_boxes_array, {scan_index => 1};
317 # if it's the last one, show the 'add field' box
318 elsif ($i==$search_boxes_count) {
319 push @search_boxes_array,
326 push @search_boxes_array,
333 $template->param(uc(C4::Context->preference("marcflavour")) => 1,
334 search_boxes_loop => \@search_boxes_array);
336 # load the language limits (for search)
337 my $languages_limit_loop = getAllLanguages($lang);
338 $template->param(search_languages_loop => $languages_limit_loop,);
340 # Expanded search options in advanced search:
341 # use the global setting by default, but let the user override it
343 my $expanded = $cgi->param('expanded_options');
344 $expanded = C4::Context->preference("expandedSearchOption") || 0
345 if !defined($expanded) || $expanded !~ /^0|1$/;
346 $template->param( expanded_options => $expanded );
349 $template->param(virtualshelves => C4::Context->preference("virtualshelves"));
351 output_html_with_http_headers $cgi, $cookie, $template->output;
355 ### OK, if we're this far, we're performing a search, not just loading the advanced search page
357 # Fetch the paramater list as a hash in scalar context:
358 # * returns paramater list as tied hash ref
359 # * we can edit the values by changing the key
360 # * multivalued CGI paramaters are returned as a packaged string separated by "\0" (null)
361 my $params = $cgi->Vars;
363 # Params that can have more than one value
364 # sort by is used to sort the query
365 # in theory can have more than one but generally there's just one
368 if ( C4::Context->preference('defaultSortField')
369 && C4::Context->preference('defaultSortOrder') ) {
371 C4::Context->preference('defaultSortField') . '_'
372 . C4::Context->preference('defaultSortOrder');
375 @sort_by = $cgi->param('sort_by');
376 $sort_by[0] = $default_sort_by unless $sort_by[0];
377 foreach my $sort (@sort_by) {
378 $template->param($sort => 1) if $sort;
380 $template->param('sort_by' => $sort_by[0]);
382 # Use the servers defined, or just search our local catalog(default)
383 my @servers = $cgi->param('server');
385 #FIXME: this should be handled using Context.pm
386 @servers = ("biblioserver");
387 # @servers = C4::Context->config("biblioserver");
389 # operators include boolean and proximity operators and are used
390 # to evaluate multiple operands
391 my @operators = $cgi->param('op');
393 # indexes are query qualifiers, like 'title', 'author', etc. They
394 # can be single or multiple parameters separated by comma: kw,right-Truncation
395 my @indexes = $cgi->param('idx');
397 # if a simple index (only one) display the index used in the top search box
398 if ($indexes[0] && (!$indexes[1] || $params->{'scan'})) {
399 my $idx = "ms_".$indexes[0];
400 $idx =~ s/\,/comma/g; # template toolkit doesnt like variables with a , in it
401 $template->param($idx => 1);
405 # an operand can be a single term, a phrase, or a complete ccl query
406 my @operands = $cgi->param('q');
408 # limits are use to limit to results to a pre-defined category such as branch or language
409 my @limits = $cgi->param('limit');
411 if($params->{'multibranchlimit'}) {
412 push @limits, '('.join( " or ", map { "branch: $_ " } @{ GetBranchesInCategory( $params->{'multibranchlimit'} ) } ).')';
416 foreach my $limit(@limits) {
417 if ($limit =~/available/) {
421 $template->param(available => $available);
423 # append year limits if they exist
426 if ($params->{'limit-yr'}) {
427 if ($params->{'limit-yr'} =~ /\d{4}-\d{4}/) {
428 my ($yr1,$yr2) = split(/-/, $params->{'limit-yr'});
429 $limit_yr = "yr,st-numeric,ge=$yr1 and yr,st-numeric,le=$yr2";
430 $limit_yr_value = "$yr1-$yr2";
432 elsif ($params->{'limit-yr'} =~ /\d{4}/) {
433 $limit_yr = "yr,st-numeric=$params->{'limit-yr'}";
434 $limit_yr_value = $params->{'limit-yr'};
436 push @limits,$limit_yr;
437 #FIXME: Should return a error to the user, incorect date format specified
440 # convert indexes and operands to corresponding parameter names for the z3950 search
441 # $ %z3950p will be a hash ref if the indexes are present (advacned search), otherwise undef
443 my $indexes2z3950 = {
444 kw=>'title', au=>'author', 'au,phr'=>'author', nb=>'isbn', ns=>'issn',
445 'lcn,phr'=>'dewey', su=>'subject', 'su,phr'=>'subject',
446 ti=>'title', 'ti,phr'=>'title', se=>'title'
448 for (my $ii = 0; $ii < @operands; ++$ii)
450 my $name = $indexes2z3950->{$indexes[$ii]};
451 if (defined $name && defined $operands[$ii])
454 $z3950par->{$name} = $operands[$ii] if !exists $z3950par->{$name};
459 # Params that can only have one value
460 my $scan = $params->{'scan'};
461 my $count = C4::Context->preference('numSearchResults') || 20;
462 my $results_per_page = $params->{'count'} || $count;
463 my $offset = $params->{'offset'} || 0;
464 my $page = $cgi->param('page') || 1;
465 #my $offset = ($page-1)*$results_per_page;
467 my $expanded_facet = $params->{'expand'};
469 # Define some global variables
470 my ( $error,$query,$simple_query,$query_cgi,$query_desc,$limit,$limit_cgi,$limit_desc,$stopwords_removed,$query_type);
474 ## I. BUILD THE QUERY
475 ( $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);
477 ## parse the query_cgi string and put it into a form suitable for <input>s
479 my $scan_index_to_use;
481 for my $this_cgi ( split('&',$query_cgi) ) {
482 next unless $this_cgi;
483 $this_cgi =~ m/(.*?)=(.*)/;
485 my $input_value = $2;
486 push @query_inputs, { input_name => $input_name, input_value => $input_value };
487 if ($input_name eq 'idx') {
488 $scan_index_to_use = $input_value; # unless $scan_index_to_use;
491 $template->param ( QUERY_INPUTS => \@query_inputs,
492 scan_index_to_use => $scan_index_to_use );
494 ## parse the limit_cgi string and put it into a form suitable for <input>s
496 for my $this_cgi ( split('&',$limit_cgi) ) {
497 next unless $this_cgi;
498 # handle special case limit-yr
499 if ($this_cgi =~ /yr,st-numeric/) {
500 push @limit_inputs, { input_name => 'limit-yr', input_value => $limit_yr_value };
503 $this_cgi =~ m/(.*=)(.*)/;
505 my $input_value = $2;
506 $input_name =~ s/=$//;
507 push @limit_inputs, { input_name => $input_name, input_value => $input_value };
509 $template->param ( LIMIT_INPUTS => \@limit_inputs );
511 ## II. DO THE SEARCH AND GET THE RESULTS
512 my $total; # the total results for the whole set
513 my $facets; # this object stores the faceted results that display on the left-hand of the results page
517 if (C4::Context->preference('NoZebra')) {
518 $query=~s/yr(:|=)\s*([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
519 $simple_query=~s/yr\s*(:|=)([\d]{1,4})-([\d]{1,4})/(yr>=$2 and yr<=$3)/g;
522 ($error, $results_hashref, $facets) = NZgetRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$query_type,$scan);
526 ($error, $results_hashref, $facets) = getRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$itemtypes,$query_type,$scan);
529 # This sorts the facets into alphabetical order
531 foreach my $f (@$facets) {
532 $f->{facets} = [ sort { uc($a->{facet_title_value}) cmp uc($b->{facet_title_value}) } @{ $f->{facets} } ];
536 $template->param(query_error => $error.$@);
537 output_html_with_http_headers $cgi, $cookie, $template->output;
541 # At this point, each server has given us a result set
542 # now we build that set for template display
543 my @sup_results_array;
544 for (my $i=0;$i<@servers;$i++) {
545 my $server = $servers[$i];
546 if ($server =~/biblioserver/) { # this is the local bibliographic server
547 $hits = $results_hashref->{$server}->{"hits"};
548 my $page = $cgi->param('page') || 0;
549 my @newresults = searchResults('intranet', $query_desc, $hits, $results_per_page, $offset, $scan,
550 $results_hashref->{$server}->{"RECORDS"});
551 $total = $total + $results_hashref->{$server}->{"hits"};
552 ## If there's just one result, redirect to the detail page
554 my $biblionumber = $newresults[0]->{biblionumber};
555 my $defaultview = C4::Context->preference('IntranetBiblioDefaultView');
556 my $views = { C4::Search::enabled_staff_search_views };
557 if ($defaultview eq 'isbd' && $views->{can_view_ISBD}) {
558 print $cgi->redirect("/cgi-bin/koha/catalogue/ISBDdetail.pl?biblionumber=$biblionumber");
559 } elsif ($defaultview eq 'marc' && $views->{can_view_MARC}) {
560 print $cgi->redirect("/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=$biblionumber");
561 } elsif ($defaultview eq 'labeled_marc' && $views->{can_view_labeledMARC}) {
562 print $cgi->redirect("/cgi-bin/koha/catalogue/labeledMARCdetail.pl?biblionumber=$biblionumber");
564 print $cgi->redirect("/cgi-bin/koha/catalogue/detail.pl?biblionumber=$biblionumber");
571 $template->param(total => $hits);
572 my $limit_cgi_not_availablity = $limit_cgi;
573 $limit_cgi_not_availablity =~ s/&limit=available//g;
574 $template->param(limit_cgi_not_availablity => $limit_cgi_not_availablity);
575 $template->param(limit_cgi => $limit_cgi);
576 $template->param(query_cgi => $query_cgi);
577 $template->param(query_desc => $query_desc);
578 $template->param(limit_desc => $limit_desc);
579 $template->param(offset => $offset);
580 $template->param(DisplayMultiPlaceHold => $DisplayMultiPlaceHold);
581 $template->param (z3950_search_params => C4::Search::z3950_search_args($query_desc));
582 if ($query_desc || $limit_desc) {
583 $template->param(searchdesc => 1);
585 $template->param(stopwords_removed => "@$stopwords_removed") if $stopwords_removed;
586 $template->param(results_per_page => $results_per_page);
587 # must define a value for size if not present in DB
588 # in order to avoid problems generated by the default size value in TT
589 foreach my $line (@newresults) {
590 if ( not exists $line->{'size'} ) { $line->{'size'} = "" }
592 $template->param(SEARCH_RESULTS => \@newresults);
593 ## FIXME: add a global function for this, it's better than the current global one
594 ## Build the page numbers on the bottom of the page
596 # total number of pages there will be
597 my $pages = ceil($hits / $results_per_page);
598 # default page number
599 my $current_page_number = 1;
600 $current_page_number = ($offset / $results_per_page + 1) if $offset;
601 my $previous_page_offset;
602 if ( $offset >= $results_per_page ) {
603 $previous_page_offset = $offset - $results_per_page;
605 my $next_page_offset = $offset + $results_per_page;
606 # If we're within the first 10 pages, keep it simple
607 #warn "current page:".$current_page_number;
608 if ($current_page_number < 10) {
609 # just show the first 10 pages
610 # Loop through the pages
611 my $pages_to_show = 10;
612 $pages_to_show = $pages if $pages<10;
613 for (my $i=1; $i<=$pages_to_show;$i++) {
614 # the offset for this page
615 my $this_offset = (($i*$results_per_page)-$results_per_page);
616 # the page number for this page
617 my $this_page_number = $i;
618 # put it in the array
620 { offset => $this_offset,
621 pg => $this_page_number,
622 # it should only be highlighted if it's the current page
623 highlight => $this_page_number == $current_page_number,
624 sort_by => join ' ', @sort_by
631 # now, show twenty pages, with the current one smack in the middle
633 for (my $i=$current_page_number; $i<=($current_page_number + 20 );$i++) {
634 my $this_offset = ((($i-9)*$results_per_page)-$results_per_page);
635 my $this_page_number = $i-9;
636 if ( $this_page_number <= $pages ) {
638 { offset => $this_offset,
639 pg => $this_page_number,
640 highlight => $this_page_number == $current_page_number,
641 sort_by => join ' ', @sort_by
646 # FIXME: no previous_page_offset when pages < 2
647 $template->param( PAGE_NUMBERS => \@page_numbers,
648 previous_page_offset => $previous_page_offset) unless $pages < 2;
649 $template->param( next_page_offset => $next_page_offset) unless $pages eq $current_page_number;
655 $template->param(searchdesc => 1,query_desc => $query_desc,limit_desc => $limit_desc);
656 $template->param (z3950_search_params => C4::Search::z3950_search_args($z3950par || $query_desc));
659 } # end of the if local
661 # asynchronously search the authority server
662 elsif ($server =~/authorityserver/) { # this is the local authority server
663 my @inner_sup_results_array;
664 for my $sup_record ( @{$results_hashref->{$server}->{"RECORDS"}} ) {
665 my $marc_record_object = MARC::Record->new_from_usmarc($sup_record);
666 # warn "Authority Found: ".$marc_record_object->as_formatted();
667 push @inner_sup_results_array, {
668 'title' => $marc_record_object->field(100)->subfield('a'),
669 'link' => "&idx=an&q=".$marc_record_object->field('001')->as_string(),
672 push @sup_results_array, { servername => $server,
673 inner_sup_results_loop => \@inner_sup_results_array} if @inner_sup_results_array;
675 # FIXME: can add support for other targets as needed here
676 $template->param( outer_sup_results_loop => \@sup_results_array);
677 } #/end of the for loop
678 #$template->param(FEDERATED_RESULTS => \@results_array);
682 #classlist => $classlist,
685 facets_loop => $facets,
686 displayFacetCount=> C4::Context->preference('displayFacetCount')||0,
688 search_error => $error,
691 if ($query_desc || $limit_desc) {
692 $template->param(searchdesc => 1);
695 # VI. BUILD THE TEMPLATE
697 # Build drop-down list for 'Add To:' menu...
698 my ($totalref, $pubshelves, $barshelves)=
699 C4::VirtualShelves::GetSomeShelfNames($borrowernumber,'COMBO',1);
701 addbarshelves => $totalref->{bartotal},
702 addbarshelvesloop => $barshelves,
703 addpubshelves => $totalref->{pubtotal},
704 addpubshelvesloop => $pubshelves,
707 output_html_with_http_headers $cgi, $cookie, $template->output;