From: berick <berick@esilibrary.com>
Date: Wed, 20 Apr 2011 15:01:45 +0000 (-0400)
Subject: format selector gets formats from CCVM fields, starting with 'mattype', falling back... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=470bf347f88c1a80ae641c40aaf99b337e74c51f;p=evergreen%2Fequinox.git

format selector gets formats from CCVM fields, starting with 'mattype', falling back to item_type
---

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
index 9577121159..524c9a6b45 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Search.pm
@@ -51,6 +51,7 @@ sub _prepare_biblio_search {
 
     $query = ('#' . $_ . ' ' . $query) foreach ($cgi->param('modifier'));
 
+    # filters
     foreach (grep /^fi:/, $cgi->param) {
         /:(\w+)$/ or next;
         my $term = join(",", $cgi->param($_));
@@ -128,6 +129,8 @@ sub load_rresults {
 
     $query = "$query $facet" if $facet; # TODO
 
+    $logger->activity("EGWeb: [search] $query");
+
     my $results;
 
     try {
diff --git a/Open-ILS/web/templates/default/opac/parts/format_selector.tt2 b/Open-ILS/web/templates/default/opac/parts/format_selector.tt2
index 2ca397b053..d52129b399 100644
--- a/Open-ILS/web/templates/default/opac/parts/format_selector.tt2
+++ b/Open-ILS/web/templates/default/opac/parts/format_selector.tt2
@@ -1,12 +1,23 @@
-[%-  name = name || "fi:format";
+[% 
+    # Start w/ mattype, fall back to item_type
+    format_spec = '';
+    formats = [];
+    FOR format_spec IN ['mattype', 'item_type'];
+        formats = ctx.search_ccvm('ctype', format_spec);
+        IF formats.size; BREAK; END;
+    END;
+    name = name || "fi:" _ format_spec;
     id = id || "format_selector";
-    values = values || CGI.param(name) -%]
+    values = values || CGI.param(name) 
+-%]
+
 <select id='[% id %]' name='[% name %]'[%
     multiple ? ' multiple="multiple"' : '';
     size ? (' size="' _ size _ '"') : ''; %]>
     <option value=''>[% l("All Formats") %]</option>
+
 [% FOR o IN formats %]
-    <option value='[% o.code %]'[% values.grep('^' _ o.code _ '$').size ? ' selected="selected"' : ''%]>[% o.name %]</option>
+    <option value='[% o.code %]'[% values.grep('^' _ o.code _ '$').size ? ' selected="selected"' : ''%]>[% o.value %]</option>
 [%- END %]
 <!--
 	<option value='at'>[% l("Books") %]</option>