Next/previous buttons should maintain the current search session

projectblacklight · Nov 4, 2015 · b24e271 · b24e271
1 parent b58dc6e
commit b24e271
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/app/controllers/concerns/blacklight/catalog.rb b/app/controllers/concerns/blacklight/catalog.rb
@@ -53,6 +53,7 @@ def show
     # updates the search counter (allows the show view to paginate)
     def track
       search_session['counter'] = params[:counter]
+      search_session['id'] = params[:search_id]
       search_session['per_page'] = params[:per_page]
 
       path = if params[:redirect] and (params[:redirect].starts_with?("/") or params[:redirect] =~ URI::regexp)

diff --git a/spec/controllers/catalog_controller_spec.rb b/spec/controllers/catalog_controller_spec.rb
@@ -218,6 +218,11 @@ def assigns_response
 
   describe "track action" do
     doc_id = '2007020969'
+
+    it "should persist the search session id value into session[:search]" do
+      put :track, :id => doc_id, :counter => 3, search_id: "123"
+      expect(session[:search]['id']).to eq "123"
+    end
 
     it "should set counter value into session[:search]" do
       put :track, :id => doc_id, :counter => 3