pennmarc 0.0.2 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b68bae786cc1cd9bcd1bb53e401be81bc3a97d4bb4319c6b37f9089898721424
-  data.tar.gz: ca840e625b1fbfe4e8e9cd9ef042eb2f8afbc8323e620b8fcb7f190d2e65c71c
+  metadata.gz: 84864756f38d54ec58b75b355931ba9b4310f7f880245c2c437ecebb63fee447
+  data.tar.gz: ee807c262dc0f5e2117b779b44a47cf10012c8e019340a789ac4c60d21223842
 SHA512:
-  metadata.gz: 5d7c2a945285aec3959c3594414f937005ed981c4ed0d7c6ffb567456f7c25ec7a08dfb06fb0cb8316b482743637724b756d348596946e909087a7ad51d4e658
-  data.tar.gz: e542bb11f7174cbfe3bb1f55aac1ac769a7fd004dc828354e09ac52911ffee680f3782189181d0d28617e2db9557c1cf654ae40f3c3f9e467b2e11657d9a33a2
+  metadata.gz: fee3c852594dcdbb871e093f0bd45efb4043e14bfe323f6da3d48bab3dc75eb58f968b26a24e3ae786742c112f3a9ebf7ae43fb1688f33011d4b89c4ded7af1c
+  data.tar.gz: 1192019b7f5b79c90a2d68fa46d7aa6ce7b8a2cea55f78072db3191401a4109c8d96fc5eb47ff86d05805a4cabb3409d4d90062c57ff96bdcbd9d987037160fd

data/.rubocop.yml

@@ -0,0 +1,4 @@
+inherit_from: .rubocop_todo.yml
+
+inherit_gem:
+  upennlib-rubocop: upennlib_rubocop_defaults.yml

data/.rubocop_todo.yml

@@ -0,0 +1,151 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config --auto-gen-only-exclude --exclude-limit 10000`
+# on 2023-07-28 16:23:59 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
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 21
+# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes, Max.
+Metrics/AbcSize:
+  Exclude:
+    - 'lib/pennmarc/helpers/creator.rb'
+    - 'lib/pennmarc/helpers/edition.rb'
+    - 'lib/pennmarc/helpers/format.rb'
+    - 'lib/pennmarc/helpers/genre.rb'
+    - 'lib/pennmarc/helpers/location.rb'
+    - 'lib/pennmarc/helpers/note.rb'
+    - 'lib/pennmarc/helpers/production.rb'
+    - 'lib/pennmarc/helpers/relation.rb'
+    - 'lib/pennmarc/helpers/series.rb'
+    - 'lib/pennmarc/helpers/subject.rb'
+    - 'lib/pennmarc/helpers/title.rb'
+    - 'lib/pennmarc/util.rb'
+
+# Offense count: 5
+# Configuration parameters: CountComments, Max, CountAsOne.
+Metrics/ClassLength:
+  Exclude:
+    - 'lib/pennmarc/helpers/creator.rb'
+    - 'lib/pennmarc/helpers/format.rb'
+    - 'lib/pennmarc/helpers/series.rb'
+    - 'lib/pennmarc/helpers/subject.rb'
+    - 'lib/pennmarc/helpers/title.rb'
+
+# Offense count: 18
+# Configuration parameters: AllowedMethods, AllowedPatterns, Max.
+Metrics/CyclomaticComplexity:
+  Exclude:
+    - 'lib/pennmarc/helpers/creator.rb'
+    - 'lib/pennmarc/helpers/database.rb'
+    - 'lib/pennmarc/helpers/edition.rb'
+    - 'lib/pennmarc/helpers/format.rb'
+    - 'lib/pennmarc/helpers/genre.rb'
+    - 'lib/pennmarc/helpers/note.rb'
+    - 'lib/pennmarc/helpers/production.rb'
+    - 'lib/pennmarc/helpers/relation.rb'
+    - 'lib/pennmarc/helpers/series.rb'
+    - 'lib/pennmarc/helpers/subject.rb'
+    - 'lib/pennmarc/helpers/title.rb'
+    - 'lib/pennmarc/util.rb'
+
+# Offense count: 22
+# Configuration parameters: CountComments, Max, CountAsOne, AllowedMethods, AllowedPatterns.
+Metrics/MethodLength:
+  Exclude:
+    - 'lib/pennmarc/helpers/creator.rb'
+    - 'lib/pennmarc/helpers/date.rb'
+    - 'lib/pennmarc/helpers/edition.rb'
+    - 'lib/pennmarc/helpers/format.rb'
+    - 'lib/pennmarc/helpers/location.rb'
+    - 'lib/pennmarc/helpers/note.rb'
+    - 'lib/pennmarc/helpers/production.rb'
+    - 'lib/pennmarc/helpers/relation.rb'
+    - 'lib/pennmarc/helpers/series.rb'
+    - 'lib/pennmarc/helpers/subject.rb'
+    - 'lib/pennmarc/helpers/title.rb'
+
+# Offense count: 1
+# Configuration parameters: CountComments, Max, CountAsOne.
+Metrics/ModuleLength:
+  Exclude:
+    - 'lib/pennmarc/util.rb'
+
+# Offense count: 12
+# Configuration parameters: AllowedMethods, AllowedPatterns, Max.
+Metrics/PerceivedComplexity:
+  Exclude:
+    - 'lib/pennmarc/helpers/creator.rb'
+    - 'lib/pennmarc/helpers/edition.rb'
+    - 'lib/pennmarc/helpers/format.rb'
+    - 'lib/pennmarc/helpers/genre.rb'
+    - 'lib/pennmarc/helpers/note.rb'
+    - 'lib/pennmarc/helpers/production.rb'
+    - 'lib/pennmarc/helpers/series.rb'
+    - 'lib/pennmarc/helpers/title.rb'
+    - 'lib/pennmarc/util.rb'
+
+# Offense count: 1
+# Configuration parameters: NamePrefix, ForbiddenPrefixes, AllowedMethods, MethodDefinitionMacros.
+# NamePrefix: is_, has_, have_
+# ForbiddenPrefixes: is_, has_, have_
+# AllowedMethods: is_a?
+# MethodDefinitionMacros: define_method, define_singleton_method
+Naming/PredicateName:
+  Exclude:
+    - 'lib/pennmarc/helpers/relation.rb'
+
+# Offense count: 2
+# Configuration parameters: EnforcedStyle, CheckMethodNames, CheckSymbols, AllowedIdentifiers, AllowedPatterns.
+# SupportedStyles: snake_case, normalcase, non_integer
+# AllowedIdentifiers: capture3, iso8601, rfc1123_date, rfc822, rfc2822, rfc3339, x86_64
+Naming/VariableNumber:
+  Exclude:
+    - 'lib/pennmarc/util.rb'
+
+# Offense count: 7
+# Configuration parameters: MinSize.
+Performance/CollectionLiteralInLoop:
+  Exclude:
+    - 'lib/pennmarc/helpers/creator.rb'
+    - 'lib/pennmarc/helpers/edition.rb'
+    - 'lib/pennmarc/helpers/series.rb'
+
+# Offense count: 6
+# Configuration parameters: Max, CountAsOne.
+RSpec/ExampleLength:
+  Exclude:
+    - 'spec/lib/pennmarc/helpers/creator_spec.rb'
+    - 'spec/lib/pennmarc/helpers/note_spec.rb'
+    - 'spec/lib/pennmarc/helpers/production_spec.rb'
+    - 'spec/lib/pennmarc/marc_util_spec.rb'
+
+# Offense count: 1
+# Configuration parameters: Include, CustomTransform, IgnoreMethods, SpecSuffixOnly.
+# Include: **/*_spec*rb*, **/spec/**/*
+RSpec/FilePath:
+  Exclude:
+    - 'spec/lib/pennmarc/parser_spec.rb'
+
+# Offense count: 4
+# Configuration parameters: Max, AllowedGroups.
+RSpec/NestedGroups:
+  Exclude:
+    - 'spec/lib/pennmarc/helpers/format_spec.rb'
+
+# Offense count: 2
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: Include.
+# Include: app/**/*.rb, config/**/*.rb, db/**/*.rb, lib/**/*.rb
+Rails/Output:
+  Exclude:
+    - 'lib/pennmarc/helpers/date.rb'
+
+# Offense count: 1
+# This cop supports safe autocorrection (--autocorrect).
+# Configuration parameters: Max, AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, AllowedPatterns.
+# URISchemes: http, https
+Layout/LineLength:
+  Exclude:
+    - 'lib/pennmarc/helpers/subject.rb'

data/Gemfile

@@ -7,6 +7,7 @@ gem 'library_stdnums', '~> 1.6'
 gem 'marc', '~> 1.2'
 gem 'nokogiri', '~> 1.15'
 gem 'rake', '~> 13.0'
+gem 'upennlib-rubocop', require: false
 
 group :test, :development do
   gem 'rspec', '~> 3.12'
@@ -17,7 +18,6 @@ group :test do
 end
 
 group :development do
-  gem 'upennlib-rubocop', require: false
   gem 'webrick', '~> 1.8'
   gem 'yard', '~> 0.9'
 end

data/lib/pennmarc/helpers/creator.rb

@@ -26,7 +26,7 @@ module PennMARC
       # @param [MARC::Record] record
       # @param [Hash] relator_mapping
       # @return [Array<String>] array of author/creator values for indexing
-      def search(record, relator_mapping)
+      def search(record, relator_mapping = relator_map)
         acc = record.fields(TAGS).map do |field|
           pieces = field.filter_map do |sf|
             if sf.code == 'a'
@@ -41,7 +41,7 @@ module PennMARC
             end
           end
           value = join_and_squish(pieces)
-          if value.end_with?('.') || value.end_with?('-')
+          if value.end_with?('.', '-')
             value
           else
             "#{value}."
@@ -60,7 +60,7 @@ module PennMARC
             end
           end
           value = join_and_squish(pieces)
-          if value.end_with?('.') || value.end_with?('-')
+          if value.end_with?('.', '-')
             value
           else
             "#{value}."
@@ -69,9 +69,9 @@ module PennMARC
         acc += record.fields(%w[880]).filter_map do |field|
           next unless field.any? { |sf| sf.code == '6' && sf.value.in?(%w[100 110]) }
 
-          suba = field.find_all(&subfield_in?(%w[a])).map do |sf|
+          suba = field.find_all(&subfield_in?(%w[a])).map { |sf|
             convert_name_order(sf.value)
-          end.first
+          }.first
           oth = join_and_squish(field.find_all(&subfield_not_in?(%w[6 8 a t])).map(&:value))
           join_and_squish [suba, oth]
         end
@@ -90,7 +90,7 @@ module PennMARC
       # @param [MARC::Record] record
       # @param [Hash] relator_mapping
       # @return [Array<String>] array of author/creator values for display
-      def values(record, relator_mapping)
+      def values(record, relator_mapping = relator_map)
         record.fields(TAGS).map do |field|
           name_from_main_entry(field, relator_mapping)
         end
@@ -135,7 +135,7 @@ module PennMARC
         }
         source_map.flat_map do |field_num, subfields|
           record.fields(field_num.to_s).map do |field|
-            trim_punctuation(join_subfields(field, &subfield_in?(subfields.split(''))))
+            trim_punctuation(join_subfields(field, &subfield_in?(subfields.chars)))
           end
         end
       end
@@ -143,11 +143,11 @@ module PennMARC
       # Conference for display, intended for results display
       # @note ported from get_conference_values
       # @param [MARC::Record] record
-      # @param [Hash] relator_map
+      # @param [Hash] relator_mapping
       # @return [Array<String>] array of conference values
-      def conference_show(record, relator_map)
+      def conference_show(record, relator_mapping = relator_map)
         record.fields('111').filter_map do |field|
-          name_from_main_entry field, relator_map
+          name_from_main_entry field, relator_mapping
         end
       end
 
@@ -183,6 +183,40 @@ module PennMARC
       # @note see get_conference_search_values
       def conference_search(record); end
 
+      # Retrieve contributor values for display from fields {https://www.oclc.org/bibformats/en/7xx/700.html 700}
+      # and {https://www.oclc.org/bibformats/en/7xx/710.html 710} and their linked alternates. Joins subfields
+      # 'a', 'b', 'c', 'd', 'j', and 'q'. Then appends resulting string with joined subfields 'e', 'u', '3', and '4'.
+      # @note legacy version returns array of hash objects including data for display link
+      # @param [MARC::Record] record
+      # @ param [Hash] relator_mapping
+      # @return [Array<String>]
+      def contributor_show(record, relator_mapping = relator_map)
+        contributors = record.fields(%w[700 710]).filter_map do |field|
+          next unless ['', ' ', '0'].member?(field.indicator2)
+          next if subfield_defined? field, 'i'
+
+          contributor = join_subfields(field, &subfield_in?(%w[a b c d j q]))
+          contributor_append = field.filter_map { |subfield|
+            next unless %w[e u 3 4].member?(subfield.code)
+
+            if subfield.code == '4'
+              ", #{translate_relator(subfield.value, relator_mapping)}"
+            else
+              " #{subfield.value}"
+            end
+          }.join
+          "#{contributor} #{contributor_append}".squish
+        end
+        contributors + record.fields('880').filter_map do |field|
+          next unless subfield_value_in?(field, '6', %w[700 710])
+          next if subfield_defined?(field, 'i')
+
+          contributor = join_subfields(field, &subfield_in?(%w[a b c d j q]))
+          contributor_append = join_subfields(field, &subfield_in?(%w[e u 3]))
+          "#{contributor} #{contributor_append}".squish
+        end
+      end
+
       private
 
       # Trim punctuation method extracted from Traject macro, to ensure consistent output
@@ -209,7 +243,7 @@ module PennMARC
       # @param [MARC::Field] field
       # @return [String] joined subfield values for value from field
       def name_from_main_entry(field, mapping)
-        s = field.filter_map do |sf|
+        s = field.filter_map { |sf|
           if %w[0 1 4 6 8].exclude?(sf.code)
             " #{sf.value}"
           elsif sf.code == '4'
@@ -218,8 +252,8 @@ module PennMARC
 
             ", #{relator}"
           end
-        end.join
-        (s + (!%w[. -].member?(s.last) ? '.' : '')).squish
+        }.join
+        (s + (%w[. -].member?(s.last) ? '' : '.')).squish
       end
 
       # Convert "Lastname, First" to "First Lastname"

data/lib/pennmarc/helpers/database.rb

@@ -21,7 +21,7 @@ module PennMARC
       def type(record)
         record.fields('944').filter_map do |field|
           # skip unless specified database format type present
-          next unless subfield_value?(field, 'a', /#{DATABASES_FACET_VALUE}/)
+          next unless subfield_value?(field, 'a', /#{DATABASES_FACET_VALUE}/o)
 
           type = field.find { |subfield| subfield.code == 'b' }
           type&.value
@@ -39,7 +39,7 @@ module PennMARC
 
         record.fields('943').filter_map do |field|
           # skip unless Community of Interest code is in subfield '2'
-          next unless subfield_value?(field, '2', /#{COI_CODE}/)
+          next unless subfield_value?(field, '2', /#{COI_CODE}/o)
 
           category = field.find { |subfield| subfield.code == 'a' }
           category&.value
@@ -60,17 +60,17 @@ module PennMARC
 
         record.fields('943').filter_map do |field|
           # skip unless Community of Interest code is in subfield '2'
-          next unless subfield_value?(field, '2', /#{COI_CODE}/)
+          next unless subfield_value?(field, '2', /#{COI_CODE}/o)
 
           category = field.find { |subfield| subfield.code == 'a' }
 
-          # skip unless category is present
-          next unless category.present?
+          # skip if category is blank
+          next if category.blank?
 
           subcategory = field.find { |subfield| subfield.code == 'b' }
 
-          # skip unless subcategory is present
-          next unless subcategory.present?
+          # skip if subcategory is blank
+          next if subcategory.blank?
 
           "#{category.value}--#{subcategory.value}"
         end
@@ -82,7 +82,7 @@ module PennMARC
       # @param [Marc::Record]
       # @return [TrueClass, FalseClass]
       def curated_db?(record)
-        record.fields('944').any? { |field| subfield_value?(field, 'a', /#{DATABASES_FACET_VALUE}/) }
+        record.fields('944').any? { |field| subfield_value?(field, 'a', /#{DATABASES_FACET_VALUE}/o) }
       end
     end
   end

data/lib/pennmarc/helpers/date.rb

@@ -10,13 +10,13 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [DateTime, nil] The publication date, or nil if date found in record is invalid
       def publication(record)
-        record.fields('008').filter_map do |field|
+        record.fields('008').filter_map { |field|
           four_digit_year = sanitize_partially_known_date(field.value[7, 4], '0')
 
-          next unless four_digit_year.present?
+          next if four_digit_year.blank?
 
           DateTime.new(four_digit_year.to_i)
-        end.first
+        }.first
       end
 
       # Retrieve date added (subfield 'q') from enriched marc 'itm' field.
@@ -24,7 +24,7 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [DateTime, nil] The date added, or nil if date found in record is invalid
       def added(record)
-        record.fields(EnrichedMarc::TAG_ITEM).flat_map do |field|
+        record.fields(EnrichedMarc::TAG_ITEM).flat_map { |field|
           field.filter_map do |subfield|
             # skip unless field has date created subfield
             next unless subfield_defined?(field, EnrichedMarc::SUB_ITEM_DATE_CREATED)
@@ -42,7 +42,7 @@ module PennMARC
             puts "Error parsing date in date added subfield: #{subfield.value} - #{e}"
             nil
           end
-        end.max
+        }.max
       end
 
       # Retrieve date last updated from {https://www.loc.gov/marc/bibliographic/bd005.html 005 field}.
@@ -51,19 +51,20 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [DateTime, nil] The date last updated, or nil if date found in record is invalid
       def last_updated(record)
-        record.fields('005').filter_map do |field|
-          date_time_string = field.value
+        record.fields('005').filter_map { |field|
+          begin
+            date_time_string = field.value
 
-          next if date_time_string.blank?
+            next if date_time_string.blank?
 
-          next if date_time_string.start_with?('0000')
+            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}"
-          nil
-        end.first
+            DateTime.iso8601(date_time_string).to_datetime
+          rescue ArgumentError => e
+            puts "Error parsing last updated date: #{date_time_string} - #{e}"
+            nil
+          end
+        }.first
       end
 
       private

data/lib/pennmarc/helpers/edition.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module PennMARC
-  # Do Edition-y stuff
+  # Do Edition and edition-related field processing.
   class Edition < Helper
     class << self
       # Edition values for display on a record page. Field 250 is information relating to the edition of a work as
@@ -14,9 +14,9 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>] array of editions and their alternates
       def show(record)
-        record.fields('250').map do |field|
+        record.fields('250').map { |field|
           join_subfields(field, &subfield_not_in?(%w[6 8]))
-        end + linked_alternate_not_6_or_8(record, '250')
+        } + linked_alternate_not_6_or_8(record, '250')
       end
 
       # Edition values for display in search results. Just grab the first 250 field.
@@ -24,7 +24,7 @@ module PennMARC
       # @return [String, NilClass] string of all first 250 subfields, excluding 6 and 8
       def values(record)
         edition = record.fields('250').first
-        return unless edition.present?
+        return if edition.blank?
 
         join_subfields(edition, &subfield_not_in?(%w[6 8]))
       end
@@ -34,18 +34,21 @@ module PennMARC
       # display.
       # https://www.loc.gov/marc/bibliographic/bd775.html
       # @param [MARC::Record] record
+      # @param [Hash] relator_mapping
       # @return [Array<String>] array of other edition strings
-      def other_show(record, relator_mapping)
-        record.fields('775').filter_map do |field|
+      def other_show(record, relator_mapping = relator_map)
+        values = record.fields('775').filter_map do |field|
           next unless subfield_defined?(field, :i)
 
           other_edition_value(field, relator_mapping)
-        end + record.fields('880').filter_map do |field|
+        end
+        values += record.fields('880').filter_map do |field|
           next unless field.indicator2.blank? && subfield_value_in?(field, '6', %w[775]) &&
                       subfield_defined?(field, 'i')
 
           other_edition_value(field, relator_mapping)
         end
+        values
       end
 
       private
@@ -56,7 +59,7 @@ module PennMARC
       # @return [String (frozen)] assembled other version string
       def other_edition_value(field, relator_mapping)
         subi = remove_paren_value_from_subfield_i(field) || ''
-        other_editions = field.filter_map do |sf|
+        other_editions = field.filter_map { |sf|
           next if %w[6 8].member?(sf.code)
 
           if %w[s x z].member?(sf.code)
@@ -67,8 +70,8 @@ module PennMARC
 
             " #{relator}. "
           end
-        end.join
-        other_editions_append = field.filter_map do |sf|
+        }.join
+        other_editions_append = field.filter_map { |sf|
           next if %w[6 8].member?(sf.code)
 
           if %w[i h s t x z e f o r w y 7].exclude?(sf.code)
@@ -76,7 +79,7 @@ module PennMARC
           elsif sf.code == 'h'
             " (#{sf.value}) "
           end
-        end.join
+        }.join
         prepend = trim_trailing(:period, subi).squish
 
         if other_editions.present? || other_editions_append.present?

data/lib/pennmarc/helpers/format.rb

@@ -68,10 +68,10 @@ module PennMARC
       #       for this helper method
       # @note ported from get_format
       # @param [MARC::Record] record
-      # @param [Hash] location_map
+      # @param [Hash] location_mapping
       # @return [Array<String>] format values for faceting
 
-      def facet(record, location_map)
+      def facet(record, location_mapping = location_map)
         formats = []
         format_code = leader_format(record.leader)
         f007 = record.fields('007').map(&:value)
@@ -88,7 +88,8 @@ module PennMARC
         end
 
         # get all specific_location values from inventory info
-        locations = Location.location record: record, location_map: location_map, display_value: :specific_location
+        locations = Location.location record: record, location_mapping: location_mapping,
+                                      display_value: :specific_location
 
         if include_manuscripts?(locations)
           formats << MANUSCRIPT
@@ -147,6 +148,16 @@ module PennMARC
         end
       end
 
+      # Retrieve cartographic reference data for map/atlas formats for display from
+      # {https://www.oclc.org/bibformats/en/2xx/255.html 255} and {https://www.oclc.org/bibformats/en/3xx/342.html 342}
+      # @param [MARC::Record] record
+      # @return [Array<String>]
+      def cartographic_show(record)
+        record.fields(%w[255 342]).map do |field|
+          join_subfields(field, &subfield_not_in?(%w[6 8]))
+        end
+      end
+
       # Check if a set of locations has any locations that include the term 'manuscripts'
       # @param [Array<String>] locations
       # @return [Boolean]
@@ -162,12 +173,12 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array]
       def curated_format(record)
-        record.fields('944').filter_map do |field|
+        record.fields('944').filter_map { |field|
           subfield_a = field.find { |sf| sf.code == 'a' }
           next if subfield_a.nil? || (subfield_a.value == subfield_a.value.to_i.to_s)
 
           subfield_a.value
-        end.uniq
+        }.uniq
       end
 
       # @param [String] format_code

data/lib/pennmarc/helpers/genre.rb

@@ -11,9 +11,9 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>] array of genre values for search
       def search(record)
-        record.fields('655').map do |field|
+        record.fields('655').map { |field|
           join_subfields(field, &subfield_not_in?(%w[0 2 5 c]))
-        end.uniq
+        }.uniq
       end
 
       # Genre values for display. We display Genre/Term values if they fulfill the following criteria:
@@ -29,34 +29,35 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>] array of genre values for display
       def show(record)
-        record.fields(%w[655 880]).filter_map do |field|
+        record.fields(%w[655 880]).filter_map { |field|
           next unless allowed_genre_field?(field)
 
           next if field.tag == '880' && subfield_values(field, '6').exclude?('655')
 
-          sub_with_hyphens = field.find_all(&subfield_not_in?(%w[0 2 5 6 8 c e w])).map do |sf|
+          sub_with_hyphens = field.find_all(&subfield_not_in?(%w[0 2 5 6 8 c e w])).map { |sf|
             sep = %w[a b].exclude?(sf.code) ? ' -- ' : ' '
             sep + sf.value
-          end.join.lstrip
+          }.join.lstrip
           "#{sub_with_hyphens} #{field.find_all(&subfield_in?(%w[e w])).join(' -- ')}".strip
-        end.uniq
+        }.uniq
       end
 
       # Genre values for faceting. We only set Genre facet values for movies (videos) and manuscripts(?)
       # @todo the Genre facet in Franklin is pretty ugly. It could be cleaned up by limiting the subfields included
       #       here and cleaning up punctuation.
       # @param [MARC::Record] record
-      # @param [Hash] location_map
+      # @param [Hash] location_mapping
       # @return [Array<String>]
-      def facet(record, location_map)
-        locations = Location.location record: record, location_map: location_map, display_value: :specific_location
+      def facet(record, location_mapping = location_map)
+        locations = Location.location record: record, location_mapping: location_mapping,
+                                      display_value: :specific_location
         manuscript = Format.include_manuscripts?(locations)
         video = record.fields('007').any? { |field| field.value.starts_with? 'v' }
         return [] unless manuscript || video
 
-        record.fields('655').filter_map do |field|
+        record.fields('655').filter_map { |field|
           join_subfields field, &subfield_not_in?(%w[0 2 5 c])
-        end.uniq
+        }.uniq
       end
 
       private