projects / koha.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Bug 9002 - Remove Problematic Logic from Patron Messaging Preferences Form
[koha.git] / koha-tmpl / intranet-tmpl / prog / en / includes / messaging-preference-form.inc
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
index 3abce71..ed6b067 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
@@ -5,10 +5,10 @@
                        if($(this).attr("checked")){
                                var rowid = $(this).attr("id");
                                newid = Number(rowid.replace("none","")) 
-                               $("#sms"+newid).attr("checked","");
-                               $("#email"+newid).attr("checked","");
-                               $("#digest"+newid).attr("checked","");
-                               $("#rss"+newid).attr("checked","");
+          $("#sms"+newid).removeAttr("checked");
+          $("#email"+newid).removeAttr("checked");
+          $("#digest"+newid).removeAttr("checked");
+          $("#rss"+newid).removeAttr("checked");
                        }
                });
        });
@@ -16,116 +16,184 @@
 </script>
 
 <input type="hidden" name="modify" value="yes" />
-<input type="hidden" name="borrowernumber" value="<!-- TMPL_VAR NAME="borrowernumber" -->" />
+<input type="hidden" name="borrowernumber" value="[% borrowernumber %]" />
   <table>
     <tr><th></th>
         <th>Days in advance</th>
-        <!-- TMPL_IF NAME="SMSSendDriver" --><th>SMS</th><!-- /TMPL_IF -->
+        [% IF ( SMSSendDriver ) %]<th>SMS</th>[% END %]
+        [% IF ( TalkingTechItivaPhone ) %]<th>Phone</th>[% END %]
         <th>Email</th><th>Digests only?</th>
         <!-- <th>RSS</th> -->
-        <!-- TMPL_UNLESS NAME="messaging_form_inactive" --><th>Do not notify</th><!-- /TMPL_UNLESS -->
+        [% UNLESS ( messaging_form_inactive ) %]<th>Do not notify</th>[% END %]
     </tr>
-    <!-- TMPL_LOOP name="messaging_preferences" -->
+    [% FOREACH messaging_preference IN messaging_preferences %]
     <tr>
-      <td><!-- TMPL_IF NAME="Item Due" -->Item Due
-          <!-- TMPL_ELSIF NAME="Advance Notice" -->Advance Notice
-          <!-- TMPL_ELSIF NAME="Upcoming Events" -->Upcoming Events
-          <!-- TMPL_ELSIF NAME="Hold Filled" -->Hold Filled
-          <!-- TMPL_ELSIF NAME="Item Check-in" -->Item Check-in
-          <!-- TMPL_ELSIF NAME="Item Checkout" -->Item Checkout
-          <!-- TMPL_ELSE -->Unknown <!-- /TMPL_IF --></td>
-      <!-- TMPL_IF NAME="takes_days" -->
+      <td>[% 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 %]</td>
+      [% IF ( messaging_preference.takes_days ) %]
       <td>
-          <!-- TMPL_IF NAME="messaging_form_inactive" -->
-            <select name="<!-- TMPL_VAR NAME="message_attribute_id" -->-DAYS" disabled="disabled">
-          <!-- TMPL_ELSE -->
-            <select name="<!-- TMPL_VAR NAME="message_attribute_id" -->-DAYS">
-          <!-- /TMPL_IF -->
-          <!-- TMPL_LOOP name="select_days" -->
-          <!-- TMPL_IF NAME="selected" --><option value="<!-- TMPL_VAR NAME="day" -->" selected="selected"><!-- TMPL_VAR NAME="day" --></option><!-- TMPL_ELSE -->
-          <option value="<!-- TMPL_VAR NAME="day" -->"><!-- TMPL_VAR NAME="day" --></option>
-      <!-- /TMPL_IF -->
-          <!-- /TMPL_LOOP -->
+          [% IF ( messaging_form_inactive ) %]
+            <select name="[% messaging_preference.message_attribute_id %]-DAYS" disabled="disabled">
+          [% ELSE %]
+            <select name="[% messaging_preference.message_attribute_id %]-DAYS">
+          [% END %]
+          [% FOREACH select_day IN messaging_preference.select_days %]
+          [% IF ( select_day.selected ) %]<option value="[% select_day.day %]" selected="selected">[% select_day.day %]</option>[% ELSE %]
+          <option value="[% select_day.day %]">[% select_day.day %]</option>
+      [% END %]
+          [% END %]
         </select>
       </td>
-      <!-- TMPL_ELSE -->
+      [% ELSE %]
       <td>-</td>
-      <!-- /TMPL_IF -->
-<!--      <!-- TMPL_IF NAME="transport_sms" -->
-      <!-- TMPL_IF NAME="SMSSendDriver" --><td>
-          <!-- TMPL_IF NAME="messaging_form_inactive" -->
+      [% END %]
+
+      [% IF ( SMSSendDriver ) %]<td>
+          [% IF ( messaging_form_inactive ) %]
+              [% IF ( messaging_preference.transports_sms ) %]
                  <input type="checkbox"
-                 id="sms<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="sms"   <!-- TMPL_VAR NAME="transport_sms" --> disabled="disabled" />
-          <!-- TMPL_ELSE -->
+                 id="sms[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="sms" checked="checked" disabled="disabled" />
+              [% ELSE %]
                  <input type="checkbox"
-                 id="sms<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="sms"   <!-- TMPL_VAR NAME="transport_sms"   --> onclick = "$('#none'+'<!-- TMPL_VAR NAME="message_attribute_id" -->').attr('checked','');" />
-          <!-- /TMPL_IF -->
-      </td><!-- /TMPL_IF -->
-      <!-- TMPL_ELSE -->
-      <td>-</td>
-      <!-- /TMPL_IF --> -->
+                 id="sms[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="sms" disabled="disabled" />
+              [% END %]
+          [% ELSE %]
+              [% IF ( messaging_preference.transports_sms ) %]
+                 <input type="checkbox"
+                 id="sms[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="sms" checked="checked" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+              [% ELSE %]
+                 <input type="checkbox"
+                 id="sms[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="sms" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+              [% END %]
+          [% END %]
+      </td>[% END %]
 
-      <!-- TMPL_IF NAME="transport_email" -->
+      [% IF ( TalkingTechItivaPhone ) %]<td>
+          [% IF ( messaging_form_inactive ) %]
+              [% IF ( messaging_preference.transports_phone ) %]
+                 <input type="checkbox"
+                 id="phone[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="phone" checked="checked" disabled="disabled" />
+              [% ELSE %]
+                 <input type="checkbox"
+                 id="phone[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="phone" disabled="disabled" />
+              [% END %]
+          [% ELSE %]
+              [% IF ( messaging_preference.transports_phone ) %]
+                 <input type="checkbox"
+                 id="phone[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="phone" checked="checked" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').attr('checked','');" />
+              [% ELSE %]
+                 <input type="checkbox"
+                 id="phone[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="phone" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').attr('checked','');" />
+              [% END %]
+          [% END %]
+      </td>[% END %]
+
+      [% IF ( messaging_preference.transport_email ) %]
       <td>
-          <!-- TMPL_IF NAME="messaging_form_inactive" -->
+          [% IF ( messaging_form_inactive ) %]
+              [% IF ( messaging_preference.transports_email ) %]
+                <input type="checkbox"
+                   id="email[% messaging_preference.message_attribute_id %]"
+                   name="[% messaging_preference.message_attribute_id %]"
+                   value="email" checked="checked" disabled="disabled" />
+              [% ELSE %]
+                <input type="checkbox"
+                   id="email[% messaging_preference.message_attribute_id %]"
+                   name="[% messaging_preference.message_attribute_id %]"
+                   value="email" disabled="disabled" />
+              [% END %]
+          [% ELSE %]
+              [% IF ( messaging_preference.transports_email ) %]
             <input type="checkbox"
-                 id="email<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="email"   <!-- TMPL_VAR NAME="transport_email"   --> disabled="disabled" />
-          <!-- TMPL_ELSE -->
+                 id="email[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="email" checked="checked" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+              [% ELSE %]
             <input type="checkbox"
-                 id="email<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="email"   <!-- TMPL_VAR NAME="transport_email"   --> onclick = "$('#none'+'<!-- TMPL_VAR NAME="message_attribute_id" -->').attr('checked','');" />
-          <!-- /TMPL_IF -->
+                 id="email[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="email" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+              [% END %]
+          [% END %]
       </td>
-      <!-- TMPL_ELSE -->
+      [% ELSE %]
       <td>-</td>
-      <!-- /TMPL_IF -->
+      [% END %]
 
-      <!-- TMPL_IF NAME="has_digest" -->
+      [% IF ( messaging_preference.has_digest ) %]
       <td>
-          <!-- TMPL_IF NAME="messaging_form_inactive" -->
-            <input type="checkbox"
-                 id="digest<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="digest"   <!-- TMPL_VAR NAME="digest"   --> disabled="disabled" />
-          <!-- TMPL_ELSE -->
-            <input type="checkbox"
-                 id="digest<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="digest"   <!-- TMPL_VAR NAME="digest"   --> onclick = "$('#none'+'<!-- TMPL_VAR NAME="message_attribute_id" -->').attr('checked','');" />
-          <!-- /TMPL_IF -->
+          [% IF ( messaging_form_inactive ) %]
+              [% IF ( messaging_preference.digest ) %]
+                <input type="checkbox"
+                     id="digest[% messaging_preference.message_attribute_id %]"
+                     value="[% messaging_preference.message_attribute_id %]"
+                     name="digest" checked="checked" disabled="disabled" />
+              [% ELSE %]
+                <input type="checkbox"
+                     id="digest[% messaging_preference.message_attribute_id %]"
+                     value="[% messaging_preference.message_attribute_id %]"
+                     name="digest" disabled="disabled" />
+              [% END %]
+          [% ELSE %]
+              [% IF ( messaging_preference.digest ) %]
+                <input type="checkbox"
+                     id="digest[% messaging_preference.message_attribute_id %]"
+                     value="[% messaging_preference.message_attribute_id %]"
+                     name="digest" checked="checked" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+              [% ELSE %]
+                <input type="checkbox"
+                     id="digest[% messaging_preference.message_attribute_id %]"
+                     value="[% messaging_preference.message_attribute_id %]"
+                     name="digest" onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+              [% END %]
+          [% END %]
       </td>
-      <!-- TMPL_ELSE -->
+      [% ELSE %]
       <td>-</td>
-      <!-- /TMPL_IF -->
+      [% END %]
 
-<!--       <!-- TMPL_IF NAME="transport_rss" -->
+<!--       [% IF ( messaging_preference.transport_rss ) %]
       <td>
-          <!-- TMPL_IF NAME="messaging_form_inactive" -->
+          [% IF ( messaging_form_inactive ) %]
             <input type="checkbox"
-                 id="rss<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="rss"   <!-- TMPL_VAR NAME="transport_rss"   --> disabled="disabled" />
-          <!-- TMPL_ELSE -->
+                 id="rss[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="rss"   [% messaging_preference.transport_rss %] disabled="disabled" />
+          [% ELSE %]
             <input type="checkbox"
-                 id="rss<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 name="<!-- TMPL_VAR NAME="message_attribute_id" -->"
-                 value="rss"   <!-- TMPL_VAR NAME="transport_rss"   --> onclick = "$('#none'+'<!-- TMPL_VAR NAME="message_attribute_id" -->').attr('checked','');" />
-          <!-- /TMPL_IF -->
+                 id="rss[% messaging_preference.message_attribute_id %]"
+                 name="[% messaging_preference.message_attribute_id %]"
+                 value="rss"   [% messaging_preference.transport_rss %] onclick = "$('#none'+'[% messaging_preference.message_attribute_id %]').removeAttr('checked');" />
+          [% END %]
       </td>
-      <!-- TMPL_ELSE -->
+      [% ELSE %]
       <td>-</td>
-      <!-- /TMPL_IF --> -->
+      [% END %] -->
 
-      <!-- TMPL_UNLESS NAME="messaging_form_inactive" -->
-      <td><input type="checkbox" class="none" id="none<!-- TMPL_VAR NAME="message_attribute_id" -->" /></td>
-      <!-- /TMPL_UNLESS -->
+      [% UNLESS ( messaging_form_inactive ) %]
+      <td><input type="checkbox" class="none" id="none[% messaging_preference.message_attribute_id %]" /></td>
+      [% END %]
     </tr>
-    <!-- /TMPL_LOOP -->
+    [% END %]
   </table>
Koha ILS @ FFZG - based on git://git.koha.org/pub/scm/koha.git