id,summary,reporter,owner,description,type,status,priority,component,severity,resolution,keywords,cc,release
10496,js bug in adding status columns,gary.martin@wandisco.com,jun66j5,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.\r\n\r\nThis appears to be a possible solution:\r\n\r\n{{{\r\n#!diff\r\nIndex: tracworkflowadmin/htdocs/scripts/main.js\r\n===================================================================\r\n--- tracworkflowadmin/htdocs/scripts/main.js	(revision 12188)\r\n+++ tracworkflowadmin/htdocs/scripts/main.js	(working copy)\r\n@@ -657_7 +657_7 @@\r\n                 var colspan = $('#status-header-bar').attr('colspan');\r\n                 var statusName = $('#new-status-input-dialog input').val();\r\n                 if (!statusName) return false;\r\n-                $('#status-header-bar').attr('colspan'_ colspan + 1);\r\n+                $('#status-header-bar').attr('colspan'_ parseInt(colspan) + 1);\r\n                 var el = $($('#status-editor-1 th')[0]).clone(true);\r\n                 $('input'_ el).val(statusName);\r\n                 $('span:first'_ el).text(statusName);\r\n}}},defect,closed,normal,TracWorkflowAdminPlugin,normal,fixed,,,1.0