[Patch] Remove obsolete jQuery syntax

[Bart Ogryczak]

This plugin uses JQuery 1.1 [@selectors] which are deprecated in JQuery 1.2 and removed completely in JQuery 1.3. Easily fixable by removing @

Patch:

--- batchmod/templates/batchmod.html.orig       2008-10-12 20:57:00.000000000 +0200
+++ batchmod/templates/batchmod.html    2009-08-24 14:26:44.000000000 +0200
@@ -79,17 +79,17 @@
       $(this).before('<th class="bmod_selector"><input type="checkbox" name="bmod_toggleGroup" /></th>');
     }
   )
-  $("input[@name='bmod_toggleGroup']").click(function() {
+  $("input[name='bmod_toggleGroup']").click(function() {
     $("tr td input.bmod_selector",$(this).parents("table.listing")).attr("checked",this.checked);
   })
   <py:for each="field in fields">
-    $("input[@id=bmod_flag_${field.name}]").click(function() { $("*[@name=bmod_value_${field.name}]").enable(this.checked);} )
+    $("input[id=bmod_flag_${field.name}]").click(function() { $("*[name=bmod_value_${field.name}]").enable(this.checked);} )
   </py:for>
   $("input#bmod_flag_comment").click(function() {enableControl("bmod_value_comment",this.checked);} )
   $("form#batchmod-form").submit(function() {
     var selectedTix=[];
-    $("input[@name=selectedTicket]:checked").each( function(){ selectedTix.push(this.value);} );
-    $("input[@name=selectedTickets]").val(selectedTix);
+    $("input[name=selectedTicket]:checked").each( function(){ selectedTix.push(this.value);} );
+    $("input[name=selectedTickets]").val(selectedTix);
   })
   $(document.getElementById("columns")).toggleClass("collapsed");
 });

[Ryan J Ollos]

Just looking through the tickets and noticed that this might potentially cause a problem on Trac 0.12, which may be moving to jQuery 1.3 according to some discussion I vaguely remember. The current 0.12dev still uses jQuery 1.2.6 though.

[CuriousCurmudgeon]

Replying to rjollos:

Just looking through the tickets and noticed that this might potentially cause a problem on Trac 0.12, which may be moving to jQuery 1.3 according to some discussion I vaguely remember. The current 0.12dev still uses jQuery 1.2.6 though.

There have been a number of tickets for this, but r7415 in #6012 removed the selectors. We should be fine on this issue if 0.12 moves to jQuery 1.3