Merge pull request #30 from brandnewbox/downloads_dont_work

fixed download button

app/views/blazer/queries/show.html.erb

@@ -1,23 +1,23 @@
 <% blazer_title @query.name %>
 
+<% if @success %>
+  <% run_data = {statement: @query.statement, query_id: @query.id, data_source: @query.data_source, variables: variable_params(@query)} %>
+  <% run_data.merge!(forecast: "t") if params[:forecast] %>
+  <% run_data.merge!(cohort_period: params[:cohort_period]) if params[:cohort_period] %>
+  <% run_data.transform_keys!(&:to_s) if Rails::VERSION::MAJOR < 6 %>
+<% end %>
+
 <% content_for :header do %>
   <%= render "blazer/nav" do %>
     <%= link_to "Edit", edit_query_path(@query, variable_params(@query)), class: "btn btn-link", disabled: [email protected]?(blazer_user) %>
     <%= link_to "Fork", new_query_path(variable_params(@query).merge(fork_query_id: @query.id, data_source: @query.data_source, name: @query.name)), class: "btn btn-link" %>
 
     <% if !@error && @success %>
-      <%= button_to "Download", run_queries_path(query_id: @query.id, format: "csv", forecast: params[:forecast], cohort_period: params[:cohort_period]), params: {statement: @statement}, class: "btn btn-link" %>
+      <%= button_to "Download", run_queries_path(format: "csv"), params: run_data, class: "btn btn-link" %>
     <% end %>
   <% end %>
 <% end %>
 
-<% if @success %>
-  <% run_data = {statement: @query.statement, query_id: @query.id, data_source: @query.data_source, variables: variable_params(@query)} %>
-  <% run_data.merge!(forecast: "t") if params[:forecast] %>
-  <% run_data.merge!(cohort_period: params[:cohort_period]) if params[:cohort_period] %>
-  <% run_data.transform_keys!(&:to_s) if Rails::VERSION::MAJOR < 6 %>
-<% end %>
-
 <div style="margin-bottom: 60px;"></div>
 
 <% if @sql_errors.any? %>