pennmarc 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb0bf4c30ad40bcee865ed6596fbfc6d6d16a5d3924d62947d270a3d5fa3aac5
-  data.tar.gz: 33ca607af919156ac45a73067605b6ac298fef9c17ef983060e6a871c54ca26f
+  metadata.gz: 2b87c66becc36828ec6331d7261d0b71472a74f4af26f2b7a145ed54316ed11a
+  data.tar.gz: 0de62b7492b92ebe0902081aee1da5e5bcabac9e8d9042bab88db7887adda62e
 SHA512:
-  metadata.gz: 5e5d316a7e1340aaf7438297e17198339d02ab797842b47ec2cece1daf75f96a387565b4fdc6ca273ef1a64630e1f486bc713ea75793ed3d783998234a83c517
-  data.tar.gz: 6f6d6824e1aa5fecbcd2dc13bbc94ea575531e96ea5f822b992719f3b28189cce4c26febc2f31848f24008a57f6a839f787383752cfd2c915695b1f1e66fe5ca
+  metadata.gz: 0ffced7042a0c00a09c1a2b465e0d17506e55bf0a77255e8f1bce1acabdfabe8faa819864cb0376d7ac6a10285c1a810bd1ede17210829326d342998ed908aa4
+  data.tar.gz: 7aac0315dda59416fdd5b5ed6409b1ddc9a3f881bc75a5aa08c07d299d9824ed9c12a6f6cbe7cacc39a45980f36bcfb97e9ed6b42c379a3aa6364b56f397a244

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config --auto-gen-only-exclude --exclude-limit 10000`
-# on 2024-07-02 15:41:46 UTC using RuboCop version 1.51.0.
+# on 2024-08-30 16:12:14 UTC using RuboCop version 1.51.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -14,7 +14,7 @@ Gemspec/RequireMFA:
   Exclude:
     - 'pennmarc.gemspec'
 
-# Offense count: 25
+# Offense count: 26
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes, Max.
 Metrics/AbcSize:
   Exclude:
@@ -31,7 +31,7 @@ Metrics/AbcSize:
     - 'lib/pennmarc/helpers/title.rb'
     - 'lib/pennmarc/util.rb'
 
-# Offense count: 8
+# Offense count: 9
 # Configuration parameters: CountComments, Max, CountAsOne.
 Metrics/ClassLength:
   Exclude:
@@ -138,9 +138,10 @@ RSpec/FilePath:
   Exclude:
     - 'spec/lib/pennmarc/parser_spec.rb'
 
-# Offense count: 23
+# Offense count: 27
 # Configuration parameters: Max, AllowedGroups.
 RSpec/NestedGroups:
   Exclude:
     - 'spec/lib/pennmarc/helpers/access_spec.rb'
     - 'spec/lib/pennmarc/helpers/format_spec.rb'
+    - 'spec/lib/pennmarc/helpers/location_spec.rb'

data/lib/pennmarc/helpers/location.rb

@@ -37,12 +37,12 @@ module PennMARC
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param record [MARC::Record]
-      # @param display_value [Symbol | String] field in location hash to retrieve
+      # @param display_value [Symbol,String] field in location hash to retrieve
       # @param location_map [Hash] hash with location_code as key and location hash as value
       # @return [Array<String>]
       def location(record:, display_value:, location_map:)
         # get enriched marc location tag and relevant subfields
-        enriched_location_tag_and_subfields(record) => {tag:, location_code_sf:, call_num_sf:}
+        enriched_location_tag_and_subfields(record) => {tag:, location_code_sf:, call_num_sf:, call_num_type_sf:}
 
         record.fields(tag).flat_map { |field|
           field.filter_map { |subfield|
@@ -53,8 +53,10 @@ module PennMARC
 
             next if location_code_to_ignore?(location_map, location_code)
 
-            override = specific_location_override(display_value: display_value, location_code: location_code,
-                                                  field: field, call_num_sf: call_num_sf)
+            override = if display_value.to_sym == :specific_location
+                         specific_location_override(location_code: location_code, field: field,
+                                                    call_num_sf: call_num_sf, call_num_type_sf: call_num_type_sf)
+                       end
 
             override || location_map[location_code.to_sym][display_value.to_sym]
           }.flatten.compact_blank
@@ -70,6 +72,8 @@ module PennMARC
       # - `:tag` (String): The enriched marc location tag
       # - `:location_code_sf` (String): The subfield code where location code is stored
       # - `:call_num_sf` (String): The subfield code where call number is stored
+      # - `:call_num_type_sf` (String, nil): The subfield code where call number type is stored. nil if unavailable in a
+      #                                   MARC field and we need to look for an indicator.
       def enriched_location_tag_and_subfields(record)
         # in holdings records, the shelving location is always the permanent location.
         # in item records, the current location takes into account
@@ -84,19 +88,22 @@ module PennMARC
           tag = Enriched::Pub::ITEM_TAG
           location_code_sf = Enriched::Pub::ITEM_CURRENT_LOCATION
           call_num_sf = Enriched::Pub::ITEM_CALL_NUMBER
+          call_num_type_sf = Enriched::Pub::ITEM_CALL_NUMBER_TYPE
           # if the record has API inventory tags, use them
         elsif field_defined?(record, Enriched::Api::PHYS_INVENTORY_TAG)
           tag = Enriched::Api::PHYS_INVENTORY_TAG
           location_code_sf = Enriched::Api::PHYS_LOCATION_CODE
           call_num_sf = Enriched::Api::PHYS_CALL_NUMBER
+          call_num_type_sf = Enriched::Api::PHYS_CALL_NUMBER_TYPE
           # otherwise use Pub holding tags
         else
           tag = Enriched::Pub::PHYS_INVENTORY_TAG
           location_code_sf = Enriched::Pub::PHYS_LOCATION_CODE
           call_num_sf = Enriched::Pub::HOLDING_CLASSIFICATION_PART
+          call_num_type_sf = nil # for hld tags, the call num type is indicator0
         end
 
-        { tag: tag, location_code_sf: location_code_sf, call_num_sf: call_num_sf }
+        { tag: tag, location_code_sf: location_code_sf, call_num_sf: call_num_sf, call_num_type_sf: call_num_type_sf }
       end
 
       # Determines whether to ignore a location code.
@@ -105,32 +112,55 @@ module PennMARC
       # map, we ignore it, for faceting purposes. We also ignore the location code 'web'. We don't facet for 'web'
       # which is the 'Penn Library Web' location used in Voyager. This location should eventually go away completely
       # with data cleanup in Alma.
-      # @param location_map [location_map] hash with location_code as key and location hash as value
-      # @param location_code [location_code] retrieved from record
+      # @param location_map [Hash] hash with location_code as key and location hash as value
+      # @param location_code [String] retrieved from record
       # @return [Boolean]
       def location_code_to_ignore?(location_map, location_code)
-        location_map.key?(location_code.to_sym) == false || location_code == WEB_LOCATION_CODE
+        !location_map.key?(location_code.to_sym) || location_code == WEB_LOCATION_CODE
       end
 
       # Retrieves a specific location override based on location code and call number. Specific location overrides are
       # located in `location_overrides.yml`.
-      # @param display_value [String | Symbol]
       # @param location_code [String]
       # @param field [MARC::Field]
       # @param call_num_sf [String]
+      # @param call_num_type_sf [String, nil]
       # @return [String, Nil]
-      def specific_location_override(display_value:, location_code:, field:, call_num_sf:)
-        return unless display_value.to_sym == :specific_location
+      def specific_location_override(location_code:, field:, call_num_sf:, call_num_type_sf:)
+        callnum_type = callnum_type(field: field, call_num_type_sf: call_num_type_sf)
+        return unless callnum_type
 
-        locations_overrides = Mappers.location_overrides
+        override = Mappers.location_overrides.find do |_key, override_data|
+          override_matching?(override_data: override_data, location_code: location_code, callnum_type: callnum_type,
+                             call_numbers: subfield_values(field, call_num_sf))
+        end
 
-        call_numbers = subfield_values(field, call_num_sf)
+        override&.last&.dig(:specific_location)
+      end
 
-        override = locations_overrides.find do |_key, value|
-          value[:location_code] == location_code && call_numbers.any? { |num| num.match?(value[:call_num_pattern]) }
+      # Check override_data hash for a matching location name override
+      # @param override_data [Hash]
+      # @param location_code [String]
+      # @param call_numbers [Array]
+      # @param callnum_type [String]
+      # @return [Boolean]
+      def override_matching?(override_data:, location_code:, call_numbers:, callnum_type:)
+        call_numbers.any? do |call_number|
+          override_data[:location_code] == location_code &&
+            override_data[:call_num_type] == callnum_type &&
+            call_number.match?(override_data[:call_num_pattern])
         end
+      end
 
-        override&.last&.dig(:specific_location)
+      # Return call num type value for a given field. If no call number subfield is expected (publishing holding
+      # inventory case), the first indicator is checked.
+      # @param field [MARC::Field]
+      # @param call_num_type_sf [String, nil]
+      # @return [String, nil]
+      def callnum_type(field:, call_num_type_sf:)
+        return field.indicator1 if call_num_type_sf.nil?
+
+        subfield_values(field, call_num_type_sf).first
       end
     end
   end

data/lib/pennmarc/helpers/series.rb

@@ -35,6 +35,23 @@ module PennMARC
         series_values.uniq
       end
 
+      # Hash that associates {https://www.loc.gov/marc/bibliographic/bd830.html 830} and
+      # {https://www.loc.gov/marc/bibliographic/bd440.html 440} show values to query values to make building fielded
+      # search links easier in the downstream app.
+      # @param record [MARC::Record]
+      # @return [Hash]
+      def show_query_map(record)
+        record.fields(%w[830 440]).each_with_object({}) do |field, hash|
+          show_values = title_show_entries(record, field.tag) + [remaining_show_value(field)]
+
+          query_value = trim_punctuation(join_subfields(field, &subfield_in?(%w[a n p])))
+
+          show_values.each do |show_value|
+            hash[show_value] = query_value
+          end
+        end
+      end
+
       # Values from series fields for display.
       # @param record [MARC::Record]
       # @param relator_map [Hash]
@@ -141,10 +158,10 @@ module PennMARC
       # @note added 2017/04/10: filter out 0 (authority record numbers) added by Alma
       # @param record [MARC::Record]
       # @param tags_present [Array<String>]
-      # @return [Array<Hash>] array of remaining show entry hashes
+      # @return [Array<String>] array of remaining show entry hashes
       def remaining_show_entries(record, tags_present)
         record.fields(tags_present.drop(1)).map do |field|
-          join_subfields(field, &subfield_not_in?(%w[0 5 6 8]))
+          remaining_show_value(field)
         end || []
       end
 
@@ -200,6 +217,13 @@ module PennMARC
           join_subfields(field, &subfield_in?(%w[i a s t n d]))
         end
       end
+
+      # Parse the value for a given remaining show field
+      # @param field [MARC::Field]
+      # @return [String]
+      def remaining_show_value(field)
+        join_subfields(field, &subfield_not_in?(%w[0 5 6 8]))
+      end
     end
   end
 end

data/lib/pennmarc/helpers/title.rb

@@ -88,7 +88,6 @@ module PennMARC
       # Single-valued Title, for use in headings. Takes the first {https://www.oclc.org/bibformats/en/2xx/245.html 245}
       # value. Special consideration for
       # {https://www.oclc.org/bibformats/en/2xx/245.html#punctuation punctuation practices}.
-      # @todo still consider ǂh? medium, which OCLC doc says DO NOT USE...but that is OCLC...
       # @todo is punctuation handling still as desired? treatment here is described in spreadsheet from 2011
       # @param record [MARC::Record]
       # @return [String] single title for display
@@ -98,15 +97,14 @@ module PennMARC
         [format_title(values[:title_or_form]), values[:punctuation], values[:other_info]].compact_blank.join(' ')
       end
 
-      # Same as show, but with inclusive dates appended. For use on show page.
+      # Same as show, but with all subfields included as found - except for subfield c.
       # @param record [MARC::Record]
       # @return [String] detailed title for display
       def detailed_show(record)
         field = record.fields('245')&.first
-        values = title_values(field)
-        title = [format_title(values[:title_or_form]), values[:punctuation],
-                 trim_trailing(:period, values[:other_info])].compact_blank.join(' ')
-        values[:inclusive_dates].present? ? [title, values[:inclusive_dates]].compact_blank.join(', ') : title
+        return unless field
+
+        join_subfields(field, &subfield_not_in?(%w[6 8]))
       end
 
       # Same structure as show, but linked alternate title.
@@ -124,17 +122,6 @@ module PennMARC
         [format_title(values[:title_or_form]), values[:punctuation], values[:other_info]].compact_blank.join(' ')
       end
 
-      # Title statement of responsibility (field 245, subfield c) and linked alternate for display.
-      # See https://www.oclc.org/bibformats/en/2xx/245.html#subfieldc for examples
-      # @param [MARC::Record] record
-      # @return [Array<String>] statement of responsibility and linked alternate
-      def statement_of_responsibility_show(record)
-        field = record.fields('245').first
-        statement = field&.find { |sf| sf.code == 'c' }&.value
-        alternate_statement = linked_alternate(record, '245', &subfield_in?(%w[c]))&.first
-        [statement, alternate_statement].compact_blank
-      end
-
       # Canonical title with non-filing characters relocated to the end.
       #
       # @note Currently we index two "title sort" fields: title_nssort (ssort type - regex token filter applied) and
@@ -269,7 +256,6 @@ module PennMARC
         title_or_form = field.find_all(&subfield_in?(%w[a k]))
                              .map { |sf| trim_trailing(:comma, trim_trailing(:slash, sf.value).rstrip) }
                              .first || ''
-        inclusive_dates = field.find { |sf| sf.code == 'f' }&.value
         other_info = field.find_all(&subfield_in?(%w[b n p]))
                           .map { |sf| trim_trailing(:slash, sf.value) }
                           .join(' ')
@@ -283,7 +269,6 @@ module PennMARC
                         [title_punctuation, medium_punctuation].include?(':') ? ':' : nil
                       end
         { title_or_form: title_or_form,
-          inclusive_dates: inclusive_dates,
           other_info: other_info,
           punctuation: punctuation }
       end

data/lib/pennmarc/mappings/location_overrides.yml

@@ -1,5 +1,6 @@
 albrecht:
   location_code: 'vanp'
   call_num_pattern: '^M.*'
+  call_num_type: '0'
   specific_location: 'Van Pelt - Albrecht Music Library'
 

data/lib/pennmarc/version.rb

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

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

@@ -4,12 +4,13 @@ describe 'PennMARC::Location' do
   let(:helper) { PennMARC::Location }
   let(:enriched_marc) { PennMARC::Enriched }
   let(:mapping) { location_map }
+  let(:record) { marc_record(fields: fields) }
 
   describe 'location' do
     context "with only 'itm' field present" do
-      let(:record) do
-        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
-                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' })])
+      let(:fields) do
+        [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                    subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor' })]
       end
 
       it 'returns expected value' do
@@ -21,9 +22,9 @@ describe 'PennMARC::Location' do
     end
 
     context "with only 'hld' field present" do
-      let(:record) do
-        marc_record(fields: [marc_field(tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
-                                        subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'stor' })])
+      let(:fields) do
+        [marc_field(tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                    subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'stor' })]
       end
 
       it 'returns expected value' do
@@ -35,11 +36,11 @@ describe 'PennMARC::Location' do
     end
 
     context 'with both holding and item tag fields present=' do
-      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::PHYS_INVENTORY_TAG,
-                                        subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'dent' })])
+      let(:fields) do
+        [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
@@ -49,7 +50,7 @@ describe 'PennMARC::Location' do
     end
 
     context 'with multiple library locations' do
-      let(:record) { marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG, subfields: { g: %w[dent] })]) }
+      let(: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,
@@ -59,7 +60,7 @@ describe 'PennMARC::Location' do
     end
 
     context 'without enriched marc location tag' do
-      let(:record) { marc_record(fields: [marc_field(tag: '852', subfields: { g: 'stor' })]) }
+      let(:fields) { [marc_field(tag: '852', subfields: { g: 'stor' })] }
 
       it 'returns expected value' do
         expect(helper.location(record: record, location_map: mapping, display_value: :library)).to be_empty
@@ -67,13 +68,11 @@ describe 'PennMARC::Location' do
     end
 
     context 'with AVA fields' do
-      let(:record) do
-        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'
-                                        })])
+      let(:fields) do
+        [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
@@ -84,23 +83,82 @@ describe 'PennMARC::Location' do
     end
 
     context 'with a specific location override' do
-      let(:record) do
-        marc_record(fields: [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
-                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'vanp',
-                                                     enriched_marc::Pub::ITEM_CALL_NUMBER => 'ML3534 .D85 1984' }),
-                             marc_field(tag: enriched_marc::Pub::ITEM_TAG,
-                                        subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor',
-                                                     enriched_marc::Pub::ITEM_CALL_NUMBER => 'L3534 .D85 1984' })])
+      context 'with item fields and LC call nums' do
+        let(:fields) do
+          [marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                      subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'vanp',
+                                   enriched_marc::Pub::ITEM_CALL_NUMBER_TYPE =>
+                                     PennMARC::Classification::LOC_CALL_NUMBER_TYPE,
+                                   enriched_marc::Pub::ITEM_CALL_NUMBER => 'ML3534 .D85 1984' }),
+           marc_field(tag: enriched_marc::Pub::ITEM_TAG,
+                      subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'stor',
+                                   enriched_marc::Pub::ITEM_CALL_NUMBER_TYPE => '8',
+                                   enriched_marc::Pub::ITEM_CALL_NUMBER => 'L3534 .D85 1984' })]
+        end
+
+        it 'returns expected values' do
+          expect(helper.location(record: record, display_value: :specific_location, location_map: mapping))
+            .to(contain_exactly(PennMARC::Mappers.location_overrides[:albrecht][:specific_location], 'LIBRA'))
+        end
+
+        it 'returns expected values when receiving a string for display_value' do
+          expect(helper.location(record: record, display_value: 'specific_location', location_map: mapping))
+            .to(contain_exactly(PennMARC::Mappers.location_overrides[:albrecht][:specific_location], 'LIBRA'))
+        end
       end
 
-      it 'returns expected values' do
-        expect(helper.location(record: record, display_value: :specific_location, location_map: mapping))
-          .to(contain_exactly(PennMARC::Mappers.location_overrides[:albrecht][:specific_location], 'LIBRA'))
+      context 'with item fields and microfilm call nums' do
+        let(:fields) do
+          [marc_field(tag: enriched_marc::Pub::ITEM_TAG, indicator1: ' ',
+                      subfields: { enriched_marc::Pub::ITEM_CURRENT_LOCATION => 'vanp',
+                                   enriched_marc::Pub::ITEM_CALL_NUMBER_TYPE => '8',
+                                   enriched_marc::Pub::ITEM_CALL_NUMBER => 'Microfilm 3140 item 8' })]
+        end
+
+        it 'returns expected values' do
+          expect(helper.location(record: record, display_value: :specific_location, location_map: mapping))
+            .to(contain_exactly(PennMARC::Mappers.location[:vanp][:specific_location]))
+        end
+      end
+
+      context 'with holding fields and both LC and non-LC call num type' do
+        let(:fields) do
+          [marc_field(indicator1: '8', tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                      subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'vanp' }),
+           marc_field(indicator1: '0', tag: enriched_marc::Pub::PHYS_INVENTORY_TAG,
+                      subfields: { enriched_marc::Pub::PHYS_LOCATION_CODE => 'vanp',
+                                   enriched_marc::Pub::HOLDING_CLASSIFICATION_PART => 'ML3534' })]
+        end
+
+        it 'returns expected values' do
+          expect(helper.location(record: record, display_value: :specific_location, location_map: mapping))
+            .to(contain_exactly(PennMARC::Mappers.location[:vanp][:specific_location],
+                                PennMARC::Mappers.location_overrides[:albrecht][:specific_location]))
+        end
       end
 
-      it 'returns expected values when receiving a string for display_value' do
-        expect(helper.location(record: record, display_value: 'specific_location', location_map: mapping))
-          .to(contain_exactly(PennMARC::Mappers.location_overrides[:albrecht][:specific_location], 'LIBRA'))
+      context 'with a variety of holding fields from the Alma API enrichment' do
+        let(:fields) do
+          [marc_field(tag: enriched_marc::Api::PHYS_INVENTORY_TAG,
+                      subfields: { enriched_marc::Api::PHYS_CALL_NUMBER => 'Locked Closet Floor',
+                                   enriched_marc::Api::PHYS_CALL_NUMBER_TYPE => '8' }),
+           marc_field(tag: enriched_marc::Api::PHYS_INVENTORY_TAG,
+                      subfields: { enriched_marc::Api::PHYS_LOCATION_CODE => 'vanp',
+                                   enriched_marc::Api::PHYS_CALL_NUMBER => ['ML123 .P567 1875', 'ML123'],
+                                   enriched_marc::Api::PHYS_CALL_NUMBER_TYPE =>
+                                     PennMARC::Classification::LOC_CALL_NUMBER_TYPE }),
+           marc_field(tag: enriched_marc::Api::PHYS_INVENTORY_TAG,
+                      subfields: { enriched_marc::Api::PHYS_LOCATION_CODE => 'vanp',
+                                   enriched_marc::Api::PHYS_CALL_NUMBER => 'P789 .D123 2012',
+                                   enriched_marc::Api::PHYS_CALL_NUMBER_TYPE =>
+                                     PennMARC::Classification::LOC_CALL_NUMBER_TYPE })]
+        end
+
+        it 'returns expected values' do
+          expect(helper.location(record: record, display_value: :specific_location, location_map: mapping))
+            .to(contain_exactly(PennMARC::Mappers.location[:vanp][:specific_location],
+                                PennMARC::Mappers.location_overrides[:albrecht][:specific_location]))
+        end
       end
     end
   end

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

@@ -15,6 +15,24 @@ describe 'PennMARC::Series' do
     marc_record fields: [marc_field(tag: '666', subfields: { a: 'test' })]
   end
 
+  describe '.show_query_map' do
+    let(:record) do
+      marc_record fields: [
+        marc_field(tag: '440', subfields: { a: 'Teachings of the feathered pillow', n: 'number', p: 'section' }),
+        marc_field(tag: '490', subfields: { a: 'Ignored' }),
+        marc_field(tag: '830', subfields: { a: 'International litigation in practice', v: 'volume 13.' })
+      ]
+    end
+
+    it 'returns expected hash' do
+      expect(helper.show_query_map(record)).to eq(
+        { 'Teachings of the feathered pillow number section' => 'Teachings of the feathered pillow number section',
+          'Teachings of the feathered pillow section number' => 'Teachings of the feathered pillow number section',
+          'International litigation in practice volume 13.' => 'International litigation in practice' }
+      )
+    end
+  end
+
   describe '.show' do
     it 'returns the series values for display' do
       expect(helper.show(record, relator_map: mapping)).to contain_exactly(

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

@@ -139,15 +139,44 @@ describe 'PennMARC::Title' do
   end
 
   describe '.detailed_show' do
-    let(:record) do
-      marc_record fields: [
-        marc_field(tag: '245', subfields: { k: 'Letters', f: '1972-1982', b: 'to Lewis Mumford.' })
-      ]
-    end
-
     context 'with subfields ǂk, ǂf and ǂc' do
+      let(:record) do
+        marc_record fields: [
+          marc_field(tag: '245', subfields: { k: 'Letters,', f: '1972-1982,', b: 'to Lewis Mumford.' })
+        ]
+      end
+
       it 'returns detailed title values' do
-        expect(helper.detailed_show(record)).to eq 'Letters to Lewis Mumford, 1972-1982'
+        expect(helper.detailed_show(record)).to eq 'Letters, 1972-1982, to Lewis Mumford.'
+      end
+    end
+
+    context 'with subfields ǂk and ǂb' do
+      let(:record) do
+        marc_record fields: [
+          marc_field(tag: '245', subfields: { k: 'Letters', b: 'to Lewis Mumford.' })
+        ]
+      end
+
+      it 'returns title value without dates' do
+        expect(helper.detailed_show(record)).to eq 'Letters to Lewis Mumford.'
+      end
+    end
+
+    # e.g., 9977704838303681
+    context 'with ǂa containing an " : " as well as inclusive dates' do
+      let(:record) do
+        marc_record fields: [
+          marc_field(tag: '245', subfields: { a: 'The frugal housewife : ',
+                                              b: 'dedicated to those who are not ashamed of economy, ',
+                                              f: '1830 / ', c: 'by the author of Hobomok.' })
+        ]
+      end
+
+      it 'returns single title value with text from ǂa and ǂn' do
+        expect(helper.detailed_show(record)).to eq(
+          'The frugal housewife : dedicated to those who are not ashamed of economy, 1830 / by the author of Hobomok.'
+        )
       end
     end
   end
@@ -179,25 +208,6 @@ describe 'PennMARC::Title' do
     end
   end
 
-  describe '.statement_of_responsibility_show' do
-    let(:record) do
-      marc_record fields: [marc_field(tag: '245', subfields: { c: 'statement of responsibility' }),
-                           marc_field(tag: '880', subfields: { '6': '245', c: 'déclaration de responsabilité' })]
-    end
-
-    context 'with ǂc defined' do
-      it 'returns statement of responsibility' do
-        expect(helper.statement_of_responsibility_show(record)).to include 'statement of responsibility'
-      end
-    end
-
-    context 'with linked alternate of 245 ǂc defined' do
-      it 'returns alternate statement of responsibility' do
-        expect(helper.statement_of_responsibility_show(record)).to include 'déclaration de responsabilité'
-      end
-    end
-  end
-
   describe '.sort' do
     context 'with a record with a valid indicator2 value' do
       let(:record) do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pennmarc
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Mike Kanning
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-08-20 00:00:00.000000000 Z
+date: 2024-08-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport