js bug in adding status columns

[gary.martin@…]

I was wondering why the limit on the number of statuses was so low when I spotted that it was probably a bug due to the differences in the way that javascript deals with + and - operators with strings.

This appears to be a possible solution:

tracworkflowadmin/htdocs/scripts/main.js

@@ -657 +657 @@
                 var colspan = $('#status-header-bar').attr('colspan');
                 var statusName = $('#new-status-input-dialog input').val();
                 if (!statusName) return false;
-                $('#status-header-bar').attr('colspan', colspan + 1);
+                $('#status-header-bar').attr('colspan', parseInt(colspan) + 1);
                 var el = $($('#status-editor-1 th')[0]).clone(true);
                 $('input', el).val(statusName);
                 $('span:first', el).text(statusName);

[Jun Omae]

I just confirmed. You're right.

$(selector).attr('colspan') returns a number on Trac 0.12.x with jQuery 1.4.x. However, it returns a string on Trac 1.0 with jQuery 1.7.2.

I'll apply the following patch later. Thanks!

tracworkflowadmin/htdocs/scripts/main.js

@@ -603 +603 @@
     // ステータスの削除
     $('#elements .del-status').click(function() {
         var colspan = $('#status-header-bar').attr('colspan');
+        colspan = parseInt(colspan || '1', 10);
         var c = 0;
         var rowNo = -1;
         var thisEl = $(this).parent().get()[0];
@@ -655 +656 @@
             'Ok': function(){
                 $('#new-status-input-dialog').dialog('close');
                 var colspan = $('#status-header-bar').attr('colspan');
+                colspan = parseInt(colspan || '1', 10);
                 var statusName = $('#new-status-input-dialog input').val();
                 if (!statusName) return false;
                 $('#status-header-bar').attr('colspan', colspan + 1);
@@ -686 +688 @@
 
     $('#elements #add-status').click(function() {
         var colspan = $('#status-header-bar').attr('colspan');
+        colspan = parseInt(colspan || '1', 10);
         if (colspan >= 30) {
             alert(_("Too many statuses. Please remove unnecessary statuses."));
             return false;

[Jun Omae]

Applied in [12214].