pennmarc 1.0.12 → 1.0.14

data/lib/pennmarc/helpers/subject.rb

@@ -173,9 +173,9 @@ module PennMARC
 
         case type.to_sym
         when :facet
-          "#{term[:parts].join('--')} #{term[:lasts].join(' ')}".strip
+          term[:parts].join('--').strip
         when :display
-          "#{term[:parts].join('--')} #{term[:lasts].join(' ')} #{term[:append].join(' ')}".strip
+          "#{term[:parts].join('--')} #{term[:append].join(' ')}".strip
         end
       end
 
@@ -227,9 +227,9 @@ module PennMARC
       # @todo do i need all this?
       # @todo do i need to handle punctuation? see append_new_part
       # @param [MARC::DataField] field
-      # @return [Hash{Symbol->Integer | Array}]
+      # @return [Hash{Symbol => Integer, Array}, Nil]
       def build_subject_hash(field)
-        term_info = { count: 0, parts: [], append: [], lasts: [], uri: nil,
+        term_info = { count: 0, parts: [], append: [], uri: nil,
                       local: field.indicator2 == '4' || field.tag.starts_with?('69'), # local subject heading
                       vernacular: field.tag == '880' }
         field.each do |subfield|
@@ -251,8 +251,8 @@ module PennMARC
             # 'e' is relator term; not sure what 'w' is. These are used to append for record-view display only
             term_info[:append] << subfield.value.strip # TODO: map relator code?
           when 'b', 'c', 'd', 'p', 'q', 't'
-            # these are appended to the last component if possible (i.e., when joined, should have no delimiter)
-            term_info[:lasts] << subfield.value.strip
+            # these are appended to the last component (part) if possible (i.e., when joined, should have no delimiter)
+            term_info[:parts].last << ", #{subfield.value.strip}"
             term_info[:count] += 1
           else
             # the usual case; add a new component to `parts`

data/lib/pennmarc/mappings/locations.yml

@@ -588,6 +588,10 @@ finecore:
   specific_location: Fisher Fine Arts Library - Core Reading Collection
   library: Fisher Fine Arts Library
   display: Fine Arts Library - Core Reading Collection
+fineexhib:
+  specific_location: Fisher Fine Arts Library - Exhibitions Corridor
+  library: Fisher Fine Arts Library
+  display: Fine Arts Library - Exhibitions Corridor
 finelock:
   specific_location: Fisher Fine Arts Library - Locked Case
   library:

data/lib/pennmarc/util.rb

@@ -14,7 +14,7 @@ module PennMARC
     end
 
     # Join subfields from a field selected based on a provided proc
-    # @param [MARC::DataField] field
+    # @param [MARC::DataField, nil] field
     # @param [Proc] selector
     # @return [String]
     def join_subfields(field, &selector)
@@ -125,6 +125,21 @@ module PennMARC
       string.sub map[trailer.to_sym], ''
     end
 
+    # Intelligently append given punctuation to the end of a string
+    # @param [Symbol] trailer
+    # @param [String] string
+    # @return [String]
+    def append_trailing(trailer, string)
+      return string if string.end_with?('.', '-')
+
+      map = { semicolon: ';',
+              colon: ':',
+              slash: '/',
+              comma: ',',
+              period: '.' }
+      string + map[trailer.to_sym]
+    end
+
     # MARC 880 field "Alternate Graphic Representation" contains text "linked" to another
     # field (e.g., 254 [Title]) used as an alternate representation. Often used to hold
     # translations of title values. A common need is to extract subfields as selected by

data/lib/pennmarc/version.rb

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

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

@@ -10,7 +10,7 @@ describe 'PennMARC::Access' do
       let(:record) { marc_record fields: [marc_field(tag: tag)] }
 
       context 'with enrichment via the Alma publishing process' do
-        let(:tag) { PennMARC::EnrichedMarc::TAG_ELECTRONIC_INVENTORY }
+        let(:tag) { PennMARC::Enriched::Pub::ELEC_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::ONLINE)
@@ -18,7 +18,7 @@ describe 'PennMARC::Access' do
       end
 
       context 'with enrichment with availability info via the Alma API' do
-        let(:tag) { PennMARC::EnrichedMarc::AlmaApi::TAG_ELECTRONIC_INVENTORY }
+        let(:tag) { PennMARC::Enriched::Api::ELEC_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::ONLINE)
@@ -30,7 +30,7 @@ describe 'PennMARC::Access' do
       let(:record) { marc_record fields: [marc_field(tag: tag)] }
 
       context 'with enrichment via the Alma publishing process' do
-        let(:tag) { PennMARC::EnrichedMarc::TAG_HOLDING }
+        let(:tag) { PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::AT_THE_LIBRARY)
@@ -38,7 +38,7 @@ describe 'PennMARC::Access' do
       end
 
       context 'with enrichment with availability info via the Alma API' do
-        let(:tag) { PennMARC::EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY }
+        let(:tag) { PennMARC::Enriched::Api::PHYS_INVENTORY_TAG }
 
         it 'returns expected access value' do
           expect(helper.facet(record)).to contain_exactly(PennMARC::Access::AT_THE_LIBRARY)
@@ -48,7 +48,7 @@ describe 'PennMARC::Access' do
 
     context 'with a record containing a link to a finding aid (as a handle link)' do
       let(:record) do
-        marc_record fields: [marc_field(tag: PennMARC::EnrichedMarc::TAG_HOLDING),
+        marc_record fields: [marc_field(tag: PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG),
                              marc_field(tag: '856', subfields: location_and_access_subfields, **indicators)]
       end
 

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

@@ -17,9 +17,9 @@ describe 'PennMARC::Classification' do
 
   describe '.facet' do
     context 'with enrichment via the Alma publishing process' do
-      let(:tag) { PennMARC::EnrichedMarc::TAG_ITEM }
-      let(:call_number_type_sf) { PennMARC::EnrichedMarc::SUB_ITEM_CALL_NUMBER_TYPE }
-      let(:call_number_sf) { PennMARC::EnrichedMarc::SUB_ITEM_CALL_NUMBER }
+      let(:tag) { PennMARC::Enriched::Pub::ITEM_TAG }
+      let(:call_number_type_sf) { PennMARC::Enriched::Pub::ITEM_CALL_NUMBER_TYPE }
+      let(:call_number_sf) { PennMARC::Enriched::Pub::ITEM_CALL_NUMBER }
 
       it 'returns expected values' do
         expect(helper.facet(record)).to contain_exactly('T - Technology', '600 - Technology',
@@ -28,9 +28,9 @@ describe 'PennMARC::Classification' do
     end
 
     context 'with enrichment with availability info via Alma Api' do
-      let(:tag) { PennMARC::EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY }
-      let(:call_number_type_sf) { PennMARC::EnrichedMarc::AlmaApi::SUB_PHYSICAL_CALL_NUMBER_TYPE }
-      let(:call_number_sf) { PennMARC::EnrichedMarc::AlmaApi::SUB_PHYSICAL_CALL_NUMBER }
+      let(:tag) { PennMARC::Enriched::Api::PHYS_INVENTORY_TAG }
+      let(:call_number_type_sf) { PennMARC::Enriched::Api::PHYS_CALL_NUMBER_TYPE }
+      let(:call_number_sf) { PennMARC::Enriched::Api::PHYS_CALL_NUMBER }
 
       it 'returns expected values' do
         expect(helper.facet(record)).to contain_exactly('T - Technology', '600 - Technology',

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

@@ -17,11 +17,11 @@ describe 'PennMARC::Creator' do
       end
 
       it 'contains the expected search field values for a single author work' do
-        expect(helper.search(record, relator_map: mapping)).to eq [
+        expect(helper.search(record, relator_map: mapping)).to contain_exactly(
           'Name Surname http://cool.uri/12345 author 1900-2000.',
           'Surname, Name http://cool.uri/12345 author 1900-2000.',
           'Alternative Surname'
-        ]
+        )
       end
     end
 
@@ -32,8 +32,33 @@ describe 'PennMARC::Creator' do
       end
 
       it 'contains the expected search field values for a corporate author work' do
-        expect(helper.search(record, relator_map: mapping)).to eq ['Group of People Annual Meeting Author.',
-                                                                   'Alt. Group Name Alt. Annual Meeting']
+        expect(helper.search(record, relator_map: mapping)).to contain_exactly(
+          'Group of People Annual Meeting, Author.',
+          'Alt. Group Name Alt. Annual Meeting'
+        )
+      end
+    end
+  end
+
+  describe '.search_aux' do
+    let(:record) { marc_record fields: fields }
+
+    context 'with a record that has an added name in the 7xx field' do
+      let(:fields) do
+        [marc_field(tag: '100', subfields: { a: 'Author', c: 'Fancy', d: 'active 24th century AD', '4': 'aut' }),
+         marc_field(tag: '700', subfields: { a: 'Author, Added' }),
+         marc_field(tag: '880', subfields: { '6': '100', a: 'Alt Author', c: 'Alt Fanciness' }),
+         marc_field(tag: '880', subfields: { '6': '700', a: 'Alt Added Author' })]
+      end
+
+      it 'contains the expected search_aux field values for a single author work' do
+        expect(helper.search_aux(record, relator_map: mapping)).to contain_exactly(
+          'Author Fancy active 24th century AD, Author.',
+          'Author, Added.',
+          'Added Author.',
+          'Alt Author Alt Fanciness',
+          'Alt Added Author'
+        )
       end
     end
   end
@@ -187,6 +212,18 @@ describe 'PennMARC::Creator' do
     end
   end
 
+  describe '.conference_search' do
+    let(:record) do
+      marc_record fields: [
+        marc_field(tag: '111', subfields: { a: 'MARC History Symposium', c: 'Moscow', '4': 'aut' })
+      ]
+    end
+
+    it 'returns conference name information for searching without relator value' do
+      expect(helper.conference_search(record)).to eq ['MARC History Symposium Moscow']
+    end
+  end
+
   describe '.contributor_show' do
     let(:record) do
       marc_record fields: [
@@ -217,7 +254,4 @@ describe 'PennMARC::Creator' do
       )
     end
   end
-
-  # describe '.conference_search'
-  # describe '.search_aux'
 end

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

@@ -68,9 +68,9 @@ describe 'PennMARC::Format' do
       context 'with API enriched fields' do
         let(:record) do
           marc_record fields: [
-            marc_field(tag: PennMARC::EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY, subfields: {
+            marc_field(tag: PennMARC::Enriched::Api::PHYS_INVENTORY_TAG, subfields: {
                          :h => 'AB123',
-                         PennMARC::EnrichedMarc::AlmaApi::SUB_PHYSICAL_CALL_NUMBER_TYPE => '.456 Microfilm'
+                         PennMARC::Enriched::Api::PHYS_CALL_NUMBER_TYPE => '.456 Microfilm'
                        })
           ]
         end
@@ -83,9 +83,9 @@ describe 'PennMARC::Format' do
       context 'with publishing enriched fields' do
         let(:record) do
           marc_record fields: [
-            marc_field(tag: PennMARC::EnrichedMarc::TAG_HOLDING,
+            marc_field(tag: PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG,
                        subfields: { :h => 'AB123',
-                                    PennMARC::EnrichedMarc::SUB_ITEM_CALL_NUMBER_TYPE => '.456 Microfilm' })
+                                    PennMARC::Enriched::Pub::ITEM_CALL_NUMBER_TYPE => '.456 Microfilm' })
           ]
         end
 

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

@@ -0,0 +1,129 @@
+# frozen_string_literal: true
+
+describe 'PennMARC::Inventory' do
+  include MarcSpecHelpers
+
+  let(:helper) { PennMARC::Inventory }
+  let(:record) do
+    marc_record fields: fields
+  end
+
+  describe 'physical' do
+    let(:fields) do
+      [marc_field(tag: mapper::PHYS_INVENTORY_TAG,
+                  subfields: subfields)]
+    end
+
+    context 'with API enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Api }
+      let(:subfields) do
+        { mapper::PHYS_CALL_NUMBER => 'AB123.4',
+          mapper::PHYS_HOLDING_ID => '123456789',
+          mapper::PHYS_LOCATION_CODE => 'vanpelt',
+          mapper::PHYS_LOCATION_NAME => 'Van Pelt Library',
+          mapper::PHYS_PRIORITY => '1' }
+      end
+
+      it 'returns expected array of hash values' do
+        expect(helper.physical(record)).to contain_exactly(
+          { call_num: 'AB123.4', holding_id: '123456789', location_code: 'vanpelt',
+            location_name: 'Van Pelt Library', priority: '1' }
+        )
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Pub }
+      let(:subfields) do
+        { mapper::HOLDING_CLASSIFICATION_PART => 'AB123',
+          mapper::HOLDING_ITEM_PART => '.4',
+          mapper::PHYS_HOLDING_ID => '123456789',
+          mapper::PHYS_LOCATION_CODE => 'vanpelt',
+          mapper::PHYS_LOCATION_NAME => 'Van Pelt Library' }
+      end
+
+      it 'returns expected array of hash values' do
+        expect(helper.physical(record)).to contain_exactly(
+          { call_num: 'AB123.4', holding_id: '123456789', location_code: 'vanpelt',
+            location_name: 'Van Pelt Library', priority: nil }
+        )
+      end
+    end
+  end
+
+  describe 'electronic' do
+    let(:fields) do
+      [marc_field(tag: mapper::ELEC_INVENTORY_TAG,
+                  subfields: subfields)]
+    end
+    let(:subfields) do
+      { mapper::ELEC_PORTFOLIO_ID => '234567890',
+        mapper::ELEC_SERVICE_URL => 'https://www.iwish.com',
+        mapper::ELEC_COLLECTION_NAME => 'All Articles Repo',
+        mapper::ELEC_COVERAGE_STMT => 'All time',
+        mapper::ELEC_PUBLIC_NOTE => 'Portfolio public note' }
+    end
+
+    context 'with API enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Api }
+
+      it 'returns expected array of hash values' do
+        expect(helper.electronic(record)).to contain_exactly(
+          { portfolio_id: '234567890', url: 'https://www.iwish.com', collection_name: 'All Articles Repo',
+            coverage: 'All time', note: 'Portfolio public note' }
+        )
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:mapper) { PennMARC::Enriched::Pub }
+
+      it 'returns expected array of hash values' do
+        expect(helper.electronic(record)).to contain_exactly(
+          { portfolio_id: '234567890', url: 'https://www.iwish.com', collection_name: 'All Articles Repo',
+            coverage: 'All time', note: 'Portfolio public note' }
+        )
+      end
+    end
+  end
+
+  describe 'electronic_portfolio_count' do
+    let(:fields) { [marc_field(tag: inventory_tag), marc_field(tag: inventory_tag)] }
+
+    context 'with API enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Api::ELEC_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.electronic_portfolio_count(record)).to eq 2
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Pub::ELEC_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.electronic_portfolio_count(record)).to eq 2
+      end
+    end
+  end
+
+  describe 'physical_holding_count' do
+    let(:fields) { [marc_field(tag: inventory_tag), marc_field(tag: inventory_tag)] }
+
+    context 'with API enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Api::PHYS_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.physical_holding_count(record)).to eq 2
+      end
+    end
+
+    context 'with Pub enrichment fields' do
+      let(:inventory_tag) { PennMARC::Enriched::Pub::PHYS_INVENTORY_TAG }
+
+      it 'returns the correct count' do
+        expect(helper.physical_holding_count(record)).to eq 2
+      end
+    end
+  end
+end

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

@@ -4,11 +4,15 @@ describe 'PennMARC::Location' do
   include MarcSpecHelpers
 
   let(:helper) { PennMARC::Location }
+  let(:enriched_marc) { PennMARC::Enriched }
   let(:mapping) { location_map }
 
   describe 'location' do
     context "with only 'itm' field present" do
-      let(:record) { marc_record(fields: [marc_field(tag: 'itm', subfields: { g: 'stor' })]) }
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' })])
+      end
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
@@ -19,7 +23,10 @@ describe 'PennMARC::Location' do
     end
 
     context "with only 'hld' field present" do
-      let(:record) { marc_record(fields: [marc_field(tag: 'hld', subfields: { c: 'stor' })]) }
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                                        subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'stor' })])
+      end
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
@@ -29,10 +36,12 @@ describe 'PennMARC::Location' do
       end
     end
 
-    context "with both 'hld' and 'itm' fields present" do
+    context 'with both holding and item tag fields present=' do
       let(:record) do
-        marc_record(fields: [marc_field(tag: 'itm', subfields: { g: 'stor' }),
-                             marc_field(tag: 'hld', subfields: { c: 'dent' })])
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' }),
+                             marc_field(tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                                        subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'dent' })])
       end
 
       it 'returns item location' do
@@ -42,7 +51,7 @@ describe 'PennMARC::Location' do
     end
 
     context 'with multiple library locations' do
-      let(:record) { marc_record(fields: [marc_field(tag: 'itm', subfields: { g: %w[dent] })]) }
+      let(:record) { marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG, subfields: { g: %w[dent] })]) }
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
@@ -60,17 +69,26 @@ describe 'PennMARC::Location' do
     end
 
     context 'with electronic inventory tag' do
-      let(:record) { marc_record(fields: [marc_field(tag: 'itm', subfields: { g: 'stor' }), marc_field(tag: 'prt')]) }
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' }),
+                             marc_field(tag: enriched_marc::Pub::ELEC_INVENTORY_TAG)])
+      end
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping,
-                               display_value: :library)).to contain_exactly('LIBRA', 'Online library')
+                               display_value: :library)).to contain_exactly('LIBRA', helper::ONLINE_LIBRARY)
       end
     end
 
     context 'with AVA fields' do
       let(:record) do
-        marc_record(fields: [marc_field(tag: 'AVA', subfields: { b: 'Libra', c: 'LIBRA', j: 'stor' })])
+        marc_record(fields: [marc_field(tag: enriched_marc::Api::PHYS_INVENTORY_TAG,
+                                        subfields: {
+                                          enriched_marc::Api::PHYS_LIBRARY_CODE => 'Libra',
+                                          enriched_marc::Api::PHYS_LOCATION_NAME => 'LIBRA',
+                                          enriched_marc::Api::PHYS_LOCATION_CODE => 'stor'
+                                        })])
       end
 
       it 'returns expected values' do
@@ -79,5 +97,18 @@ describe 'PennMARC::Location' do
         )
       end
     end
+
+    context 'with AVE fields' do
+      let(:record) do
+        marc_record(fields: [marc_field(tag: enriched_marc::Api::ELEC_INVENTORY_TAG,
+                                        subfields: { enriched_marc::Api::ELEC_COLLECTION_NAME => 'Nature' })])
+      end
+
+      it 'returns expected values' do
+        expect(helper.location(record: record, location_map: mapping, display_value: :library)).to(
+          contain_exactly(helper::ONLINE_LIBRARY)
+        )
+      end
+    end
   end
 end

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

@@ -101,13 +101,14 @@ describe 'PennMARC::Subject' do
       let(:fields) do
         [marc_field(tag: '650', indicator2: '7',
                     subfields: {
-                      a: 'Libraries', x: 'History', e: 'relator', d: '22nd Century',
+                      a: 'Libraries', d: '22nd Century', x: 'History', e: 'relator',
                       '2': 'fast', '0': 'http://fast.org/libraries'
                     })]
       end
 
       it 'properly concatenates heading components' do
-        expect(values.first).to include 'Libraries--History'
+        expect(values.first).to start_with 'Libraries'
+        expect(values.first).to end_with '--History'
       end
 
       it 'excludes URI values from ǂ0 or ǂ1' do
@@ -123,14 +124,14 @@ describe 'PennMARC::Subject' do
       end
 
       it 'joins all values in the expected way' do
-        expect(values.first).to eq 'Libraries--History 22nd Century'
+        expect(values.first).to eq 'Libraries, 22nd Century--History'
       end
     end
   end
 
   describe '.show' do
     let(:record) { marc_record fields: fields }
-    let(:values) { helper.facet(record) }
+    let(:values) { helper.show(record) }
 
     context 'with a variety of headings' do
       let(:fields) do
@@ -162,8 +163,28 @@ describe 'PennMARC::Subject' do
       end
 
       it 'properly formats the heading parts' do
-        expect(values.first).to eq 'Subways--Pennsylvania--Philadelphia Metropolitan Area--Maps--1989'
-        expect(values.first).not_to include 'relator'
+        expect(values.first).to eq 'Subways--Pennsylvania--Philadelphia Metropolitan Area--Maps--1989 relator'
+      end
+    end
+
+    context 'with the record including trailing punctuation in the parts' do
+      let(:fields) do
+        [marc_field(tag: '600', indicator2: '7', subfields: {
+                      a: 'Franklin, Benjamin,',
+                      d: '1706-1790',
+                      '2': 'fast',
+                      '0': 'http://id.worldcat.org/fast/34115'
+                    }),
+         marc_field(tag: '600', indicator1: '1', indicator2: '0', subfields: {
+                      a: 'Franklin, Benjamin,',
+                      d: '1706-1790.',
+                      x: 'As inventor.'
+                    })]
+      end
+
+      it 'returns what Franklin shows', pending: 'proper handling of punctuation in subject parts' do
+        expect(values).to contain_exactly 'Franklin, Benjamin, 1706-1790.',
+                                          'Franklin, Benjamin, 1706-1790--As inventor.'
       end
     end
 
@@ -187,14 +208,14 @@ describe 'PennMARC::Subject' do
       let(:fields) do
         [marc_field(tag: '611', indicator2: '0', subfields: {
                       a: 'Conference',
-                      d: '(2002',
-                      n: '2nd',
-                      c: ['Johannesburg, South Africa', 'Cape Town, South Africa)']
+                      c: ['(Johannesburg, South Africa', 'Cape Town, South Africa'],
+                      d: '2002)',
+                      n: '2nd'
                     })]
       end
 
       it 'properly formats the heading parts' do
-        expect(values.first).to eq 'Conference--2nd (2002 Johannesburg, South Africa Cape Town, South Africa)'
+        expect(values.first).to eq 'Conference, (Johannesburg, South Africa, Cape Town, South Africa, 2002)--2nd'
       end
     end
 
@@ -202,14 +223,17 @@ describe 'PennMARC::Subject' do
       let(:fields) do
         [marc_field(tag: '600', indicator2: '0', subfields: {
                       a: 'Person, Significant Author',
+                      b: 'Numerator',
+                      c: %w[Title Rank],
                       d: '1899-1971',
-                      v: 'Early works to 1950',
-                      t: 'Collection'
+                      t: 'Collection',
+                      v: 'Early works to 1950'
                     })]
       end
 
       it 'properly formats the heading parts' do
-        expect(values.first).to eq 'Person, Significant Author--Early works to 1950 1899-1971 Collection'
+        expect(values.first).to eq('Person, Significant Author, Numerator, Title, Rank, 1899-1971, Collection--' \
+                                   'Early works to 1950')
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pennmarc
 version: !ruby/object:Gem::Version
-  version: 1.0.12
+  version: 1.0.14
 platform: ruby
 authors:
 - Mike Kanning
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-17 00:00:00.000000000 Z
+date: 2024-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -88,7 +88,7 @@ files:
 - README.md
 - lib/pennmarc.rb
 - lib/pennmarc/encoding_level.rb
-- lib/pennmarc/enriched_marc.rb
+- lib/pennmarc/enriched.rb
 - lib/pennmarc/heading_control.rb
 - lib/pennmarc/helpers/access.rb
 - lib/pennmarc/helpers/citation.rb
@@ -101,6 +101,10 @@ files:
 - lib/pennmarc/helpers/genre.rb
 - lib/pennmarc/helpers/helper.rb
 - lib/pennmarc/helpers/identifier.rb
+- lib/pennmarc/helpers/inventory.rb
+- lib/pennmarc/helpers/inventory_entry/base.rb
+- lib/pennmarc/helpers/inventory_entry/electronic.rb
+- lib/pennmarc/helpers/inventory_entry/physical.rb
 - lib/pennmarc/helpers/language.rb
 - lib/pennmarc/helpers/link.rb
 - lib/pennmarc/helpers/location.rb
@@ -132,6 +136,7 @@ files:
 - spec/lib/pennmarc/helpers/format_spec.rb
 - spec/lib/pennmarc/helpers/genre_spec.rb
 - spec/lib/pennmarc/helpers/identifer_spec.rb
+- spec/lib/pennmarc/helpers/inventory_spec.rb
 - spec/lib/pennmarc/helpers/language_spec.rb
 - spec/lib/pennmarc/helpers/link_spec.rb
 - spec/lib/pennmarc/helpers/location_spec.rb