From 8afa8bc2bbe0c12f4777a91acf432f4f98b411b1 Mon Sep 17 00:00:00 2001 From: Justin Coyne Date: Wed, 28 Nov 2018 20:15:58 -0600 Subject: [PATCH] There is no need to assert_content_model in ActiveFedora 8. It already does it when you create a resource: https://github.com/samvera/active_fedora/blob/release-8.x/lib/active_fedora/persistence.rb#L137 Reverts part of https://github.com/sul-dlss/dor-services/commit/b5cb9829961c01d1cc1e8acb6b356e2e0927d686 --- lib/dor/indexers/identifiable_indexer.rb | 2 -- lib/dor/services/registration_service.rb | 2 -- 2 files changed, 4 deletions(-) diff --git a/lib/dor/indexers/identifiable_indexer.rb b/lib/dor/indexers/identifiable_indexer.rb index 2f05ab56..ba38d1e0 100644 --- a/lib/dor/indexers/identifiable_indexer.rb +++ b/lib/dor/indexers/identifiable_indexer.rb @@ -17,8 +17,6 @@ def initialize(resource:) # @return [Hash] the partial solr document for identifiable concerns def to_solr solr_doc = {} - resource.assert_content_model - solr_doc[Dor::INDEX_VERSION_FIELD] = Dor::VERSION solr_doc['indexed_at_dtsi'] = Time.now.utc.xmlschema resource.datastreams.values.each do |ds| diff --git a/lib/dor/services/registration_service.rb b/lib/dor/services/registration_service.rb index 6be7d3ee..7ce611c2 100644 --- a/lib/dor/services/registration_service.rb +++ b/lib/dor/services/registration_service.rb @@ -132,8 +132,6 @@ def register_object(params = {}) Array(params[:seed_datastream]).each { |datastream_name| new_item.build_datastream(datastream_name) } Array(params[:initiate_workflow]).each { |workflow_id| new_item.create_workflow(workflow_id, !new_item.new_record?, workflow_priority) } - new_item.assert_content_model - new_item.class.ancestors.select { |x| x.respond_to?(:to_class_uri) && x != ActiveFedora::Base }.each do |parent_class| new_item.add_relationship(:has_model, parent_class.to_class_uri) end