Merge pull request #3211 from AlchemyCMS/backport/7.4-stable/pr-3209

[7.4-stable] Resource "Download CSV" link: Use all filter params

Author: tvdeyen

app/views/alchemy/admin/resources/index.html.erb

@@ -17,7 +17,7 @@
     ) %>
     <%= render Alchemy::Admin::ToolbarButton.new(
       icon: :download,
-      url: resource_url_proxy.url_for(action: 'index', format: 'csv', q: search_filter_params[:q], sort: params[:sort]),
+      url: resource_url_proxy.url_for(action: 'index', format: 'csv', **search_filter_params),
       label: Alchemy.t(:download_csv),
       title: Alchemy.t(:download_csv),
       dialog: false,

spec/features/admin/resources_integration_spec.rb

@@ -94,6 +94,10 @@
               expect(page).to have_content("today 2")
               expect(page).to_not have_content("yesterday")
             end
+
+            within "#toolbar" do
+              expect(page).to have_link(href: %r{/admin/events\.csv\?filter%5Bstart%5D=starting_today})
+            end
           end
 
           it "can combine multiple filters" do