pennmarc 1.4.1 → 1.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81ebb88f882d1b92922aa5dc7338056b6831d4857f91ba1ed1710deb2b678f09
-  data.tar.gz: 9db8caa363ce8ab4515d62612e033c4e73c185fef68c2a5c115a3520928f95b8
+  metadata.gz: 0b30436efad760f5e0ba2299abd4b95bc7209d3239e50e28f7bf983c388fa7e6
+  data.tar.gz: 4cf860b94dacc4f51a0198f907d5a3cd071204e984f59a46ce439de16505357a
 SHA512:
-  metadata.gz: d3a18e0ff157bdecd8a10cad8c72c4aeffece93dc06117740fb122e27ff972f5940f45b332c8bb3d6d57b05cff2c7b610fc57487b9ce7ae38700df96ca077764
-  data.tar.gz: 0cb04ac15f6f5aa1f7c86005b953bd86e5c952ab2de99479d04688d131bff991ec3a2bbe17a3cf2707bcfa613c71ed8beb2711951c5868c0d5cd4e240d0fca5a
+  metadata.gz: 164c8711a0540db31617a1d91a787a216ed24c8d7539656c30b0410693d3284f4ccdb6e838ff5220d0dbf3c7bd359ba83e305938a0f9c32dfc4de5b7213b305b
+  data.tar.gz: 693967b2f782cdae859656da8894e28ab329abfbb363ff2e51adbae940e3a6b0ff7b03f398ce4709b946967a89eabec6b5210725e094c2f645e462fb48f0779d

data/Gemfile

@@ -11,6 +11,7 @@ gem 'rake', '~> 13.0'
 gem 'upennlib-rubocop', require: false
 
 group :test, :development do
+  gem 'debug', platforms: %i[mri mingw x64_mingw]
   gem 'rspec', '~> 3.12'
 end
 
@@ -21,5 +22,5 @@ end
 group :development do
   gem 'puma'
   gem 'rackup'
-  gem 'yard', '~> 0.9'
+  gem 'yard'
 end

data/Gemfile.lock

@@ -1,14 +1,14 @@
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (8.0.3)
+    activesupport (8.1.2)
       base64
-      benchmark (>= 0.3)
       bigdecimal
       concurrent-ruby (~> 1.0, >= 1.3.1)
       connection_pool (>= 2.2.5)
       drb
       i18n (>= 1.6, < 2)
+      json
       logger (>= 1.4.2)
       minitest (>= 5.1)
       securerandom (>= 0.3)
@@ -16,63 +16,85 @@ GEM
       uri (>= 0.13.1)
     ast (2.4.3)
     base64 (0.3.0)
-    benchmark (0.4.1)
-    bigdecimal (3.3.1)
-    concurrent-ruby (1.3.5)
-    connection_pool (2.5.4)
+    bigdecimal (4.0.1)
+    concurrent-ruby (1.3.6)
+    connection_pool (3.0.2)
+    date (3.5.1)
+    debug (1.11.1)
+      irb (~> 1.10)
+      reline (>= 0.3.8)
     diff-lcs (1.6.2)
-    docile (1.4.0)
+    docile (1.4.1)
     drb (2.2.3)
-    i18n (1.14.7)
+    erb (6.0.1)
+    i18n (1.14.8)
       concurrent-ruby (~> 1.0)
-    json (2.13.2)
+    io-console (0.8.2)
+    irb (1.16.0)
+      pp (>= 0.6.0)
+      rdoc (>= 4.0.0)
+      reline (>= 0.4.2)
+    json (2.18.1)
     language_server-protocol (3.17.0.5)
     lcsort (0.9.1)
     library_stdnums (1.6.0)
     lint_roller (1.1.0)
     logger (1.7.0)
-    marc (1.3.0)
+    marc (1.4.0)
       nokogiri (~> 1.0)
       rexml
-    minitest (5.26.0)
+    minitest (6.0.1)
+      prism (~> 1.5)
     nio4r (2.7.4)
-    nokogiri (1.18.9-arm64-darwin)
+    nokogiri (1.19.0-arm64-darwin)
       racc (~> 1.4)
-    nokogiri (1.18.9-x64-mingw-ucrt)
+    nokogiri (1.19.0-x64-mingw-ucrt)
       racc (~> 1.4)
-    nokogiri (1.18.9-x86_64-darwin)
+    nokogiri (1.19.0-x86_64-darwin)
       racc (~> 1.4)
-    nokogiri (1.18.9-x86_64-linux-gnu)
+    nokogiri (1.19.0-x86_64-linux-gnu)
       racc (~> 1.4)
     parallel (1.27.0)
-    parser (3.3.9.0)
+    parser (3.3.10.1)
       ast (~> 2.4.1)
       racc
-    prism (1.4.0)
+    pp (0.6.3)
+      prettyprint
+    prettyprint (0.2.0)
+    prism (1.9.0)
+    psych (5.3.1)
+      date
+      stringio
     puma (7.1.0)
       nio4r (~> 2.0)
     racc (1.8.1)
-    rack (3.2.3)
+    rack (3.2.4)
     rackup (2.2.1)
       rack (>= 3)
     rainbow (3.1.1)
     rake (13.3.0)
-    regexp_parser (2.11.2)
-    rexml (3.4.1)
-    rspec (3.13.1)
+    rdoc (7.1.0)
+      erb
+      psych (>= 4.0.0)
+      tsort
+    regexp_parser (2.11.3)
+    reline (0.6.3)
+      io-console (~> 0.5)
+    rexml (3.4.4)
+    rspec (3.13.2)
       rspec-core (~> 3.13.0)
       rspec-expectations (~> 3.13.0)
       rspec-mocks (~> 3.13.0)
-    rspec-core (3.13.5)
+    rspec-core (3.13.6)
       rspec-support (~> 3.13.0)
     rspec-expectations (3.13.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.5)
+    rspec-mocks (3.13.7)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-support (3.13.4)
-    rubocop (1.79.2)
+    rspec-support (3.13.7)
+    rubocop (1.84.1)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
       lint_roller (~> 1.1.0)
@@ -80,20 +102,20 @@ GEM
       parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 2.9.3, < 3.0)
-      rubocop-ast (>= 1.46.0, < 2.0)
+      rubocop-ast (>= 1.49.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 4.0)
-    rubocop-ast (1.46.0)
+    rubocop-ast (1.49.0)
       parser (>= 3.3.7.2)
-      prism (~> 1.4)
+      prism (~> 1.7)
     rubocop-capybara (2.22.1)
       lint_roller (~> 1.1)
       rubocop (~> 1.72, >= 1.72.1)
-    rubocop-performance (1.25.0)
+    rubocop-performance (1.26.1)
       lint_roller (~> 1.1)
       rubocop (>= 1.75.0, < 2.0)
-      rubocop-ast (>= 1.38.0, < 2.0)
-    rubocop-rails (2.33.3)
+      rubocop-ast (>= 1.47.1, < 2.0)
+    rubocop-rails (2.34.3)
       activesupport (>= 4.2.0)
       lint_roller (~> 1.1)
       rack (>= 1.1)
@@ -102,22 +124,24 @@ GEM
     rubocop-rake (0.7.1)
       lint_roller (~> 1.1)
       rubocop (>= 1.72.1)
-    rubocop-rspec (3.6.0)
+    rubocop-rspec (3.9.0)
       lint_roller (~> 1.1)
-      rubocop (~> 1.72, >= 1.72.1)
+      rubocop (~> 1.81)
     ruby-progressbar (1.13.0)
     securerandom (0.4.1)
     simplecov (0.22.0)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
       simplecov_json_formatter (~> 0.1)
-    simplecov-html (0.12.3)
+    simplecov-html (0.13.2)
     simplecov_json_formatter (0.1.4)
+    stringio (3.2.0)
+    tsort (0.2.0)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    unicode-display_width (3.1.5)
-      unicode-emoji (~> 4.0, >= 4.0.4)
-    unicode-emoji (4.0.4)
+    unicode-display_width (3.2.0)
+      unicode-emoji (~> 4.1)
+    unicode-emoji (4.2.0)
     upennlib-rubocop (1.3.0)
       rubocop (~> 1.72)
       rubocop-capybara
@@ -125,18 +149,20 @@ GEM
       rubocop-rails
       rubocop-rake
       rubocop-rspec
-    uri (1.0.4)
-    yard (0.9.37)
+    uri (1.1.1)
+    yard (0.9.38)
 
 PLATFORMS
   arm64-darwin-21
   arm64-darwin-22
+  arm64-darwin-24
   x64-mingw-ucrt
   x86_64-darwin-21
   x86_64-linux
 
 DEPENDENCIES
   activesupport (>= 7)
+  debug
   lcsort
   library_stdnums (~> 1.6)
   marc (~> 1.2)
@@ -147,7 +173,7 @@ DEPENDENCIES
   rspec (~> 3.12)
   simplecov (~> 0.22)
   upennlib-rubocop
-  yard (~> 0.9)
+  yard
 
 BUNDLED WITH
    2.4.10

data/lib/pennmarc/helpers/creator.rb

@@ -357,25 +357,33 @@ module PennMARC
       # @param relator_map [Hash]
       # @return [Array<String>] name values from given tags
       def name_search_values(record:, tags:, relator_map:)
-        acc = record.fields(tags).filter_map do |field|
-          name_from_main_entry field, relator_map, should_convert_name_order: false, for_display: false
+        # Standard tags
+        standard_names = record.fields(tags).flat_map do |field|
+          [false, true].map do |should_convert|
+            name_from_main_entry(field, relator_map, should_convert_name_order: should_convert,
+                                                     for_display: false)
+          end
         end
 
-        acc += record.fields(tags).filter_map do |field|
-          name_from_main_entry field, relator_map, should_convert_name_order: true, for_display: false
+        # Author 700 fields
+        added_entry_personal_names = record.fields('700').filter_map do |field|
+          next unless describes_author?(field)
+
+          convert_name_order(field['a']) if field['a']
         end
 
-        acc += record.fields(['880']).filter_map do |field|
+        # Linked 880 fields
+        linked_names = record.fields('880').filter_map do |field|
           next unless subfield_value?(field, '6', /^(#{tags.join('|')})/)
 
-          suba = field.find_all(&subfield_in?(%w[a])).filter_map { |sf|
-            convert_name_order(sf.value)
-          }.first
-          oth = join_and_squish(field.find_all(&subfield_not_in?(%w[6 8 a t])).map(&:value))
-          join_and_squish [suba, oth]
+          sub_a = field['a'] ? convert_name_order(field['a']) : nil
+          others = join_and_squish(field.subfields.reject { |sf| %w[6 8 a t].include?(sf.code) }.map(&:value))
+
+          join_and_squish([sub_a, others])
         end
 
-        acc.uniq
+        # Merge and deduplicate
+        (standard_names + added_entry_personal_names + linked_names).compact.uniq
       end
 
       # Extract the information we care about from 1xx fields, map relator codes, and use appropriate punctuation

data/lib/pennmarc/helpers/production.rb

@@ -52,7 +52,7 @@ module PennMARC
         added_2xx = record.fields(%w[260 261 262])
                           .first(1)
                           .map do |field|
-          join_subfields(field, &subfield_not_in?(['6'])).squish
+                            join_subfields(field, &subfield_not_in?(['6'])).squish
         end
 
         if added_2xx.present?

data/lib/pennmarc/helpers/subject.rb

@@ -59,6 +59,7 @@ module PennMARC
       end
 
       # All Subjects for faceting
+      # When a library of congress subject heading has subdivisions, we also facet the decomposed main term ($a)
       #
       # @note this is ported mostly form MG's new-style Subject parsing
       # @param record [MARC::Record]
@@ -69,13 +70,18 @@ module PennMARC
           term_hash = build_subject_hash(field)
           next if term_hash.blank? || term_hash[:count]&.zero?
 
-          format_term type: :facet, term: term_hash
-        }.uniq
+          heading = format_term type: :facet, term: term_hash
+          main_term = trim_trailing(:period, term_hash[:subfield_a]) if term_hash[:lcsh] && term_hash[:subfield_a]
+
+          [heading, main_term].compact_blank
+        }.flatten.uniq
+
         override ? HeadingControl.term_override(values) : values
       end
 
       # All Subjects for display. This includes all {DISPLAY_TAGS} and {LOCAL_TAGS}. For tags that specify a source,
       # only those with an allowed source code (see ALLOWED_SOURCE_CODES) are included.
+      # When a library of congress subject heading has subdivisions, we also display the decomposed main term ($a)
       #
       # @param record [MARC::Record]
       # @param override [Boolean] to remove undesirable terms or not
@@ -85,8 +91,11 @@ module PennMARC
           term_hash = build_subject_hash(field)
           next if term_hash.blank? || term_hash[:count]&.zero?
 
-          format_term type: :display, term: term_hash
-        }.uniq
+          heading = format_term type: :display, term: term_hash
+          main_term = "#{trim_trailing(:period, term_hash[:subfield_a])}." if term_hash[:lcsh] && term_hash[:subfield_a]
+
+          [heading, main_term].compact_blank
+        }.flatten.uniq
         override ? HeadingControl.term_override(values) : values
       end
 
@@ -98,10 +107,10 @@ module PennMARC
       def childrens_show(record, override: true)
         values = subject_fields(record, type: :display, options: { tags: DISPLAY_TAGS, indicator2: '1' })
                  .filter_map { |field|
-          term_hash = build_subject_hash(field)
-          next if term_hash.blank? || term_hash[:count]&.zero?
+                   term_hash = build_subject_hash(field)
+                   next if term_hash.blank? || term_hash[:count]&.zero?
 
-          format_term type: :display, term: term_hash
+                   format_term type: :display, term: term_hash
         }.uniq
         override ? HeadingControl.term_override(values) : values
       end
@@ -114,10 +123,10 @@ module PennMARC
       def medical_show(record, override: true)
         values = subject_fields(record, type: :display, options: { tags: DISPLAY_TAGS, indicator2: '2' })
                  .filter_map { |field|
-          term_hash = build_subject_hash(field)
-          next if term_hash.blank? || term_hash[:count]&.zero?
+                   term_hash = build_subject_hash(field)
+                   next if term_hash.blank? || term_hash[:count]&.zero?
 
-          format_term type: :display, term: term_hash
+                   format_term type: :display, term: term_hash
         }.uniq
         override ? HeadingControl.term_override(values) : values
       end
@@ -242,11 +251,13 @@ module PennMARC
       #       heading. - MG
       # @todo do i need all this?
       # @param field [MARC::DataField]
-      # @return [Hash{Symbol => Integer, Array, Boolean}, Nil]
+      # @return [Hash{Symbol => Integer, Array, Boolean, String}, Nil]
       def build_subject_hash(field)
         term_info = { count: 0, parts: [], append: [], uri: nil,
                       local: field.indicator2 == '4' || field.tag.starts_with?('69'), # local subject heading
-                      vernacular: field.tag == '880' }
+                      vernacular: field.tag == '880',
+                      subfield_a: nil,
+                      lcsh: lcsh?(field) }
         field.each do |subfield|
           case subfield.code
           when '0', '6', '8', '5', '7'
@@ -263,6 +274,7 @@ module PennMARC
 
             term_info[:parts] << subfield.value.strip
             term_info[:count] += 1
+            term_info[:subfield_a] = trim_trailing(:comma, subfield.value.strip)
           when '2'
             term_info[:source] = subfield.value.strip
           when 'e', 'w'
@@ -289,6 +301,18 @@ module PennMARC
         term_info
       end
 
+      # Determine if field contains a library of congress subject heading
+      # @param field [MARC::Field]
+      # @return [Boolean]
+      def lcsh?(field)
+        return false unless field.respond_to?(:indicator2)
+
+        return true if field.indicator2.in? %w[0]
+        return true if field.indicator2 == '7' && subfield_values(field, '2').first == 'lcsh'
+
+        false
+      end
+
       # Determine if a field should be considered for Subject search inclusion. It must be either contained in
       # {SEARCH_TAGS}, be an 880 field otherwise linked to a valid Search tag, or be in {LOCAL_TAGS}.
       # @param field [MARC::DataField]

data/lib/pennmarc/helpers/title.rb

@@ -8,6 +8,7 @@ module PennMARC
     # We use these fields when retrieving auxiliary titles in the *search_aux methods:
     # {https://www.loc.gov/marc/bibliographic/bd130.html 130},
     # {https://www.loc.gov/marc/bibliographic/bd210.html 210},
+    # {https://www.loc.gov/marc/bibliographic/bd222.html 222},
     # {https://www.loc.gov/marc/bibliographic/bd245.html 245},
     # {https://www.loc.gov/marc/bibliographic/bd246.html 246},
     # {https://www.loc.gov/marc/bibliographic/bd247.html 247},
@@ -25,7 +26,7 @@ module PennMARC
     # {https://www.loc.gov/marc/bibliographic/bd711.html 711},
     # {https://www.loc.gov/marc/bibliographic/bd505.html 505}
     AUX_TITLE_TAGS = {
-      main: %w[130 210 240 245 246 247 440 490 730 740 830],
+      main: %w[130 210 222 240 245 246 247 440 490 730 740 830],
       related: %w[773 774 780 785],
       entity: %w[700 710 711],
       note: %w[505]
@@ -230,9 +231,9 @@ module PennMARC
         end
         other_titles += record.fields('740')
                               .filter_map do |field|
-          next unless field.indicator2.in? ['', ' ', '0', '1', '3']
+                                next unless field.indicator2.in? ['', ' ', '0', '1', '3']
 
-          join_subfields(field, &subfield_not_in?(%w[5 6 8]))
+                                join_subfields(field, &subfield_not_in?(%w[5 6 8]))
         end
         titles = other_titles + record.fields('880').filter_map do |field|
           next unless subfield_value? field, '6', /^(246|740)/

data/lib/pennmarc/mappings/locations.yml

@@ -995,12 +995,11 @@ pahref:
     - Pennsylvania Hospital Library
   display: Pennsylvania Hospital Library - Reference
 pahres:
-  specific_location: Pennsylvania Hospital Library - Reserve
+  specific_location: Pennsylvania Hospital Library
   library:
     - Health Sciences Libraries
     - Pennsylvania Hospital Library
-    - Reserve
-  display: Pennsylvania Hospital Library - Reserve
+  display: Pennsylvania Hospital Library
 pahresrm:
   specific_location: Pennsylvania Hospital Library - Reserve Room
   library:

data/lib/pennmarc/version.rb

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

data/pennmarc.gemspec

@@ -18,11 +18,11 @@ Gem::Specification.new do |s|
 
   s.required_ruby_version = '>= 3.2'
 
-  s.add_dependency 'activesupport', '>= 7'
+  s.add_dependency 'activesupport', '>= 8'
   s.add_dependency 'lcsort', '~> 0.9'
   s.add_dependency 'library_stdnums', '~> 1.6'
-  s.add_dependency 'marc', '~> 1.2'
-  s.add_dependency 'nokogiri', '~> 1.15'
+  s.add_dependency 'marc', '~> 1.4'
+  s.add_dependency 'nokogiri', '~> 1.19'
 
   s.metadata['rubygems_mfa_required'] = 'false'
 end

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

@@ -11,14 +11,16 @@ describe 'PennMARC::Creator' do
       let(:fields) do
         [marc_field(tag: '100', subfields: { a: 'Surname, Name,', '0': 'http://cool.uri/12345',
                                              e: 'author', d: '1900-2000' }),
-         marc_field(tag: '880', subfields: { a: 'Surname, Alternative,', '6': '100' })]
+         marc_field(tag: '880', subfields: { a: 'Surname, Alternative,', '6': '100' }),
+         marc_field(tag: '700', subfields: { a: 'Surname, Third', e: 'author.', '6': '100' })]
       end
 
       it 'contains the expected search field values for a single author work' do
         expect(helper.search(record, relator_map: mapping)).to contain_exactly(
           'Name Surname http://cool.uri/12345 1900-2000, author.',
           'Surname, Name http://cool.uri/12345 1900-2000, author.',
-          'Alternative Surname'
+          'Alternative Surname',
+          'Third Surname'
         )
       end
     end

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

@@ -57,6 +57,18 @@ describe 'PennMARC::Subject' do
         expect(values.first).to eq 'Unicorns dpc Depicted'
       end
     end
+
+    context 'with subject heading subdivisions' do
+      let(:fields) do
+        [marc_field(tag: '610', indicator2: '0', subfields: { a: 'University of Pennsylvania',
+                                                              v: 'Ivy league', x: 'Higher education',
+                                                              y: '19th century', z: 'Philly' })]
+      end
+
+      it 'includes subdivisions' do
+        expect(values).to contain_exactly('University of Pennsylvania Ivy league Higher education 19th century Philly')
+      end
+    end
   end
 
   describe '.facet' do
@@ -152,7 +164,7 @@ describe 'PennMARC::Subject' do
 
       it 'drops the final trailing period' do
         expect(values).to contain_exactly('R.G. (Robert Gordon). Spiritual order and Christian liberty proved ' \
-                                          'to be consistent in the Churches of Christ')
+                                          'to be consistent in the Churches of Christ', 'R.G')
       end
     end
 
@@ -162,7 +174,29 @@ describe 'PennMARC::Subject' do
       end
 
       it 'treats the first part it comes across as a main subject part' do
-        expect(values).to contain_exactly('Italian--Architectural theory')
+        expect(values).to include('Italian--Architectural theory')
+      end
+
+      it 'returns subfield "a" for library of congress subject headings' do
+        expect(values).to contain_exactly('Architectural theory', 'Italian--Architectural theory')
+      end
+    end
+
+    context 'with a record with subject heading subdivisions' do
+      let(:fields) do
+        [marc_field(tag: '650', indicator2: '0', subfields: { a: 'Franklin, Benjamin,', d: '1706-1790',
+                                                              x: 'Books and reading' }),
+         marc_field(tag: '610', indicator2: '7',
+                    subfields: { '2': 'lcsh', a: 'Philadelphia (Pa.)', y: '18th century.' }),
+         marc_field(tag: '600', indicator2: '7',
+                    subfields: { '2': 'fast', a: 'Do not decompose', v: 'Penn Libraries' })]
+      end
+
+      it 'returns decomposed subfield "a" values for library of congress subject headings' do
+        expect(values).to contain_exactly('Franklin, Benjamin, 1706-1790--Books and reading',
+                                          'Philadelphia (Pa.)--18th century',
+                                          'Do not decompose--Penn Libraries',
+                                          'Franklin, Benjamin', 'Philadelphia (Pa.)')
       end
     end
   end
@@ -225,7 +259,8 @@ describe 'PennMARC::Subject' do
 
       it 'properly handles punctuation in subject parts' do
         expect(values).to contain_exactly 'Franklin, Benjamin, 1706-1790.',
-                                          'Franklin, Benjamin, 1706-1790--As inventor.', 'Franklin stoves.'
+                                          'Franklin, Benjamin, 1706-1790--As inventor.', 'Franklin stoves.',
+                                          'Franklin, Benjamin.'
       end
     end
 
@@ -326,6 +361,24 @@ describe 'PennMARC::Subject' do
         )
       end
     end
+
+    context 'with a record with subject heading subdivisions' do
+      let(:fields) do
+        [marc_field(tag: '650', indicator2: '0', subfields: { a: 'Franklin, Benjamin,', d: '1706-1790',
+                                                              x: 'Books and reading' }),
+         marc_field(tag: '610', indicator2: '7',
+                    subfields: { '2': 'lcsh', a: 'Philadelphia (Pa.)', y: '18th century.' }),
+         marc_field(tag: '600', indicator2: '7',
+                    subfields: { '2': 'fast', a: 'Do not decompose', v: 'Penn Libraries' })]
+      end
+
+      it 'returns decomposed subfield "a" values for library of congress subject headings' do
+        expect(values).to contain_exactly('Franklin, Benjamin, 1706-1790--Books and reading.',
+                                          'Philadelphia (Pa.)--18th century.',
+                                          'Do not decompose--Penn Libraries.',
+                                          'Franklin, Benjamin.', 'Philadelphia (Pa.).')
+      end
+    end
   end
 
   describe '.childrens_show' do

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

@@ -107,6 +107,7 @@ describe 'PennMARC::Title' do
     let(:leader) { 'ZZZZZnasZa22ZZZZZzZZ4500' }
     let(:fields) do
       [marc_field(tag: '130', subfields: { a: 'Uniform Title', c: '130 not included' }),
+       marc_field(tag: '222', subfields: { a: 'Key Title', b: '(Qualifying Info)' }),
        marc_field(tag: '880', subfields: { '6': '130', a: 'Alternative Uniform Title' }),
        marc_field(tag: '773', subfields: { a: 'Host Item - Main entry heading', s: 'Host Item - Uniform title',
                                            t: 'Host Item - Title' }),
@@ -118,6 +119,7 @@ describe 'PennMARC::Title' do
 
     it 'returns auxiliary journal search titles' do
       expect(helper.journal_search_aux(record)).to contain_exactly('Uniform Title', 'Alternative Uniform Title',
+                                                                   'Key Title (Qualifying Info)',
                                                                    'Host Item - Uniform title Host Item - Title',
                                                                    'Personal Entry Title',
                                                                    'Formatted Contents Note Title')

data/spec/spec_helper.rb

@@ -8,6 +8,7 @@ Dir[File.join(__dir__, 'support', '**', '*.rb')].each { |f| require f }
 # Require test helpers from gem lib
 Dir[File.join(__dir__, '../lib/pennmarc/test', '*.rb')].each { |f| require f }
 
+require 'debug'
 require 'pennmarc'
 
 # This file was generated by the `rspec --init` command. Conventionally, all

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pennmarc
 version: !ruby/object:Gem::Version
-  version: 1.4.1
+  version: 1.4.2
 platform: ruby
 authors:
 - Mike Kanning
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-11-13 00:00:00.000000000 Z
+date: 2026-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -20,14 +20,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7'
+        version: '8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7'
+        version: '8'
 - !ruby/object:Gem::Dependency
   name: lcsort
   requirement: !ruby/object:Gem::Requirement
@@ -62,28 +62,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.4'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: '1.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.15'
+        version: '1.19'
 description: |-
   This gem provides methods for parsing a Penn Libraries MARCXML record into string, array and date
                      objects for use in discovery or preservation applications.