Bug 32655: Move phone and itiva variables into templates
To test:
1. Go to a patron record or create a new one.
2. Toggle the 'Patron messaging preferences' fieldset so it is hidden.
3. Notice the variables.
4. Apply patch
5. Try steps 1 & 2 again, no varibales.
6. Attempt to toggle on/off options for email/sms. When email/sms are off the digest option next to it should be disabled.
7. Go to Administration > Patron categories and try step 6 there as well.
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 176397d761
)
Signed-off-by: Jacob O'Mara <jacob.omara@ptfs-europe.com>
This commit is contained in:
parent
b7f976866d
commit
ec70e54af5
3 changed files with 7 additions and 4 deletions
|
@ -1,10 +1,6 @@
|
|||
[% USE Koha %]
|
||||
<!-- snippet for form to set borrower and patron category messaging preferences -->
|
||||
|
||||
<script>
|
||||
var TalkingTechItivaPhoneNotification = [% Koha.Preference('TalkingTechItivaPhoneNotification') || 0 | html %];
|
||||
var PhoneNotification = [% Koha.Preference('PhoneNotification') || 0 | html %];
|
||||
</script>
|
||||
<input type="hidden" name="modify" value="yes" />
|
||||
<input type="hidden" name="borrowernumber" value="[% borrowernumber | html %]" />
|
||||
<table>
|
||||
|
|
|
@ -733,6 +733,10 @@
|
|||
</div> <!-- /.row -->
|
||||
|
||||
[% MACRO jsinclude BLOCK %]
|
||||
<script>
|
||||
var TalkingTechItivaPhoneNotification = [% Koha.Preference('TalkingTechItivaPhoneNotification') || 0 | html %];
|
||||
var PhoneNotification = [% Koha.Preference('PhoneNotification') || 0 | html %];
|
||||
</script>
|
||||
[% Asset.js("js/admin-menu.js") | $raw %]
|
||||
[% Asset.js("js/messaging-preference-form.js") | $raw %]
|
||||
[% INCLUDE 'calendar.inc' %]
|
||||
|
|
|
@ -1930,6 +1930,9 @@ legend:hover {
|
|||
[% END %]
|
||||
$("#guarantor_template").hide();
|
||||
|
||||
var TalkingTechItivaPhoneNotification = [% Koha.Preference('TalkingTechItivaPhoneNotification') || 0 | html %];
|
||||
var PhoneNotification = [% Koha.Preference('PhoneNotification') || 0 | html %];
|
||||
|
||||
</script>
|
||||
[% Asset.js("js/members.js") | $raw %]
|
||||
[% Asset.js("js/messaging-preference-form.js") | $raw %]
|
||||
|
|
Loading…
Reference in a new issue