Merge pull request #1837 from samvera/fix-add_works_forward

Adding works to collection forwards to collection admin show

app/controllers/hyrax/dashboard/collections_controller.rb

@@ -214,7 +214,8 @@ def uploaded_files(uploaded_file_ids)
         end
 
         def update_referer
-          edit_dashboard_collection_path(@collection) + (params[:referer_anchor] || '')
+          return edit_dashboard_collection_path(@collection) + (params[:referer_anchor] || '') if params[:stay_on_edit]
+          dashboard_collection_path(@collection)
         end
 
         def determine_banner_data

app/views/hyrax/dashboard/collections/_form.html.erb

@@ -58,6 +58,7 @@
               </div>
             <% end %>
             <%= hidden_field_tag :type, params[:type] %>
+            <%= hidden_field_tag :stay_on_edit, true %>
             <%= hidden_field_tag :collection_type_gid, @collection.collection_type_gid %>
             <% if params[:batch_document_ids].present? %>
               <% params[:batch_document_ids].each do |batch_item| %>

spec/controllers/hyrax/dashboard/collections_controller_spec.rb

@@ -133,16 +133,27 @@
         end
       end
 
-      it "adds members to the collection" do
+      it "adds members to the collection from edit form" do
         expect do
           put :update, params: { id: collection,
                                  collection: { members: 'add' },
-                                 batch_document_ids: [asset3.id] }
+                                 batch_document_ids: [asset3.id],
+                                 stay_on_edit: true }
         end.to change { collection.reload.member_objects.size }.by(1)
         expect(response).to redirect_to routes.url_helpers.edit_dashboard_collection_path(collection, locale: 'en')
         expect(assigns[:collection].member_objects).to match_array [asset1, asset2, asset3]
       end
 
+      it "adds members to the collection from other than the edit form" do
+        expect do
+          put :update, params: { id: collection,
+                                 collection: { members: 'add' },
+                                 batch_document_ids: [asset3.id] }
+        end.to change { collection.reload.member_objects.size }.by(1)
+        expect(response).to redirect_to routes.url_helpers.dashboard_collection_path(collection, locale: 'en')
+        expect(assigns[:collection].member_objects).to match_array [asset1, asset2, asset3]
+      end
+
       it "removes members from the collection" do
         # TODO: Using size until count is fixed https://github.com/projecthydra-labs/activefedora-aggregation/issues/78
         expect do