5105: Include form data with report_issue request.

diff --git a/apps/workbench/app/assets/javascripts/report_issue.js b/apps/workbench/app/assets/javascripts/report_issue.js
index 371dacc1afe1eb8bfc8d11144db59cb724c44b15..539489d5fb2874584a3270452b57aa7b31133428 100644 (file)
--- a/apps/workbench/app/assets/javascripts/report_issue.js
+++ b/apps/workbench/app/assets/javascripts/report_issue.js
@@ -8,14 +8,15 @@ $(document).
     }
     $('div').remove('.modal-footer-status');
 
-    $.ajax('/report_issue', {type: 'POST'}).
-      success(function(data, status, jqxhr) {
+    $.ajax('/report_issue', {
+        type: 'POST',
+        data: $(this).parents('form').serialize()
+    }).success(function(data, status, jqxhr) {
         var $sendButton = $('#report-issue-submit');
         $sendButton.html('Report sent');
         $('div').remove('.modal-footer-status');
         $('.modal-footer').append('<div><br/></div><div class="modal-footer-status alert alert-success"><p class="contain-align-left">Thanks for reporting this issue!</p></div>');
-      }).
-      fail(function(jqxhr, status, error) {
+    }).fail(function(jqxhr, status, error) {
         var $sendButton = $('#report-issue-submit');
         if ($sendButton && $sendButton.prop('disabled')) {
           $('div').remove('.modal-footer-status');
@@ -25,6 +26,6 @@ $(document).
         }
         var $cancelButton = $('#report-issue-cancel');
         $cancelButton.html('Cancel');
-      });
-
+    });
+    return false;
   });

diff --git a/apps/workbench/app/views/application/_report_issue_popup.html.erb b/apps/workbench/app/views/application/_report_issue_popup.html.erb
index 315116bbc8926e1c8dfd9953a8bf32561b575090..1c964abfe0491b8ad1c77c02767aaf845927093e 100644 (file)
--- a/apps/workbench/app/views/application/_report_issue_popup.html.erb
+++ b/apps/workbench/app/views/application/_report_issue_popup.html.erb
@@ -27,7 +27,7 @@
   <div class="modal-content">
 
     <%= form_tag report_issue_path, {name: 'report-issue-form', method: 'post',
-        class: 'form-horizontal', remote: true} do %>
+        class: 'form-horizontal'} do %>
 
       <%
         title = 'Version / debugging info'