Bugzilla – Attachment 139062 Details for
Bug 31166
Digest option is not selectable for phone when PhoneNotification is enabled
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 31166: (QA follow-up) Implement on OPAC side
Bug-31166-QA-follow-up-Implement-on-OPAC-side.patch (text/plain), 3.28 KB, created by
Kyle M Hall (khall)
on 2022-08-12 15:08:35 UTC
(
hide
)
Description:
Bug 31166: (QA follow-up) Implement on OPAC side
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2022-08-12 15:08:35 UTC
Size:
3.28 KB
patch
obsolete
>From 2bdcc4b140a796a85e35fba7b7c9acf48cb7ca43 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Fri, 12 Aug 2022 15:08:01 +0000 >Subject: [PATCH] Bug 31166: (QA follow-up) Implement on OPAC side > >--- > .../bootstrap/en/modules/opac-messaging.tt | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt >index e8eaaac6b8a..f61f939942c 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt >@@ -108,13 +108,13 @@ > <td class="selectcol"> > [% IF ( messaging_preference.transports_phone ) %] > <input type="checkbox" >- class="phone" >+ class="pmp_phone" > id="phone[% messaging_preference.message_attribute_id | html %]" > name="[% messaging_preference.message_attribute_id | html %]" > value="phone" checked="checked" /> > [% ELSE %] > <input type="checkbox" >- class="phone" >+ class="pmp_phone" > id="phone[% messaging_preference.message_attribute_id | html %]" > name="[% messaging_preference.message_attribute_id | html %]" > value="phone" /> >@@ -256,7 +256,11 @@ > $("#info_digests").tooltip(); > > function toggle_digest(id){ >- if ( $("#email"+id).prop("checked") || $("#sms"+id).prop("checked") ) { >+ let PhoneNotification = [% Koha.Preference('PhoneNotification') || 0 | html %]; >+ let TalkingTechItivaPhoneNotification = [% Koha.Preference('TalkingTechItivaPhoneNotification') || 0 | html %]; >+ let phone_checked = TalkingTechItivaPhoneNotification ? false : PhoneNotification ? $("#phone"+id).prop("checked") : false; >+ >+ if ( $("#email"+id).prop("checked") || $("#sms"+id).prop("checked") || phone_checked ) { > $("#digest"+id).attr("disabled", false).tooltip('disable'); > } else { > $("#digest"+id).attr("disabled", true).prop("checked",false).tooltip('enable'); >@@ -275,6 +279,9 @@ > $(".pmp_sms").click(function(){ > toggle_digest(Number($(this).attr("id").replace("sms", ""))); > }); >+ $(".pmp_phone").click(function(){ >+ toggle_digest(Number($(this).attr("id").replace("phone", ""))); >+ }); > > }); > >-- >2.30.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 31166
:
137788
|
137789
|
139061
|
139062
|
139063
|
139064
|
139065
|
144556