kpac : search link cgi improvements
authorBill Erickson <berick@esilibrary.com>
Mon, 9 Apr 2012 19:45:49 +0000 (15:45 -0400)
committerMike Rylander <mrylander@gmail.com>
Mon, 30 Jul 2012 19:03:22 +0000 (15:03 -0400)
1. avoid showing the search terms from canned searches in the simple
search box.  Patrons, kids especially, probably don't want to see
potentially complicated canned search queries in the search box when the
whole point is that they are accessible by clicking on images.

2. Avoid propagating the search qtype param to canned searches.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/src/templates/kpac/parts/grid.tt2
Open-ILS/src/templates/kpac/parts/searchbox.tt2

index 2e1b459..3d9722c 100644 (file)
@@ -27,7 +27,8 @@ MACRO render_kpac_grid(page_id) BLOCK;
 
         [% ELSIF cell.type == 'search'; %]
 
-        <td width="[% width %]%"><a href="[% mkurl(ctx.kpac_root _ '/results', {query => cell.content}, ['page']) %]"><img 
+        <td width="[% width %]%"><a href="[% 
+            mkurl(ctx.kpac_root _ '/results', {query => cell.content, nosprop => 1}, ['page', 'qtype']) %]"><img 
             src="[% kpac_img_path(cell.img) %]" alt="" /><span>[% cell.name %]</span></a></td>
 
         [% ELSIF cell.type == 'link'; %]
index 1d6e39e..8f598d5 100644 (file)
@@ -1,7 +1,8 @@
 <div class="sub_sidebar_header">[% l('New Search') %]</div>
 <form action="[% ctx.kpac_root _ '/results' %]">
     <div class="sub_sidebar_search_wrapper">
-        <input type="text" name='query' value="[% CGI.param('query') | html %]"
+        <input type="text" name='query' 
+            value="[% CGI.param('nosprop') ? '' : CGI.param('query') | html %]"
             [% seed = l(' Enter Text... ') | html %]
             class="sub_sidebar_search_box" style="color:#aaa;" value="[% seed %]"
             onfocus="if(this.value=='[% seed %]'){this.value='';this.style.color='#424242';}"