ddr-core 1.4.0 → 1.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1b06e941172b8ffcea5c52a0fe4eb0a975c3538139583a7f1c53a9e57d6b2582
-  data.tar.gz: 823d412215cd84eeaf55cecff8653f44737781c09f279363f23c5412ecabe950
+  metadata.gz: 77b5fee9288f31ca931ecca5d6a39bbbb114512ccc6a998dd05849359a149c3a
+  data.tar.gz: 4148903d813c21aeafffbdb1d5d5366bf9941db50b6a94289eb7840ea0786b51
 SHA512:
-  metadata.gz: e7b0295a95960c7af8be022f008f67d6ca3055f48210c0aa6109d48fac98d0a3fa7df9f686f99f14581e29eab347efdd67ae50f844d2c4fca40e5f68c35f48f3
-  data.tar.gz: f20c6dad3c41af199dab00061e24d2b4df696a0e915e0ff118944dcabfdda9e6bf2fb82fe503314340affab6f2def02bb34b911985ea6f72a989e891c417c6bd
+  metadata.gz: 5f7fd2db07bb6d6e8c7bcec2b887891989103abb4c92fb9ea5d5be996f8a1063ced102f9ca5901a1d17919c898e4d8d8a2ea70c175fa3f14a6e2c3b306017e12
+  data.tar.gz: 54811e462c8e216a757f0eb6e77a95ae834b5f87a99ef23f5fe6773194409537dd745bdbb0a8e3a92c3003adc5604e5bc05cc99680c9d0d3ad157720285638e5

data/app/models/concerns/ddr/solr_document_behavior.rb

@@ -5,34 +5,41 @@ module Ddr
   module SolrDocumentBehavior
     extend ActiveSupport::Concern
 
+    class NotFound < Error; end
+
     included do
-      alias_method :pid, :id
+      delegate :query_service,
+               :resource_factory,
+               to: :metadata_adapter
     end
 
-    class NotFound < Error; end
-
     module ClassMethods
       def find(id)
-        query = Ddr::Index::Query.new { id id }
-        if doc = query.docs.first
-          return doc
-        end
-        raise NotFound, "SolrDocument not found for \"#{id}\"."
+        raise ArgumentError, "ID argument must be present." if id.blank?
+
+        response = Blacklight.default_index.find(id)
+        response.documents.first
+
+      rescue Blacklight::Exceptions::RecordNotFound => e
+        raise NotFound, e
       end
 
       def find_by_permanent_id(ark)
-        if ark.blank?
-          raise ArgumentError, "ARK argument must be present."
-        end
-        query = Ddr::Index::Query.new do
-          q *:*
-          where permanent_id: ark
-        end
-        if doc = query.docs.first
-          return doc
+        raise ArgumentError, "ARK argument must be present." if ark.blank?
+
+        response = Blacklight.default_index.search(q: "#{Ddr::Index::Fields::PERMANENT_ID}:\"#{ark}\"", rows: 1)
+
+        if doc = response.documents.first
+          doc
+        else
+          raise NotFound, "Permanent ID: #{ark}"
         end
-        raise NotFound, "SolrDocument not found for permanent id \"#{ark}\"."
       end
+    end # ClassMethods
+
+    def pid
+      warn "DEPRECATION: #pid is deprecated. Use #id instead."
+      id
     end
 
     def inspect
@@ -56,8 +63,12 @@ module Ddr
       super
     end
 
+    def metadata_adapter
+      Valkyrie::MetadataAdapter.find(:index_solr)
+    end
+
     def resource
-      @resource ||= Valkyrie::MetadataAdapter.find(:index_solr).query_service.find_by(id: id)
+      @resource ||= resource_factory.to_resource(object: self)
     end
 
     def object_create_date
@@ -134,7 +145,7 @@ module Ddr
     end
 
     def targets_count
-      @targets_count = targets.count
+      @targets_count ||= targets.count
     end
 
     def has_target?
@@ -150,7 +161,7 @@ module Ddr
     end
 
     def roles
-      @roles ||= resource.roles
+      resource.roles
     end
 
     def resource_roles
@@ -176,9 +187,14 @@ module Ddr
       Ddr::Contact.call(research_help_contact) if research_help_contact
     end
 
-    def parent_id
-      is_part_of || is_member_of_collection
-    end
+    #
+    # Fall back to PARENT_ID field, which supersedes IS_PART_OF and _IS_MEMBER_OF_COLLECTION
+    # for this usage.
+    # Cf. https://duldev.atlassian.net/browse/DDR-2296
+    #
+    # def parent_id
+    #   is_part_of || is_member_of_collection
+    # end
 
     def has_parent?
       parent_id.present?
@@ -307,10 +323,6 @@ module Ddr
 
     private
 
-    def query_service
-      @query_service ||= Valkyrie::MetadataAdapter.find(:index_solr).query_service
-    end
-
     def get_date(field)
       parse_date(self[field])
     end

data/lib/ddr/index/connection.rb

@@ -13,8 +13,6 @@ module Ddr::Index
       delegate [:get, :post, :paginate] => :solr
 
       def solr
-        # Changed for Valkyrie
-        # RSolr.connect(ActiveFedora.solr_config)
         Blacklight.default_index.connection
       end
 

data/lib/ddr/index/fields.rb

@@ -37,8 +37,8 @@ module Ddr::Index
     DATE_SORT                     = Field.new :date_sort, :sortable
     DC_IS_PART_OF                 = Field.new :isPartOf, :symbol
     DEPOSITOR                     = Field.new :depositor, :stored_sortable
-    # DDR-1997 - description should always be indexed as _tesim regardless of length - Valkyrie indexes fields > 1000
-    # characters in length as _tsim only
+    # DDR-1997 - description should always be indexed as _tesim regardless of length.
+    # Valkyrie indexes fields > 1000 characters in length as _tsim only.
     DESCRIPTION                   = Field.new :description, :stored_searchable # stored_searchable produces _tesim
     DISPLAY_FORMAT                = Field.new :display_format, :stored_sortable
     DOI                           = Field.new :doi, :symbol

data/lib/ddr/index/query_result.rb

@@ -92,9 +92,10 @@ module Ddr::Index
     end
 
     def page(num)
-      page_params = params.dup
-      page_size = page_params.delete(:rows) || PAGE_SIZE
-      response = Connection.page(num, page_size, "select", params: page_params)
+      data = params.dup
+      page_size = data.delete(:rows) || PAGE_SIZE
+      opts = { method: :post, data: data }
+      response = Connection.page(num, page_size, 'select', opts)
       response.docs
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ddr-core
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.5.3
 platform: ruby
 authors:
 - Jim Coble
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-18 00:00:00.000000000 Z
+date: 2021-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activeresource
@@ -89,14 +89,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: '2.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: '2.1'
 - !ruby/object:Gem::Dependency
   name: grouper-rest-client
   requirement: !ruby/object:Gem::Requirement
@@ -157,16 +157,16 @@ dependencies:
   name: omniauth-shibboleth
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: virtus
   requirement: !ruby/object:Gem::Requirement
@@ -295,7 +295,6 @@ dependencies:
         version: '0'
 description: Models used in the Duke Digital Repository
 email:
-- lib-drs@duke.edu
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -388,7 +387,6 @@ files:
 - lib/ddr/auth/web_auth_context.rb
 - lib/ddr/core.rb
 - lib/ddr/core/engine.rb
-- lib/ddr/core/version.rb
 - lib/ddr/error.rb
 - lib/ddr/files.rb
 - lib/ddr/fits.rb

data/lib/ddr/core/version.rb

@@ -1,5 +0,0 @@
-module Ddr
-  module Core
-    VERSION = '1.4.0'
-  end
-end