uc3-dmp-id 0.1.59 → 0.1.61

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59344bea0d31538592f45cc51bda5971e9834ba66ba6b75b99f25bc188307782
-  data.tar.gz: b57a971d2734b20cda13bd67df9f8c1e1f14f3c80d17eb6dc92d0a00ac28e3e0
+  metadata.gz: bb07288c4e1b21217a2ca9ed4df6a8c7243968c2cea3dbd0474796b6399d2a09
+  data.tar.gz: 665c7eb28c3fd99e73c50f06b9bd95be7f96bb95b0a0c551634c6b5c941fd9db
 SHA512:
-  metadata.gz: 90bee71bd0005f49adb736c0ba5fad0d75e40b12815cb46bc0a73352e841bd7b2372e7038282427d7dbfda05fd7f060e04500c4588ce578424a7632bd19510fd
-  data.tar.gz: 958e3d10549189e8e7d47d950cb27dddb84623f1d8b13d23386b555c773cfe3b8ecbef41d4221a225b84dbcd7f5079701f98218947aff58156fd2e8fe5b18946
+  metadata.gz: 8f196c1f061e5fcbd9085f5d483936797b3fe97bdb3e74b9bb8e2fc83effdefc4ecffbb28a0f0b478418fff6e8b6edd21b35cb2e103e6d7db1696e33e94d9d2f
+  data.tar.gz: b4adad6f533baff187391f415123134b93ffc0c01a93771f6519d8e935f91966c32c0d2ff2c051a98b09a596a615c86226b1f7ba6eeda72092b34b127edac403

data/lib/uc3-dmp-id/finder.rb

@@ -17,19 +17,36 @@ module Uc3DmpId
     MSG_MISSING_PROV_ID = 'No Provenance identifier was provided. \
                            Expected: `{ "dmp_id": { "identifier": "value", "type": "value" }`'
 
-    ORCID_DOMAIN = 'orcid.org'
-    ROR_DOMAIN = 'ror.org'
+    ORCID_DOMAIN = 'https://orcid.org/'
+    ROR_DOMAIN = 'https://ror.org/'
 
     class << self
       # TODO: Replace this with ElasticSearch
       def search_dmps(args:, logger: nil)
+        # Fetch the DMPs for each of the possible filter options
         client = Uc3DmpDynamo::Client.new(table: ENV['DYNAMO_INDEX_TABLE'])
-        return _by_owner(owner: args['owner'], client:, logger:) unless args['owner'].nil?
-        return _by_org(org: args['org'], client:, logger:) unless args['org'].nil?
-        return _by_funder(funder: args['funder'], client:, logger:) unless args['funder'].nil?
-        return _by_featured(client:, logger:) if args.fetch('featured', 'false').to_s.downcase == 'true'
+        owner = args['owner']
+        org = args['org']
+        funder = args['funder']
 
-        return _publicly_visible(client:, logger:)
+        owner_pks = owner.nil? ? [] : _by_owner(owner: owner, client:, logger:)
+        org_pks = org.nil? ? [] : _by_org(org: org, client:, logger:)
+        funder_pks = funder.nil? ? [] : _by_funder(funder: funder, client:, logger:)
+        return [] if owner_pks.empty? && org_pks.empty? && funder_pks.empty?
+
+        # Only use the DMPs that fit all of the filter criteria
+        dmps = owner_pks & org_pks & funder_pks
+        return [] if dmps.nil? || dmps.empty?
+
+        # Perfom search operations
+
+        # Handle sort
+
+        # Handle pagination
+
+        # Fetch full DMP records
+        client = Uc3DmpDynamo::Client.new(table: ENV['DYNAMO_TABLE'])
+        _fetch_dmps(client:, pks: dmps.map { |dmp| dmp['pk'] }, logger:)
       end
 
       # Find a DMP based on the contents of the incoming JSON
@@ -133,73 +150,41 @@ module Uc3DmpId
 
       # Fetch the DMP IDs for the specified person's ORCID (or email)
       def _by_owner(owner:, client: nil, logger: nil)
-        orcid_regex = /^([0-9A-Z]{4}-){3}[0-9A-Z]{4}$/
+        orcid_regex = /^([0-9a-zA-Z]{4}-){3}[0-9a-zA-Z]{4}$/
         email_regex = /^[^\s@]+@[^\s@]+\.[^\s@]+$/
-        email = owner.strip unless (owner.to_s.strip.gsub('%40', '@') =~ email_regex).nil?
-
-        if !(owner.to_s.strip =~ orcid_regex).nil?
-          args = {
-            filter_expression: 'contains(people_ids, :orcids) AND SK = :sk',
-            expression_attribute_values: {
-              ':sk': 'METADATA',
-              ':orcids': [
-                "http://#{ORCID_DOMAIN}/#{owner}",
-                "https://#{ORCID_DOMAIN}/#{owner}"
-              ]
-            }
-          }
-        elsif !email.nil?
-          args = {
-            filter_expression: 'contains(people, :email) AND SK = :sk',
-            expression_attribute_values: { ':sk': 'METADATA', ':email': [email.gsub('%40', '@')] }
-          }
-        else
-          # It wasn't an email or ORCID, so return an empty array
-          return []
-        end
-        logger&.debug(message: 'Fetch relevant DMPs _by_owner - scan args', details: args)
-        client = Uc3DmpDynamo::Client.new if client.nil?
-        _process_search_response(response: client.scan(args:))
+        orcid = owner.to_s.strip
+        return [] if (orcid =~ orcid_regex).nil? && (orcid =~ email_regex).nil?
+
+        orcid = "#{ORCID_DOMAIN}#{orcid}" unless (orcid =~ orcid_regex).nil?
+        resp = client.get_item(key: { PK: 'PERSON_INDEX', SK: orcid }, logger:)
+        return [] unless resp.is_a?(Hash)
+
+        logger&.debug(message: "DMPs for PERSON #{orcid}", details: resp)
+        resp.fetch('dmps', [])
       end
 
       # Fetch the DMP IDs for the specified organization/institution
       def _by_org(org:, client: nil, logger: nil)
         regex = /^[a-zA-Z0-9]+$/
-        ror = org.strip unless (org.to_s =~ regex).nil?
+        ror = "#{ROR_DOMAIN}/#{org.strip}" unless (org.to_s =~ regex).nil?
 
-        args = {
-          filter_expression: 'contains(affiliation_ids, :rors) AND SK = :sk',
-          expression_attribute_values: {
-            ':sk': 'METADATA',
-            ':rors': [
-              "http://#{ROR_DOMAIN}/#{ror}",
-              "https://#{ROR_DOMAIN}/#{ror}"
-            ]
-          }
-        }
-        logger&.debug(message: 'Fetch relevant DMPs _by_org - scan args', details: args)
-        client = Uc3DmpDynamo::Client.new if client.nil?
-        _process_search_response(response: client.scan(args:))
+        resp = client.get_item(key: { PK: 'AFFILIATION_INDEX', SK: ror }, logger:)
+        return [] unless resp.is_a?(Hash)
+
+        logger&.debug(message: "DMPs for AFFILIATION #{ror}", details: resp)
+        resp.fetch('dmps', [])
       end
 
       # Fetch the DMP IDs for the specified funder
       def _by_funder(funder:, client: nil, logger: nil)
         regex = /^[a-zA-Z0-9]+$/
-        ror = funder.strip unless (funder.to_s =~ regex).nil?
+        ror = "#{ROR_DOMAIN}/#{funder.strip}" unless (funder.to_s =~ regex).nil?
 
-        args = {
-          filter_expression: 'contains(funder_ids, :rors) AND SK = :sk',
-          expression_attribute_values: {
-            ':sk': 'METADATA',
-            ':rors': [
-              "http://#{ROR_DOMAIN}/#{ror}",
-              "https://#{ROR_DOMAIN}/#{ror}"
-            ]
-          }
-        }
-        logger&.debug(message: 'Fetch relevant DMPs _by_funder - scan args', details: args)
-        client = Uc3DmpDynamo::Client.new if client.nil?
-        _process_search_response(response: client.scan(args:))
+        resp = client.get_item(key: { PK: 'FUNDER_INDEX', SK: ror }, logger:)
+        return [] unless resp.is_a?(Hash)
+
+        logger&.debug(message: "DMPs for FUNDER #{ror}", details: resp)
+        resp.fetch('dmps', [])
       end
 
       # Fetch the DMP IDs that are marked as featured
@@ -224,6 +209,11 @@ module Uc3DmpId
         _process_search_response(response: client.scan(args:))
       end
 
+      # Fetches all of the DMPs by their PKs
+      def _fetch_dmps(client:, pks:, logger: null)
+
+      end
+
       # Transform the search results so that we do not include any of the DMPHub specific metadata
       def _process_search_response(response:)
         return [] unless response.is_a?(Array) && response.any?

data/lib/uc3-dmp-id/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Uc3DmpId
-  VERSION = '0.1.59'
+  VERSION = '0.1.61'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: uc3-dmp-id
 version: !ruby/object:Gem::Version
-  version: 0.1.59
+  version: 0.1.61
 platform: ruby
 authors:
 - Brian Riley
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-07-09 00:00:00.000000000 Z
+date: 2024-07-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json