@@ -, +, @@ --- .../intranet-tmpl/prog/en/modules/circ/circulation.tt | 7 ++++++- .../intranet-tmpl/prog/en/modules/circ/pendingreserves.tt | 5 +++++ .../intranet-tmpl/prog/en/modules/members/moremember.tt | 8 +++++++- .../intranet-tmpl/prog/en/modules/reserve/request.tt | 7 ++++++- 4 files changed, 24 insertions(+), 3 deletions(-) --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -970,7 +970,7 @@ [% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %] [% IF hold_cancellation %] - [% FOREACH reason IN hold_cancellation %] @@ -1187,6 +1187,11 @@ } }); [% END %] + $("#cancellation-reason").on("change", function(e){ + $("#cancellation-notify-patron").prop("disabled", $(this).val() == ""); + }); + $("#cancellation-reason").change(); + }); [% INCLUDE 'str/members-menu.inc' %] --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt @@ -323,6 +323,11 @@ holdst.fnFilter( $(this).val(), 10 ); }); }); + + $("#cancellation-reason").on("change", function(e){ + $("#cancellation-notify-patron").prop("disabled", $(this).val() == ""); + }); + $("#cancellation-reason").change(); }); [% END %] --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt @@ -847,7 +847,7 @@ [% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %] [% IF hold_cancellation %] - [% FOREACH reason IN hold_cancellation %] @@ -998,6 +998,12 @@ $("#picture-upload, #show-picture-upload").toggle(); }); + + $("#cancellation-reason").on("change", function(e){ + $("#cancellation-notify-patron").prop("disabled", $(this).val() == ""); + }); + $("#cancellation-reason").change(); + }); function uncheck_sibling(me){ nodename=me.getAttribute("name"); --- a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt @@ -1310,7 +1310,7 @@ if ( reason ) { link += "&cancellation-reason=" + reason } - if ( notify ) { + if ( notify && reason != "" ) { link += "&cancellation-notify-patron=1"; } window.location.href = link; @@ -1357,6 +1357,11 @@ stickTo: "#existing_holds", stickyClass: "floating" }); + + $("#modal-cancellation-reason").on("change", function(e){ + $("#modal-cancellation-notify-patron").prop("disabled", $(this).val() == ""); + }); + $("#modal-cancellation-reason").change(); }); [% END %] --