Modify ↓
      
        Opened 10 years ago
Closed 9 years ago
#12736 closed defect (fixed)
javascript error in testman4trac/testmanager/htdocs/js/testmanager.js
| Reported by: | Owned by: | Roberto Longobardi | |
|---|---|---|---|
| Priority: | normal | Component: | TestManagerForTracPlugin | 
| Severity: | normal | Keywords: | |
| Cc: | Trac Release: | 0.12 | 
Description
Release 2015-11-22 . Safari and chrome show a syntax error, the following patch fixes these:
diff --git a/testman4trac/testmanager/htdocs/js/testmanager.js b/testman4trac/testmanager/htdocs/js/testmanager.js
index 3442102..5bc09dc 100644
--- a/testman4trac/testmanager/htdocs/js/testmanager.js
+++ b/testman4trac/testmanager/htdocs/js/testmanager.js
@@ -83,15 +83,14 @@ function createTestPlanConfirm(catName) {
     var testplanSnapshot = $("input[@name=testplan_snapshot]:checked").val();
        */
 
-    var nodes = $("input[@name='testplan_contains_all']:checked");
+    var nodes = $("input[name='testplan_contains_all']:checked");
     for (var i=0; i<nodes.length; i++) {
         var node = nodes[i];
         if (node.name == 'testplan_contains_all') {
             testplanContainsAll = node.value;
         }
     }
-
-    nodes = $("input[@name='testplan_snapshot']:checked");
+    nodes = $("input[name='testplan_snapshot']:checked");
     for (var i=0; i<nodes.length; i++) {
         var node = nodes[i];
         if (node.name == 'testplan_snapshot') {
    Attachments (0)
Note: See
        TracTickets for help on using
        tickets.
    


Thanks a lot for the patch.
Included in release 1.9.1 now available from SourceForge.
https://sourceforge.net/projects/testman4trac/files