Merge pull request #161 from AT-WH/master

Making sure query string parameters are encoded

Author: maikebing

src/SilkierQuartz/Views/Jobs/Index.hbs

@@ -139,8 +139,8 @@
     function bindContextMenu(menu, row, jobName, jobGroup) {
 
         menu.find('>.menu-edit').attr('href', row.find('>.job-name a').attr('href'));
-        menu.find('>.menu-copy').attr('href', '{{ActionUrl "Duplicate"}}?group=' + jobGroup + '&name=' + jobName);
-        menu.find('>.menu-trigger').attr('href', '{{ActionUrl "Trigger"}}?group=' + jobGroup + '&name=' + jobName);
+        menu.find('>.menu-copy').attr('href', '{{ActionUrl "Duplicate"}}?group=' + encodeURIComponent(jobGroup) + '&name=' + encodeURIComponent(jobName));
+        menu.find('>.menu-trigger').attr('href', '{{ActionUrl "Trigger"}}?group=' + encodeURIComponent(jobGroup) + '&name=' + encodeURIComponent(jobName));
         menu.find('>.menu-delete').click(function () {
             $('#delete-dialog .confirm-item').text(jobGroup +'.' + jobName);
             deleteItem({ name: jobName, group: jobGroup }, $('#msg-panel'),

src/SilkierQuartz/Views/Triggers/Index.hbs

@@ -206,7 +206,7 @@
         mnuResume.click(pauseResumeFunc);
 
         menu.find('>.menu-edit').attr('href', row.find('>.trigger-name a').attr('href'));
-        menu.find('>.menu-copy').attr('href', '{{ActionUrl "Duplicate"}}?group=' + triggerGroup + '&name=' + triggerName);
+        menu.find('>.menu-copy').attr('href', '{{ActionUrl "Duplicate"}}?group=' + encodeURIComponent(triggerGroup) + '&name=' + encodeURIComponent(triggerName));
         menu.find('>.menu-delete').click(function () {
             $('#delete-dialog .confirm-item').text(triggerGroup + '.' + triggerName);
             deleteItem({ name: triggerName, group: triggerGroup }, $('#msg-panel'),