uc3-dmp-id 0.1.74 → 0.1.76

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 216eee2772c1e88ec1426ef8d0414e61958cfd3185149c999d19c02e38a9d32d
-  data.tar.gz: f97512390c550f0714381c75a8d8c9365de356d0d02db87be16081608084ac89
+  metadata.gz: 9d18daee8127652273ede255b35ce9c8401354f5082c4f1806737b2486dc7178
+  data.tar.gz: a2b7599f05bc790822d80e5ec7dd83a64ef42ef5fd52cc4eae5db90792d0185a
 SHA512:
-  metadata.gz: 26491648692773b00eeae89efeb47f1f3c9c33897705974c6497193b21cbfc4a4b184b7ba0e9976751b2dad9e4c7d72c2cfc2867458bbbc81c4803614a4bff12
-  data.tar.gz: 59a869cb83864807167ccb21a4ce9f409afd73d60895d32ba91270da5fa7715e59d8e5310f576d6c582e40ec33961f2fe6fa2715622d5a71c5448234ad915830
+  metadata.gz: 72080a3809333bd1c3dacd7540c7bf108bf99b8fa0fa6b0450aae2180af8ff175ab273f9509f26043c01a14c0d45a478c4c242798c5f571a1befcdfe5a9784d8
+  data.tar.gz: 887b4651957305c709eaec3139735b83854d432dd8894777c0437ee4d243d09ff98a2244f9dd9486474ec89e56c8470d8b8ddad527f2507d83b7c0c903c544e2

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

@@ -45,35 +45,32 @@ module Uc3DmpId
         raise UpdaterError, Helper::MSG_DMP_FORBIDDEN if owner != updater
 
         # Handle any changes to the dmphub_modifications section
-        payload = _process_harvester_mods(client:, p_key:, json: payload, logger:)
-        logger&.debug(message: 'Mods after process_harvester_mods', details: payload)
+        version = _process_harvester_mods(client:, p_key:, json: payload, logger:)
+        logger&.debug(message: 'Version after process_harvester_mods', details: version)
 
-        # Remove the version info because we don't want to save it on the record
-        payload.delete('dmphub_versions')
-
-        # Splice the assertions
-        payload = _process_modifications(owner:, updater:, version:, mods: payload, logger:)
-        logger&.debug(message: 'Mods after process_modifications', details: payload)
+        # Remove the version info any any lingering modification blocks
+        version.delete('dmphub_versions')
+        version.delete('dmphub_modifications')
 
         # Set the :modified timestamps
         now = Time.now.utc
-        payload['modified'] = now.iso8601
-        payload['dmphub_modification_day'] = now.strftime('%Y-%m-%d')
+        version['modified'] = now.iso8601
+        version['dmphub_modification_day'] = now.strftime('%Y-%m-%d')
 
         # Save the changes
-        resp = client.put_item(json: payload, logger:)
+        resp = client.put_item(json: version, logger:)
         raise UpdaterError, Helper::MSG_DMP_UNABLE_TO_VERSION if resp.nil?
 
         # Send the updates to EZID
-        _post_process(provenance:, json: payload, logger:)
+        _post_process(provenance:, json: version, logger:)
 
         # Return the new version record
         logger.info(message: "Updated DMP ID: #{p_key}") if logger.respond_to?(:debug)
 
         # Append the :dmphub_versions Array
-        json = JSON.parse({ dmp: payload }.to_json)
-        json = Versioner.append_versions(p_key:, dmp: json, client:, logger:)
-        Helper.cleanse_dmp_json(json:)
+        out = JSON.parse({ dmp: version }.to_json)
+        out = Versioner.append_versions(p_key:, dmp: out, client:, logger:)
+        Helper.cleanse_dmp_json(json: out)
       end
       # rubocop:enable Metrics/AbcSize, Metrics/MethodLength
       # rubocop:enable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
@@ -134,25 +131,6 @@ module Uc3DmpId
       end
       # rubocop:enable Metrics/AbcSize
 
-      # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
-      def _process_modifications(owner:, updater:, version:, mods:, logger: nil)
-        return version unless mods.is_a?(Hash) && !updater.nil?
-        return mods unless version.is_a?(Hash) && !owner.nil?
-
-        logger.debug(message: 'Modifications before merge.', details: mods) if logger.respond_to?(:debug)
-        keys_to_retain = version.keys.select do |key|
-          (key.start_with?('dmphub_') && !%w[dmphub_modifications dmphub_versions].include?(key)) ||
-            key.start_with?('PK') || key.start_with?('dmproadmap_related_identifiers')
-        end
-        keys_to_retain.each do |key|
-          mods[key] = version[key]
-        end
-        mods['SK'] = Helper::DMP_LATEST_VERSION
-        logger.debug(message: 'Modifications after merge.', details: mods) if logger.respond_to?(:debug)
-        mods
-      end
-      # rubocop:enable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
-
       # Once the DMP has been updated, we need to update it's DOI metadata
       # -------------------------------------------------------------------------
       # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
@@ -229,31 +207,29 @@ module Uc3DmpId
             logger&.debug(message: "Updating status for #{key} from #{original['related_works'][key]['status']} to #{related['status']}")
             original['related_works'][key]['status'] = related['status']
 
-            existing = json['dmproadmap_related_identifiers'].select do |ri|
+            existing = version['dmproadmap_related_identifiers'].select do |ri|
               ri['identifier'] == key
             end
 
             # Add it if it was approved and doesn't exist in dmproadmap_related_identifiers
             if related['status'] == 'approved' && existing.empty?
-              json['dmproadmap_related_identifiers'] << {
+              version['dmproadmap_related_identifiers'] << {
                 identifier: key,
                 work_type: related['work_type'],
                 type: related['type'],
                 descriptor: related['descriptor'],
                 citation: related['citation']
-              }.to_json
-            elsif related['status'] != 'approved' && existing.any?
+              }
+            elsif related['status'] == 'rejected' && existing.any?
               # otherwise remove it
-              json['dmproadmap_related_identifiers'] = json['dmproadmap_related_identifiers'].reject { |ri| ri == existing.first }
+              version['dmproadmap_related_identifiers'] = version['dmproadmap_related_identifiers'].reject { |ri| ri == existing.first }
             end
           end
         end
 
         logger&.debug(message: 'Updating HARVESTER_MODS with:', details: original)
-        client.put_item(json: original, logger:)
-        json.delete('dmphub_modifications')
-        logger&.debug(message: 'After deleting dmphub_modifications:', details: json)
-        json
+        logger&.debug(message: 'Returning updated VERSION:', details: version)
+        version
       end
     end
   end

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

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: uc3-dmp-id
 version: !ruby/object:Gem::Version
-  version: 0.1.74
+  version: 0.1.76
 platform: ruby
 authors:
 - Brian Riley