Skip to content

Commit

Permalink
use unsafe params explicitly (Rails upgrade)
Browse files Browse the repository at this point in the history
  • Loading branch information
mejackreed committed Feb 28, 2018
1 parent c1bfb17 commit 779ec4a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions app/views/catalog/_refine_facets.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
<div class="row">
<div class="col-xs-3">
<% if query_has_facet_constraints?(params) %>
<div><%= render_clear_all(t('blacklight.search.filters.clear-all'), params) %></div>
<div><%= render_clear_all(t('blacklight.search.filters.clear-all'), params.to_unsafe_h) %></div>
<% end %>
<ul class="nav nav-tabs tabs-left">
<%= render_facet_tabs %>
Expand All @@ -21,4 +21,4 @@
</div>
</ul>
</div>
<% end %>
<% end %>
2 changes: 1 addition & 1 deletion spec/views/catalog/_refine_facets.html.erb_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
describe 'facet limits with facet constraints in query' do
before(:each) do
allow(view).to receive(:has_facet_values?).and_return(true)
allow(view).to receive(:params).and_return(double(f: { language: ['Latin'] }))
allow(view).to receive(:params).and_return(double(to_unsafe_h: {f: { language: ['Latin'] }}))
allow(view).to receive(:query_has_facet_constraints?).and_return(true)
allow(view).to receive(:render_clear_all).and_return(raw('<div class="clear_all">Clear all</div>'))
allow(view).to receive(:render_facet_tabs).and_return(raw('<div class="facet_tabs">Facet tabs</div>'))
Expand Down

0 comments on commit 779ec4a

Please sign in to comment.