Bug 16397 - Unable to delete audio alerts
authorOwen Leonard <oleonard@myacpl.org>
Fri, 29 Apr 2016 16:29:01 +0000 (12:29 -0400)
committerKyle M Hall <kyle@bywatersolutions.com>
Tue, 3 May 2016 14:58:38 +0000 (14:58 +0000)
This patch corrects a JavaScript error which has arisen due to the
jQuery upgrade. This should have been included in Bug 16321.

To test, apply the patch and go to Administration -> Audio alerts.

- In the list of existing audio alerts, check one or more checkboxes.
  The "Delete selected" button should become enabled.
- Uncheck all checkboxes and check that the "Delete selected" button is
  now disabled.
- Confirm that deletions are completed correctly.

Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/js/audio_alerts.js

index 154c33a..f16f050 100644 (file)
@@ -2,7 +2,7 @@ $( document ).ready(function() {
     var checkboxes = $("#delete-alert-form input[type='checkbox']");
     var checkedcheckboxes = 0;
     checkboxes.on("change",function(){
-        if( $(checkboxes+":checked").length > 0){
+        if( $("#delete-alert-form").find("input:checked").length > 0){
             checkedcheckboxes = 1;
             $("#delete-alerts").removeClass("disabled");
         } else {
@@ -86,4 +86,4 @@ function EditAlert( elt, id, precedence, selector, sound ) {
     $("#sound").val(sound);
     $("#koha-sounds").val(sound);
     enablePlayButton(sound,$('#play-sound'));
-}
\ No newline at end of file
+}