diff --git a/.travis.yml b/.travis.yml index ad50a08216..63396dac58 100644 --- a/.travis.yml +++ b/.travis.yml @@ -2,6 +2,7 @@ notifications: email: false rvm: + - 2.1.1 - 2.1.0 - 2.0.0 - 1.9.3 diff --git a/app/helpers/blacklight/search_history_constraints_helper_behavior.rb b/app/helpers/blacklight/search_history_constraints_helper_behavior.rb index ab7a9a502e..1a8b699bcb 100644 --- a/app/helpers/blacklight/search_history_constraints_helper_behavior.rb +++ b/app/helpers/blacklight/search_history_constraints_helper_behavior.rb @@ -19,13 +19,13 @@ def render_search_to_s(params) ## # Render the search query constraint def render_search_to_s_q(params) - return "".html_safe if params[:q].blank? + return "".html_safe if params['q'].blank? label = (default_search_field && params[:search_field] == default_search_field[:key]) ? nil : label_for_search_field(params[:search_field]) - render_search_to_s_element(label , render_filter_value(params[:q]) ) + render_search_to_s_element(label , render_filter_value(params['q']) ) end ## diff --git a/app/helpers/blacklight/url_helper_behavior.rb b/app/helpers/blacklight/url_helper_behavior.rb index 0b36546e6e..4d6cffd96a 100644 --- a/app/helpers/blacklight/url_helper_behavior.rb +++ b/app/helpers/blacklight/url_helper_behavior.rb @@ -140,7 +140,7 @@ def sanitize_search_params source_params # Reset any search parameters that store search context # and need to be reset when e.g. constraints change def reset_search_params source_params - sanitize_search_params(source_params).except :page, :counter + sanitize_search_params(source_params).except(:page, :counter).with_indifferent_access end # adds the value and/or field to params[:f] @@ -215,4 +215,4 @@ def remove_facet_params(field, item, source_params=params) p end -end \ No newline at end of file +end diff --git a/spec/controllers/catalog_controller_spec.rb b/spec/controllers/catalog_controller_spec.rb index acfedbfe5d..29459570de 100644 --- a/spec/controllers/catalog_controller_spec.rb +++ b/spec/controllers/catalog_controller_spec.rb @@ -76,7 +76,7 @@ def assigns_response expect(session[:search].keys).to include :id search = Search.find(session[:search][:id]) - expect(search.query_params[:q]).to eq @user_query + expect(search.query_params['q']).to eq @user_query end end diff --git a/spec/lib/blacklight/solr_helper_spec.rb b/spec/lib/blacklight/solr_helper_spec.rb index 789821f620..ffa0eb24bc 100644 --- a/spec/lib/blacklight/solr_helper_spec.rb +++ b/spec/lib/blacklight/solr_helper_spec.rb @@ -730,9 +730,9 @@ def params blacklight_config.stub(:default_solr_params).and_return({:qt => 'custom_request_handler'}) blacklight_solr.should_receive(:send_and_receive) do |path, params| expect(path).to eq 'select' - expect(params[:params][:'facet.field']).to eq ["format", "{!ex=pub_date_single}pub_date", "subject_topic_facet", "language_facet", "lc_1letter_facet", "subject_geo_facet", "subject_era_facet"] - expect(params[:params][:"facet.query"]).to eq ["pub_date:[#{5.years.ago.year} TO *]", "pub_date:[#{10.years.ago.year} TO *]", "pub_date:[#{25.years.ago.year} TO *]"] - expect(params[:params]).to include(:rows => 10, :qt=>"custom_request_handler", :q=>"", "spellcheck.q"=>"", :"f.subject_topic_facet.facet.limit"=>21, :sort=>"score desc, pub_date_sort desc, title_sort asc") + expect(params[:params]['facet.field']).to eq ["format", "{!ex=pub_date_single}pub_date", "subject_topic_facet", "language_facet", "lc_1letter_facet", "subject_geo_facet", "subject_era_facet"] + expect(params[:params]["facet.query"]).to eq ["pub_date:[#{5.years.ago.year} TO *]", "pub_date:[#{10.years.ago.year} TO *]", "pub_date:[#{25.years.ago.year} TO *]"] + expect(params[:params]).to include('rows' => 10, 'qt'=>"custom_request_handler", 'q'=>"", "spellcheck.q"=>"", "f.subject_topic_facet.facet.limit"=>21, 'sort'=>"score desc, pub_date_sort desc, title_sort asc") end.and_return({'response'=>{'docs'=>[]}}) get_search_results(:q => @all_docs_query) end