use valkyrie to power collection member removal from Dashboard

[straleyb]

Fixes #4914

This object has been updated to use Valkyrie when deleting collections. Instead of responding to #member_of_collections, it responds to #member_of_collection_ids. Then you can call delete using the id of the collection and then persist the valkyrie object.

[jeremyf]

I don't know if we need a test case for use valkyrie?

hyrax/spec/controllers/hyrax/dashboard/collections_controller_spec.rb

Lines 179 to 187 in 5faf403

	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
	put :update, params: { id: collection,
	collection: { members: 'remove' },
	batch_document_ids: [asset2] }
	end.to change { asset2.reload.member_of_collections.size }.by(-1)
	expect(assigns[:collection].member_objects).to match_array [asset1]
	end

[no-reply]

@jeremyf i think the use_valkyrie: case is tested in the CI matrix.

[no-reply]

@straleyb since this seems to work with valkyrie, i'd like to just squash it down to the one branch.

we don't return these objects, so doing this behavior with valkyrie behind the scenes is a no-impact refactor. i'm going to approve and merge this, then submit a follow-up to remove the argument and case block.