$Data::Dumper::Indent = 0;
my $U = 'OpenILS::Application::AppUtils';
-# when fetching "all" search results for staff client
-# start/end paging, fetch this many IDs at most
-my $all_recs_limit = 10000;
-
-
sub _prepare_biblio_search_basics {
my ($cgi) = @_;
next unless $query =~ /\S/;
+ # Hack for journal title
+ my $q = $qtype;
+ if ($q eq 'jtitle') {
+ $qtype = 'title';
+ }
+
# This stuff probably will need refined or rethought to better handle
# the weird things Real Users will surely type in.
$contains = "" unless defined $contains; # silence warning
}
$query = "$qtype:$query" unless $qtype eq 'keyword' and $i == 0;
+ # Hack for journal title - completed!
+ if ($q eq 'jtitle') {
+ $query = "bib_level:s $query";
+ }
+
$bool = ($bool and $bool eq 'or') ? '||' : '&&';
$full_query = $full_query ? "($full_query $bool $query)" : $query;
}
my $query = _prepare_biblio_search_basics($cgi) || '';
+ $query .= ' ' . $ctx->{global_search_filter} if $ctx->{global_search_filter};
+
foreach ($cgi->param('modifier')) {
# The unless bit is to avoid stacking modifiers.
$query = ('#' . $_ . ' ' . $query) unless $query =~ qr/\#\Q$_/;
$query .= " $1($term)" if length $term;
}
- # sort is treated specially, even though it's actually a filter
- if ($cgi->param('sort')) {
- $query =~ s/sort\([^\)]*\)//g; # override existing sort(). no stacking.
- my ($axis, $desc) = split /\./, $cgi->param('sort');
- $query .= " sort($axis)";
- if ($desc and not $query =~ /\#descending/) {
- $query .= '#descending';
- } elsif (not $desc) {
- $query =~ s/\#descending//;
- }
+ # filter group entries. Entries from like filters are grouped into a single
+ # filter_group_entry() filter (ORed). Each collection is ANDed together.
+ # fg:foo_group=foo_entry_id
+ foreach (grep /^fg:/, $cgi->param) {
+ /:(-?\w+)$/ or next;
+ my $term = join(",", $cgi->param($_));
+ $query .= " filter_group_entry($term)" if length $term;
+ }
+
+ if ($cgi->param("bookbag")) {
+ $query .= " container(bre,bookbag," . int($cgi->param("bookbag")) . ")";
}
if ($cgi->param('pubdate') && $cgi->param('date1')) {
}
}
- my $site;
+ # ---------------------------------------------------------------------
+ # Nothing below here constitutes a query by itself. If the query value
+ # is still empty up to this point, there is no query. abandon ship.
+ return () unless $query;
+
+ # sort is treated specially, even though it's actually a filter
+ if ($cgi->param('sort')) {
+ $query =~ s/sort\([^\)]*\)//g; # override existing sort(). no stacking.
+ my ($axis, $desc) = split /\./, $cgi->param('sort');
+ $query .= " sort($axis)";
+ if ($desc and not $query =~ /\#descending/) {
+ $query .= '#descending';
+ } elsif (not $desc) {
+ $query =~ s/\#descending//;
+ }
+ }
+
+ my (@naive_query_re, $site);
+
my $org = $ctx->{search_ou};
if (defined($org) and $org ne '' and ($org ne $ctx->{aou_tree}->()->id) and not $query =~ /site\(\S+\)/) {
- $site = $ctx->{get_aou}->($org)->shortname;
- $query .= " site($site)";
+ my $thing = " site(" . $ctx->{get_aou}->($org)->shortname . ")";
+
+ $query .= $thing;
+ push @naive_query_re, $thing;
+ }
+
+ my $pref_ou = $ctx->{pref_ou};
+ if (defined($pref_ou) and $pref_ou ne '' and $pref_ou != $org and ($pref_ou ne $ctx->{aou_tree}->()->id)) {
+ my $plib = $ctx->{get_aou}->($pref_ou)->shortname;
+ $query .= " pref_ou($plib)";
+ }
+
+ if (my $grp = $ctx->{copy_location_group}) {
+ $query .= " location_groups($grp)";
}
if(!$site) {
my ($org) = grep { $_->shortname eq $site } @{$ctx->{aou_list}->()};
$depth = $org->ou_type->depth;
}
- $query .= " depth($depth)";
+ my $thing = " depth($depth)";
+
+ $query .= $thing;
+ push @naive_query_re, $thing;
}
+ # This gives templates a way to take site() and depth() back out of
+ # query strings when they shouldn't be there (because they're controllable
+ # with other widgets).
+ $ctx->{naive_query_scrub} = sub {
+ my ($query) = @_;
+ $query =~ s/\Q$_\E// foreach (@naive_query_re);
+ return $query;
+ };
+
$logger->info("tpac: site=$site, depth=$depth, query=$query");
return ($query, $site, $depth);
return $limit if $limit;
if($self->editor->requestor) {
+ $self->timelog("Checking for opac.hits_per_page preference");
# See if the user has a hit count preference
my $lset = $self->editor->search_actor_user_setting({
usr => $self->editor->requestor->id,
name => 'opac.hits_per_page'
})->[0];
+ $self->timelog("Got opac.hits_per_page preference");
return OpenSRF::Utils::JSON->JSON2perl($lset->value) if $lset;
}
my $self = shift;
my $e = $self->editor;
+ $self->timelog("Tag circed items?");
return 0 unless $e->requestor;
+ $self->timelog("Checking for opac.search.tag_circulated_items");
return 0 unless $self->ctx->{get_org_setting}->(
$e->requestor->home_ou,
'opac.search.tag_circulated_items');
# user has to be opted-in to circ history in some capacity
+ $self->timelog("Checking for history.circ.retention_*");
my $sets = $e->search_actor_user_setting({
usr => $e->requestor->id,
name => [
]
});
+ $self->timelog("Return from checking for history.circ.retention_*");
+
return 0 unless @$sets;
return 1;
+
+}
+
+# This only loads the bookbag itself (in support of a record results page)
+# if a "bookbag" CGI parameter is specified and if the bookbag is public
+# or owned by the logged-in user (if any). Bookbag notes are fetched
+# later if applicable.
+sub load_rresults_bookbag {
+ my ($self) = @_;
+
+ my $bookbag_id = int($self->cgi->param("bookbag") || 0);
+ return if $bookbag_id < 1;
+
+ my %authz = $self->ctx->{"user"} ?
+ ("-or" => {"pub" => "t", "owner" => $self->ctx->{"user"}->id}) :
+ ("pub" => "t");
+
+ $self->timelog("Load results bookbag");
+ my $bbag = $self->editor->search_container_biblio_record_entry_bucket(
+ {"id" => $bookbag_id, "btype" => "bookbag", %authz}
+ );
+ $self->timelog("Got results bookbag");
+
+ if (!$bbag) {
+ $self->apache->log->warn(
+ "error from cstore retrieving bookbag $bookbag_id!"
+ );
+ return Apache2::Const::HTTP_INTERNAL_SERVER_ERROR;
+ } elsif (@$bbag) {
+ $self->ctx->{"bookbag"} = shift @$bbag;
+ }
+
+ return;
+}
+
+# assumes context has a bookbag we're already authorized to look at, and
+# a list of rec_ids, reasonably sized (from paged search).
+sub load_rresults_bookbag_item_notes {
+ my ($self, $rec_ids) = @_;
+
+ $self->timelog("Load results bookbag item notes");
+ my $items_with_notes =
+ $self->editor->search_container_biblio_record_entry_bucket_item([
+ {"target_biblio_record_entry" => $rec_ids,
+ "bucket" => $self->ctx->{"bookbag"}->id},
+ {"flesh" => 1, "flesh_fields" => {"cbrebi" => ["notes"]},
+ "order_by" => {"cbrebi" => ["id"]}}
+ ]);
+ $self->timelog("Got results bookbag item notes");
+
+ if (!$items_with_notes) {
+ $self->apache->log->warn("error from cstore retrieving cbrebi objects");
+ return Apache2::Const::HTTP_INTERNAL_SERVER_ERROR;
+ }
+
+ $self->ctx->{"bookbag_items_by_bre_id"} = +{
+ map { $_->target_biblio_record_entry => $_ } @$items_with_notes
+ };
+
+ return;
}
# context additions:
my $ctx = $self->ctx;
my $e = $self->editor;
+ # find the last record in the set, then redirect
+ my $find_last = $cgi->param('find_last');
+
+ $self->timelog("Loading results");
+ # load bookbag metadata, if requested.
+ if (my $bbag_err = $self->load_rresults_bookbag) {
+ return $bbag_err;
+ }
+
$ctx->{page} = 'rresult' unless $internal;
$ctx->{ids} = [];
$ctx->{records} = [];
# Special alternative searches here. This could all stand to be cleaner.
if ($cgi->param("_special")) {
+ $self->timelog("Calling MARC expert search");
return $self->marc_expert_search(%args) if scalar($cgi->param("tag"));
+ $self->timelog("Calling item barcode search");
return $self->item_barcode_shortcut if (
$cgi->param("qtype") and ($cgi->param("qtype") eq "item_barcode")
);
+ $self->timelog("Calling call number browse");
return $self->call_number_browse_standalone if (
$cgi->param("qtype") and ($cgi->param("qtype") eq "cnbrowse")
);
}
+ $self->timelog("Getting search parameters");
my $page = $cgi->param('page') || 0;
my @facets = $cgi->param('facet');
my $limit = $self->_get_search_limit;
$ctx->{search_ou} = $self->_get_search_lib();
+ $ctx->{pref_ou} = $self->_get_pref_lib() || $ctx->{search_ou};
my $offset = $page * $limit;
my $metarecord = $cgi->param('metarecord');
my $results;
my $tag_circs = $self->tag_circed_items;
+ $self->timelog("Got search parameters");
$ctx->{page_size} = $limit;
$ctx->{search_page} = $page;
- # fetch the first hit from the next page
+ # fetch this page plus the first hit from the next page
if ($internal) {
- $limit = $all_recs_limit;
+ $limit = $offset + $limit + 1;
$offset = 0;
}
$self->get_staff_search_settings;
- if ($ctx->{staff_saved_search_size}) {
+ if (!$find_last and $ctx->{staff_saved_search_size}) {
my ($key, $list) = $self->staff_save_search($query);
if ($key) {
$self->apache->headers_out->add(
if ($metarecord and !$internal) {
# TODO: other limits, like SVF/format, etc.
+ $self->timelog("Getting metarecords to records");
$results = $U->simplereq(
'open-ils.search',
'open-ils.search.biblio.metarecord_to_records',
$metarecord, {org => $ctx->{search_ou}, depth => $depth}
);
+ $self->timelog("Got metarecords to records");
# force the metarecord result blob to match the format of regular search results
$results->{ids} = [map { [$_] } @{$results->{ids}}];
my $method = 'open-ils.search.biblio.multiclass.query';
$method .= '.staff' if $ctx->{is_staff};
+
+ $self->timelog("Firing off the multiclass query");
$results = $U->simplereq('open-ils.search', $method, $args, $query, 1);
+ $self->timelog("Returned from the multiclass query");
} catch Error with {
my $err = shift;
$ctx->{hit_count} = $results->{count};
$ctx->{parsed_query} = $results->{parsed_query};
+ if ($find_last) {
+ # redirect to the record detail page for the last record in the results
+ my $rec_id = pop @$rec_ids;
+ $cgi->delete('find_last');
+ my $url = $cgi->url(-full => 1, -path => 1, -query => 1);
+ $url =~ s|/results|/record/$rec_id|;
+ return $self->generic_redirect($url);
+ }
+
return Apache2::Const::OK if @$rec_ids == 0 or $internal;
+ $self->load_rresults_bookbag_item_notes($rec_ids) if $ctx->{bookbag};
+
+ $self->timelog("Calling get_records_and_facets()");
my ($facets, @data) = $self->get_records_and_facets(
$rec_ids, $results->{facet_key},
{
flesh => '{holdings_xml,mra,acp,acnp,acns,bmp}',
site => $site,
- depth => $depth
+ depth => $depth,
+ pref_lib => $ctx->{pref_ou},
}
);
+ $self->timelog("Returned from get_records_and_facets()");
if ($page == 0) {
my $stat = $self->check_1hit_redirect($rec_ids);
return $stat if $stat;
}
+ # load temporary_list settings for user and ou:
+ $self->_load_lists_and_settings if ($ctx->{user});
+
# shove recs into context in search results order
for my $rec_id (@$rec_ids) {
push(
my ($sname, $org);
+ $self->timelog("Checking whether to jump to details on a single hit");
if ($ctx->{is_staff}) {
$sname = 'opac.staff.jump_to_details_on_single_hit';
$org = $ctx->{user}->ws_ou;
$org = $self->_get_search_lib();
}
+ $self->timelog("Return from checking whether to jump to details on a single hit");
+
return undef unless
$self->ctx->{get_org_setting}->($org, $sname);
my $base_url = sprintf(
'%s://%s%s/record/%s',
- $ctx->{proto},
- $self->apache->hostname,
+ $self->ctx->{proto},
+ $self->ctx->{hostname},
$self->ctx->{opac_root},
$$rec_ids[0],
);
# If we get here from the same record detail page to which we
# now wish to redirect, do not perform the redirect. This
# approach seems to work well, with the rare exception of
- # performing a new serach directly from the detail page that
+ # performing a new search directly from the detail page that
# happens to result in the same single hit. In this case, the
# user will be left on the search results page. This could be
# overcome w/ additional CGI, etc., but I'm not sure it's necessary.
sub item_barcode_shortcut {
my ($self) = @_;
+ $self->timelog("Searching for item_barcode");
my $method = "open-ils.search.multi_home.bib_ids.by_barcode";
if (my $search = create OpenSRF::AppSession("open-ils.search")) {
my $rec_ids = $search->request(
$method, $self->cgi->param("query")
)->gather(1);
$search->kill_me;
+ $self->timelog("Finished searching for item_barcode");
if (ref $rec_ids ne 'ARRAY') {
}
}
+ $self->timelog("Calling get_records_and_facets() for item_barcode");
my ($facets, @data) = $self->get_records_and_facets(
$rec_ids, undef, {flesh => "{holdings_xml,mra,acnp,acns,bmp}"}
);
+ $self->timelog("Returned from calling get_records_and_facets() for item_barcode");
$self->ctx->{records} = [@data];
$self->ctx->{search_facets} = {};
$self->ctx->{hit_count} = scalar @data;
$self->ctx->{page_size} = $self->ctx->{hit_count};
+ # load temporary_list settings for user and ou:
+ $self->_load_lists_and_settings if ($self->ctx->{user});
return Apache2::Const::OK;
} {
$logger->info("query for expert search: " . Dumper($query));
+ $self->timelog("Getting search parameters");
# loc, limit and offset
my $page = $self->cgi->param("page") || 0;
my $limit = $self->_get_search_limit;
$self->ctx->{search_ou} = $self->_get_search_lib();
+ $self->ctx->{pref_ou} = $self->_get_pref_lib();
my $offset = $page * $limit;
+ $self->timelog("Got search parameters");
$self->ctx->{records} = [];
$self->ctx->{search_facets} = {};
return Apache2::Const::OK if @$query == 0;
if ($args{internal}) {
- $limit = $all_recs_limit;
+ $limit = $offset + $limit + 1;
$offset = 0;
}
+ $self->timelog("Searching for MARC expert");
+ my $method = 'open-ils.search.biblio.marc';
+ $method .= '.staff' if $self->ctx->{is_staff};
my $timeout = 120;
my $ses = OpenSRF::AppSession->create('open-ils.search');
my $req = $ses->request(
- 'open-ils.search.biblio.marc',
+ $method,
{searches => $query, org_unit => $self->ctx->{search_ou}},
$limit, $offset, $timeout);
my $resp = $req->recv($timeout);
my $results = $resp ? $resp->content : undef;
$ses->kill_me;
+ $self->timelog("Got our MARC expert results");
if (defined $U->event_code($results)) {
$self->apache->log->warn(
return $stat if $stat;
}
+ $self->timelog("Calling get_records_and_facets() for MARC expert");
my ($facets, @data) = $self->get_records_and_facets(
- $self->ctx->{ids}, undef, {flesh => "{holdings_xml,mra,acnp,acns}"}
+ $self->ctx->{ids}, undef, {
+ flesh => "{holdings_xml,mra,acnp,acns}",
+ pref_lib => $self->ctx->{pref_ou},
+ }
);
+ $self->timelog("Returned from calling get_records_and_facets() for MARC expert");
+
+ # load temporary_list settings for user and ou:
+ $self->_load_lists_and_settings if ($self->ctx->{user});
$self->ctx->{records} = [@data];
if (my $cnfrag = $self->cgi->param("query")) {
my $url = sprintf(
- 'http%s://%s%s/cnbrowse?cn=%s',
- $self->cgi->https ? "s" : "",
- $self->apache->hostname,
+ '%s://%s%s/cnbrowse?cn=%s',
+ $self->ctx->{proto},
+ $self->ctx->{hostname},
$self->ctx->{opac_root},
$cnfrag # XXX some kind of escaping needed here?
);
+ $url .= '&locg=' . $self->_get_search_lib() if ($self->_get_search_lib());
return $self->generic_redirect($url);
} else {
return $self->generic_redirect; # return to search page
return;
}
+ $self->timelog("Getting staff search size");
my $sss_size = $self->ctx->{get_org_setting}->(
$self->ctx->{physical_loc} || $self->ctx->{aou_tree}->()->id,
"opac.staff_saved_search.size",
);
+ $self->timelog("Got staff search size");
# Sic: 0 is 0 (off), but undefined is 10.
$sss_size = 10 unless defined $sss_size;
my $list = [];
if ($cache_key) {
+ $self->timelog("Getting anon_cache value");
$list = $U->simplereq(
"open-ils.actor",
"open-ils.actor.anon_cache.get_value",
$cache_key, (ref $self)->ANON_CACHE_STAFF_SEARCH
);
+ $self->timelog("Got anon_cache value");
unless ($list) {
undef $cache_key;
unshift @$list, $query unless $already{$query};
- splice @$list, $sss_size;
+ splice @$list, $sss_size if scalar @$list > $sss_size;
+ $self->timelog("Setting anon_cache value");
$cache_key = $U->simplereq(
"open-ils.actor",
"open-ils.actor.anon_cache.set_value",
$cache_key, (ref $self)->ANON_CACHE_STAFF_SEARCH, $list
);
+ $self->timelog("Set anon_cache value");
return ($cache_key, $list);
}