From 647c18de7b3966adc8084992bc6a5abb25716011 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 10 May 2021 11:52:12 +0200 Subject: [PATCH] Bug 13613: Move similar JS code to a function --- .../bootstrap/en/modules/opac-messaging.tt | 29 +++++++------------ 1 file changed, 10 insertions(+), 19 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 ef9e266d3d1..a870c35f8bc 100644 --- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt +++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-messaging.tt @@ -250,36 +250,27 @@ $(document).ready(function(){ $("#info_digests").tooltip(); - // At load time, we want digest disabled if no digest using transport is enabled - $(".pmp_email").each(function(){ - var rowid = $(this).attr("id"); - id = Number(rowid.replace("email","")); + function toggle_digest(id){ if ( $("#email"+id).prop("checked") || $("#sms"+id).prop("checked") ) { $("#digest"+id).attr("disabled", false).tooltip('disable'); } else { - $("#digest"+id).attr("disabled", true).tooltip('enable'); + $("#digest"+id).attr("disabled", true).prop("checked",false).tooltip('enable'); } + + } + // At load time, we want digest disabled if no digest using transport is enabled + $(".pmp_email").each(function(){ + toggle_digest(Number($(this).attr("id").replace("email", ""))); }); // If user clears all digest using transports for a notice, disable digest checkbox $(".pmp_email").click(function(){ - var rowid = $(this).attr("id"); - id = Number(rowid.replace("email","")); - if ( $("#email"+id).prop("checked") || $("#sms"+id).prop("checked") ) { - $("#digest"+id).attr("disabled", false).tooltip('disable'); - } else { - $("#digest"+id).attr("disabled", true).prop("checked",false).tooltip('enable'); - } + toggle_digest(Number($(this).attr("id").replace("email", ""))); }); $(".pmp_sms").click(function(){ - var rowid = $(this).attr("id"); - id = Number(rowid.replace("sms","")); - if ( $("#email"+id).prop("checked") || $("#sms"+id).prop("checked") ) { - $("#digest"+id).attr("disabled", false).tooltip('disable'); - } else { - $("#digest"+id).attr("disabled", true).prop("checked",false).tooltip('enable'); - } + toggle_digest(Number($(this).attr("id").replace("sms", ""))); }); + }); function normalizeSMS(value){ -- 2.20.1