Attachment 65451 Details for Bug 18692 – Bug 18692 - intranet part

[patch] Bug 18692 - intranet part

Bug-18692---intranet-part.patch (text/plain), 2.83 KB, created by Fridolin Somers on 2017-08-03 08:04:20 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2017-08-03 08:04:20 UTC

Size: 2.83 KB

patch

obsolete

>From 5b0d331a0e4ea31a01f576182035a9a9d1d53e33 Mon Sep 17 00:00:00 2001
>From: Lari Taskula <lari.taskula@jns.fi>
>Date: Wed, 31 May 2017 17:03:54 +0300
>Subject: [PATCH] Bug 18692 - intranet part
>
>Fixes misplaced columns introduced by previous patch and adds the "-" for phone
>transport type.
>
>To test:
>1. Set SMSSendDriver system preference on
>2. Go to intranet messaging preferences
>3. By default you should see checkboxes for all messages for SMS
>4. Ensure columns are not misplaced (pushing one column too much to the right)
>5. Delete sms method from one of the messages in message_transports table
>6. Observe that "-" is displayed instead of checkbox for that message for SMS
>7. Repeat same for TalkingTechItivaPhoneNotification system preference.
>   By default it may not have transports in message_transports, so make sure
>   to assign some in order to have the checkboxes visible.
>---
> .../prog/en/includes/messaging-preference-form.inc   | 20 ++++++++++++++++----
> 1 file changed, 16 insertions(+), 4 deletions(-)
>
>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 a5da438..70761a0 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc
>@@ -63,7 +63,9 @@ $(document).ready(function(){
>       <td>-</td>
>       [% END %]
> 
>-      [% IF Koha.Preference('SMSSendDriver') %]<td>
>+      [% IF Koha.Preference('SMSSendDriver') %]
>+        [% IF ( messaging_preference.transport_sms ) %]
>+          <td>
>           [% IF ( messaging_form_inactive ) %]
>               [% IF ( messaging_preference.transports_sms ) %]
>                  <input type="checkbox"
>@@ -89,9 +91,15 @@ $(document).ready(function(){
>                  value="sms" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id %]" />
>               [% END %]
>           [% END %]
>-      </td>[% END %]
>+          </td>
>+        [% ELSE %]
>+          <td>-</td>
>+        [% END %]
>+      [% END %]
> 
>-      [% IF Koha.Preference('TalkingTechItivaPhoneNotification') %]<td>
>+      [% IF Koha.Preference('TalkingTechItivaPhoneNotification') %]
>+        [% IF ( messaging_preference.transport_phone ) %]
>+          <td>
>           [% IF ( messaging_form_inactive ) %]
>               [% IF ( messaging_preference.transports_phone ) %]
>                  <input type="checkbox"
>@@ -117,7 +125,11 @@ $(document).ready(function(){
>                  value="phone" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id %]" />
>               [% END %]
>           [% END %]
>-      </td>[% END %]
>+          </td>
>+        [% ELSE %]
>+          <td>-</td>
>+        [% END %]
>+      [% END %]
> 
>       [% IF ( messaging_preference.transport_email ) %]
>       <td>
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 18692: 63814 | 63830 | 63872 | 63897 | 63898 | 63900 | 64409 | 64410 | 65170 | 65171 | 65451 | 65906 | 66731