From 4c58011f4821a8d8ff84006dfc0c87b434a27865 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Wed, 13 Jul 2011 15:45:38 -0400
Subject: [PATCH] Fix for 6426 - messaging not showing on patron categories

T:T variable name problems were causing the problem.

I used this opportunity to change the way the data is
output: Now the messaging preferences are only output
if there is a current setting.

Also corrected the js table sorter configuration for
the right columns and moved the template-based table
row striping to the tablesorter plugin config.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
---
 .../prog/en/modules/admin/categorie.tt        | 38 +++++++++----------
 1 file changed, 19 insertions(+), 19 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt
index a916a4bf8e..c2e5062c7a 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt
@@ -9,7 +9,8 @@
 <script type="text/javascript" id="js">$(document).ready(function() {
 	$("#table_categorie").tablesorter({
 		sortList: [[0,0]],
-		headers: { 10: { sorter: false}}
+		widgets: ['zebra'],
+		headers: { 11: { sorter: false}}
 	}).tablesorterPager({container: $("#pagertable_categorie"),positionFixed: false,size: 20});
 }); </script>
 [% INCLUDE 'calendar.inc' %]
@@ -300,11 +301,7 @@ Confirm Deletion of Category [% categorycode |html %][% END %]</legend>
 			<th scope="col" colspan="2">&nbsp; </th>
 		</thead>
 		[% FOREACH loo IN loop %]
-		[% UNLESS ( loop.odd ) %]
-			<tr class="highlight">
-		[% ELSE %]
 			<tr>
-		[% END %]
                         <td>[% loo.categorycode |html %]</td>
                         <td>
                             <a href="[% loo.script_name %]?op=add_form&amp;categorycode=[% loo.categorycode |url %]">[% loo.description |html %]</a>
@@ -332,22 +329,25 @@ Confirm Deletion of Category [% categorycode |html %][% END %]</legend>
                         <td>[% IF ( loo.hidelostitems ) %]Hidden[% ELSE %]Shown[% END %]</td>
                         <td>[% loo.reservefee %]</td>
                         [% IF ( EnhancedMessagingPreferences ) %]
-                        <td>
+                        <td style="white-space: nowrap; font-size:80%;">
                             [% IF ( loo.messaging_prefs ) %]
-                                [% IF ( messaging_preference.Item_Due ) %]Item Due
-                                [% ELSIF ( messaging_preference.Advance_Notice ) %]Advance Notice
-                                [% ELSIF ( messaging_preference.Upcoming_Events ) %]Upcoming Events
-                                [% ELSIF ( messaging_preference.Hold_Filled ) %]Hold Filled
-                                [% ELSIF ( messaging_preference.Item_Check_in ) %]Item Check-in
-                                [% ELSIF ( messaging_preference.Item_Checkout ) %]Item Checkout
-                                [% ELSE %]Unknown 
-                                [% END %] :
-                                [% FOREACH transport IN messaging_pref.transports %]
-                                    [% transport.transport %]
-                                [% END %]                                
-                                <br />
+                              [% FOREACH prefs IN loo.messaging_prefs %]
+	                                [% FOREACH transport IN prefs.transports %]
+                                         [% IF ( transport.transport ) %]
+			                                [% IF ( prefs.Item_Due ) %]Item Due
+			                                [% ELSIF ( prefs.Advance_Notice ) %]Advance Notice
+			                                [% ELSIF ( prefs.Upcoming_Events ) %]Upcoming Events
+			                                [% ELSIF ( prefs.Hold_Filled ) %]Hold Filled
+			                                [% ELSIF ( prefs.Item_Check_in ) %]Item Check-in
+			                                [% ELSIF ( prefs.Item_Checkout ) %]Item Checkout
+			                                [% ELSE %]Unknown
+			                                [% END %]:
+				                            <strong>[% transport.transport %]</strong><br />
+				                         [% ELSE %]None<br />[% END %]
+	                                [% END %]
+                                [% END %]
                             [% ELSE %]
-                                none
+                                None
                             [% END %]
                         </td>
                         [% END %]
-- 
2.39.5