pennmarc 1.0.22 → 1.0.23

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d9222d0bfd9ca8704ec91ef01d1e27ce90711506b648f2d0006fc2270dd6d82d
-  data.tar.gz: c250833003d00f8d2b12a6c8e4043d9eadcdaaa33bba8ae70468e3210cdb1780
+  metadata.gz: 7c906223a9bcdb4ba4792ac50462b8c57e41e71fa1e83cd8e63018d360ebfe0a
+  data.tar.gz: 5d1a420e07734370c95ab1f83d3c519a1d75a7faaa807f1d3ee436808779c586
 SHA512:
-  metadata.gz: b9a1c42746b7d1448df78aad4528c1bf20731feeadc073e4937d0722bcfbed6ec42c104ae2d1241cf31830e6c7e50b0b6ab125140cf9d232511530a7b82861ef
-  data.tar.gz: c143ab4e9056f6a62a06a90d01938d70a6b7088704930e0dcf402c392954c08164a9821f39f35d03d07023393ca1d64844fa56d16522155ec55ca4a9a35fcf9b
+  metadata.gz: 32e47c254c1a2bb78e861003c864bdbc22833439cff22ebe00148df081c74fed85f60e2f743b5ea65065e5db576e3a304710b1a68b2cdc1e08a1a5f7808e5fc3
+  data.tar.gz: 6051840a95afdcb41640abf2c42ce699deddc93970c5d117cb445474d4f5998996ee05651ec1e59da3e45886e00f6c058b881b1bf9103de92deb5cf6d868ce80

data/lib/pennmarc/helpers/date.rb

@@ -8,14 +8,14 @@ module PennMARC
       # Publication date is a four-digit year found in position 7-10 and may contain 'u' characters to represent
       # partially known dates. We replace any occurrences of 'u' with '0' before converting to DateTime object.
       # @param [MARC::Record] record
-      # @return [DateTime, nil] The publication date, or nil if date found in record is invalid
+      # @return [Time, nil] The publication date, or nil if date found in record is invalid
       def publication(record)
         record.fields('008').filter_map { |field|
           four_digit_year = sanitize_partially_known_date(field.value[7, 4], '0')
 
           next if four_digit_year.blank?
 
-          DateTime.new(four_digit_year.to_i)
+          Time.new(four_digit_year.to_i)
         }.first
       end
 
@@ -23,7 +23,7 @@ module PennMARC
       # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing. The enriched
       # metadata provided by the Alma API does not include the date created value, so we can't work with that here.
       # @param [MARC::Record] record
-      # @return [DateTime, nil] The date added, or nil if date found in record is invalid
+      # @return [Time, nil] The date added, or nil if date found in record is invalid
       def added(record)
         record.fields(Enriched::Pub::ITEM_TAG).flat_map { |field|
           subfield_values(field, Enriched::Pub::ITEM_DATE_CREATED).filter_map do |date_added|
@@ -34,9 +34,10 @@ module PennMARC
 
             format = date_added.size == 10 ? '%Y-%m-%d' : '%Y-%m-%d %H:%M:%S'
 
-            DateTime.strptime(date_added, format)
+            Time.strptime(date_added, format)
           rescue StandardError => e
-            puts "Error parsing date in date added subfield: #{date_added} - #{e}"
+            puts 'Error parsing date in date added subfield. ' \
+                 "mmsid: #{Identifier.mmsid(record)}, value: #{date_added}, error: #{e}"
             nil
           end
         }.max
@@ -46,7 +47,7 @@ module PennMARC
       # Date last updated is a sixteen character String recorded in
       # {https://www.iso.org/iso-8601-date-and-time-format.html ISO 8601} format.
       # @param [MARC::Record] record
-      # @return [DateTime, nil] The date last updated, or nil if date found in record is invalid
+      # @return [Time, nil] The date last updated, or nil if date found in record is invalid
       def last_updated(record)
         record.fields('005').filter_map { |field|
           begin
@@ -56,9 +57,10 @@ module PennMARC
 
             next if date_time_string.start_with?('0000')
 
-            DateTime.iso8601(date_time_string).to_datetime
-          rescue ArgumentError => e
-            puts "Error parsing last updated date: #{date_time_string} - #{e}"
+            Time.strptime(date_time_string, '%Y%m%d%H%M%S.%N')
+          rescue StandardError => e
+            puts 'Error parsing last updated date. ' \
+                 "mmsid: #{Identifier.mmsid(record)}, value: #{date_time_string}, error: #{e}"
             nil
           end
         }.first

data/lib/pennmarc/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PennMARC
-  VERSION = '1.0.22'
+  VERSION = '1.0.23'
 end

data/spec/lib/pennmarc/helpers/date_spec.rb

@@ -10,7 +10,7 @@ describe 'PennMARC::Date' do
     let(:fields) { [marc_control_field(tag: '008', value: '130827s2010 nyu o 000 1 eng d')] }
 
     it 'returns publication date' do
-      expect(helper.publication(record)).to eq(DateTime.new(2010))
+      expect(helper.publication(record)).to eq(Time.new(2010))
     end
 
     it 'returns a year value' do
@@ -33,7 +33,7 @@ describe 'PennMARC::Date' do
       end
 
       it 'returns only the expected date_added value' do
-        expect(helper.added(record)).to eq DateTime.strptime('2023-10-19', '%Y-%m-%d')
+        expect(helper.added(record)).to eq Time.strptime('2023-10-19', '%Y-%m-%d')
       end
 
       it 'does not output any warning to STDOUT' do
@@ -47,7 +47,7 @@ describe 'PennMARC::Date' do
       let(:fields) { [marc_field(tag: 'itm', subfields: { q: '2023-06-28' })] }
 
       it 'returns expected value' do
-        expect(helper.added(record)).to eq(DateTime.strptime('2023-06-28', '%Y-%m-%d'))
+        expect(helper.added(record)).to eq(Time.strptime('2023-06-28', '%Y-%m-%d'))
       end
 
       it 'returns a year value' do
@@ -59,7 +59,7 @@ describe 'PennMARC::Date' do
       let(:fields) { [marc_field(tag: 'itm', subfields: { q: '2023-06-29 11:04:30:10' })] }
 
       it 'returns expected value' do
-        expect(helper.added(record)).to eq(DateTime.strptime('2023-06-29 11:04:30:10', '%Y-%m-%d %H:%M:%S'))
+        expect(helper.added(record)).to eq(Time.strptime('2023-06-29 11:04:30:10', '%Y-%m-%d %H:%M:%S'))
       end
 
       it 'returns a year value' do
@@ -74,12 +74,15 @@ describe 'PennMARC::Date' do
       end
 
       it 'returns most recent date' do
-        expect(helper.added(record)).to eq(DateTime.strptime('2023-06-29', '%Y-%m-%d'))
+        expect(helper.added(record)).to eq(Time.strptime('2023-06-29', '%Y-%m-%d'))
       end
     end
 
     context 'with invalid date' do
-      let(:fields) { [marc_field(tag: 'itm', subfields: { q: 'invalid date' })] }
+      let(:fields) do
+        [marc_control_field(tag: '001', value: 'mmsid'),
+         marc_field(tag: 'itm', subfields: { q: 'invalid date' })]
+      end
 
       it 'returns nil' do
         expect(helper.added(record)).to be_nil
@@ -88,7 +91,8 @@ describe 'PennMARC::Date' do
       it 'outputs error message' do
         expect {
           helper.added(record)
-        }.to output("Error parsing date in date added subfield: invalid date - invalid date\n").to_stdout
+        }.to output('Error parsing date in date added subfield. mmsid: mmsid, value: invalid date, ' \
+                    "error: invalid date or strptime format - `invalid date' `%Y-%m-%d %H:%M:%S'\n").to_stdout
       end
     end
   end
@@ -97,7 +101,7 @@ describe 'PennMARC::Date' do
     let(:fields) { [marc_field(tag: '005', subfields: { q: '20230213163851.1' })] }
 
     it 'returns date last updated' do
-      expect(helper.last_updated(record)).to eq(DateTime.iso8601('20230213163851.1').to_datetime)
+      expect(helper.last_updated(record)).to eq(Time.strptime('20230213163851.1', '%Y%m%d%H%M%S.%N'))
     end
 
     it 'returns year value' do
@@ -105,7 +109,10 @@ describe 'PennMARC::Date' do
     end
 
     context 'with invalid date' do
-      let(:fields) { [marc_field(tag: '005', subfields: { q: 'invalid date' })] }
+      let(:fields) do
+        [marc_control_field(tag: '001', value: 'mmsid'),
+         marc_field(tag: '005', subfields: { q: 'invalid date' })]
+      end
 
       it 'returns nil' do
         expect(helper.last_updated(record)).to be_nil
@@ -114,7 +121,8 @@ describe 'PennMARC::Date' do
       it 'outputs error message' do
         expect {
           helper.last_updated(record)
-        }.to output("Error parsing last updated date: invalid date - invalid date\n").to_stdout
+        }.to output('Error parsing last updated date. mmsid: mmsid, value: invalid date, ' \
+                    "error: invalid date or strptime format - `invalid date' `%Y%m%d%H%M%S.%N'\n").to_stdout
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pennmarc
 version: !ruby/object:Gem::Version
-  version: 1.0.22
+  version: 1.0.23
 platform: ruby
 authors:
 - Mike Kanning
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-24 00:00:00.000000000 Z
+date: 2024-06-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport