From c873055829a359bda3505e887e53ba5cd368a0f9 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berick@esilibrary.com>
Date: Thu, 9 Jun 2011 09:17:04 -0400
Subject: [PATCH] added read support for holds history setting

---
 .../default/opac/myopac/prefs_settings.tt2         | 31 +++++++++++-----------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/Open-ILS/web/templates/default/opac/myopac/prefs_settings.tt2 b/Open-ILS/web/templates/default/opac/myopac/prefs_settings.tt2
index 85594f7551..cb5335ac24 100644
--- a/Open-ILS/web/templates/default/opac/myopac/prefs_settings.tt2
+++ b/Open-ILS/web/templates/default/opac/myopac/prefs_settings.tt2
@@ -15,11 +15,10 @@
         </div>
 
 
-        <table class="full-width" id="acct_search_main">
+        <table class="full-width data_grid" id="acct_search_main">
             <tbody>
-                <tr id='myopac_prefs_row'>
+                <tr >
                     <td width='20%'>[% l("Search hits per page") %]</td>
-
                     <td>
                         <div style="position:absolute">
                             <div style="position:relative;top:0px;left:55px;">
@@ -27,28 +26,30 @@
                                     src="[% ctx.media_prefix %]/images/question-mark.png" /></a>
                             </div>
                         </div>
-                        <select name='opac.hits_per_page'>
-                            [%  key = 'opac.hits_per_page';
-                                FOR val IN [5, 8, 10, 15, 20, 25, 50] %]
+                        [% setting = 'opac.hits_per_page' %]
+                        <select name='[% setting %]'>
+                            [%  FOR val IN [5, 8, 10, 15, 20, 25, 50] %]
                                 <option value='[% val %]' 
-                                    [% IF ctx.user_setting_map.$key == val %]
+                                    [% IF ctx.user_setting_map.$setting == val %]
                                         selected='selected'[% END %]>[% val %]</option>
                             [% END %]
                         </select>
                     </td>
                 </tr>
-
                 <tr>
+                    <td>[% l('Keep history of checked out items?') %]</td>
                     <td>
-                        [% l('Keep history of checked out items?') %]
+                        [% setting = 'history.circ.retention_start' %]
+                        <input name='[% setting %]' type="checkbox"
+                            [% IF ctx.user_setting_map.$setting; %] checked='checked' [% END %]/>
                     </td>
+                </tr>
+                <tr>
+                    <td>[% l('Keep history of holds?') %]</td>
                     <td>
-                        <input name='history.circ.retention_start' type="checkbox"
-                            [% 
-                                key = 'history.circ.retention_start';
-                                IF ctx.user_setting_map.$key; # any value
-                            %] checked='checked' [% END %]
-                        />
+                        [% setting = 'history.hold.retention_start' %]
+                        <input name='[% setting %]' type="checkbox"
+                            [% IF ctx.user_setting_map.$setting; %] checked='checked' [% END %]/>
                     </td>
                 </tr>
 
-- 
2.11.0