pennmarc 1.0.12 → 1.0.15.pre

data/lib/pennmarc/helpers/helper.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative '../enriched_marc'
+require_relative '../enriched'
 require_relative '../mappers'
 require_relative '../util'
 

data/lib/pennmarc/helpers/identifier.rb

@@ -34,11 +34,12 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>]
       def isbn_show(record)
-        isbn_values = record.fields('020').filter_map do |field|
+        values = record.fields('020').filter_map do |field|
           joined_isbn = join_subfields(field, &subfield_in?(%w[a]))
           joined_isbn.presence
         end
-        isbn_values + linked_alternate(record, '020', &subfield_in?(%w[a]))
+        isbn_values = values + linked_alternate(record, '020', &subfield_in?(%w[a]))
+        isbn_values.uniq
       end
 
       # Get ISSN values for display from the {https://www.oclc.org/bibformats/en/0xx/022.html 022 field} and related
@@ -47,11 +48,12 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>]
       def issn_show(record)
-        issn_values = record.fields('022').filter_map do |field|
+        values = record.fields('022').filter_map do |field|
           joined_issn = join_subfields(field, &subfield_in?(%w[a]))
           joined_issn.presence
         end
-        issn_values + linked_alternate(record, '022', &subfield_in?(%w[a]))
+        issn_values = values + linked_alternate(record, '022', &subfield_in?(%w[a]))
+        issn_values.uniq
       end
 
       # Get numeric OCLC ID of first {https://www.oclc.org/bibformats/en/0xx/035.html 035 field}
@@ -82,7 +84,7 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>]
       def oclc_id_search(record)
-        record.fields('035').flat_map do |field|
+        record.fields('035').flat_map { |field|
           field.filter_map do |subfield|
             # skip unless subfield 'a' or 'z'
             next unless subfield.code.in?(%w[a z])
@@ -98,7 +100,7 @@ module PennMARC
 
             match[1]
           end
-        end
+        }.uniq
       end
 
       # Get publisher issued identifiers from fields {https://www.oclc.org/bibformats/en/0xx/024.html 024},
@@ -109,7 +111,7 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<string>]
       def publisher_number_show(record)
-        record.fields(%w[024 028 880]).filter_map do |field|
+        record.fields(%w[024 028 880]).filter_map { |field|
           next if field.tag == '880' && subfield_value_not_in?(field, '6', %w[024 028])
 
           # do not return doi values from 024 ǂ2
@@ -118,7 +120,7 @@ module PennMARC
           else
             join_subfields(field, &subfield_not_in?(%w[5 6])).presence
           end
-        end
+        }.uniq
       end
 
       # Get publisher issued identifiers for searching of a record. Values extracted from fields
@@ -127,19 +129,19 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>]
       def publisher_number_search(record)
-        record.fields(%w[024 028]).filter_map do |field|
+        record.fields(%w[024 028]).filter_map { |field|
           joined_identifiers = join_subfields(field, &subfield_in?(%w[a]))
           joined_identifiers.presence
-        end
+        }.uniq
       end
 
       # Retrieve fingerprint for display from the {https://www.oclc.org/bibformats/en/0xx/026.html 026} field
       # @param [MARC::Record] record
       # @return [Array<String>]
       def fingerprint_show(record)
-        record.fields('026').map do |field|
+        record.fields('026').map { |field|
           join_subfields(field, &subfield_not_in?(%w[2 5 6 8]))
-        end
+        }.uniq
       end
 
       # Retrieve DOI values stored in {https://www.oclc.org/bibformats/en/0xx/024.html 024}.
@@ -147,14 +149,14 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>]
       def doi_show(record)
-        record.fields('024').filter_map do |field|
+        record.fields('024').filter_map { |field|
           # skip unless indicator1 is '7'
           next unless field.indicator1.in?(%w[7])
           # skip unless ǂ2 is the string literal 'doi'
           next unless subfield_value_in?(field, '2', %w[doi])
 
           join_subfields(field, &subfield_in?(%w[a]))
-        end
+        }.uniq
       end
 
       private

data/lib/pennmarc/helpers/inventory.rb

@@ -0,0 +1,92 @@
+# frozen_string_literal: true
+
+require_relative 'inventory_entry/electronic'
+require_relative 'inventory_entry/physical'
+
+module PennMARC
+  # Methods for extracting holdings information ("inventory") when available
+  class Inventory < Helper
+    PHYSICAL_INVENTORY_TAGS = [
+      Enriched::Pub::PHYS_INVENTORY_TAG,
+      Enriched::Api::PHYS_INVENTORY_TAG
+    ].freeze
+
+    ELECTRONIC_INVENTORY_TAGS = [
+      Enriched::Pub::ELEC_INVENTORY_TAG,
+      Enriched::Api::ELEC_INVENTORY_TAG
+    ].freeze
+
+    class << self
+      # Hash of Physical holdings information
+      # @param [MARC::Record] record
+      # @return [Array, nil]
+      def physical(record)
+        source = enrichment_source(record)
+        return unless source
+
+        record.fields(PHYSICAL_INVENTORY_TAGS).map do |entry|
+          InventoryEntry::Physical.new(entry, source).to_h
+        end
+      end
+
+      # Hash of Electronic inventory information
+      # @param [MARC::Record] record
+      # @return [Array, nil]
+      def electronic(record)
+        source = enrichment_source(record)
+        return unless source
+
+        record.fields(ELECTRONIC_INVENTORY_TAGS).map do |entry|
+          InventoryEntry::Electronic.new(entry, source).to_h
+        end
+      end
+
+      # Count of all electronic portfolios
+      # @param [MARC::Record] record
+      # @return [Integer]
+      def electronic_portfolio_count(record)
+        record.count { |field| field.tag.in? %w[AVE prt] }
+      end
+
+      # Count of all physical holdings
+      # @param [MARC::Record] record
+      # @return [Integer]
+      def physical_holding_count(record)
+        record.count { |field| field.tag.in? %w[AVA hld] }
+      end
+
+      private
+
+      # Determine the source of the MARC inventory enrichment
+      # @param [MARC::Record] record
+      # @return [Symbol, nil]
+      def enrichment_source(record)
+        if pub_enrichment_tags?(record)
+          :pub
+        elsif api_enrichment_tags?(record)
+          :api
+        end
+      end
+
+      # Does the record include tags from Publishing inventory enrichment?
+      # @todo move to Util?
+      # @param [MARC::Record] record
+      # @return [Boolean]
+      def pub_enrichment_tags?(record)
+        record.tags.intersect?(
+          [Enriched::Pub::PHYS_INVENTORY_TAG, Enriched::Pub::ELEC_INVENTORY_TAG, Enriched::Pub::ITEM_TAG]
+        )
+      end
+
+      # Does the record include tags from API inventory enrichment?
+      # @todo move to Util?
+      # @param [MARC::Record] record
+      # @return [Boolean]
+      def api_enrichment_tags?(record)
+        record.tags.intersect?(
+          [Enriched::Api::PHYS_INVENTORY_TAG, Enriched::Api::ELEC_INVENTORY_TAG]
+        )
+      end
+    end
+  end
+end

data/lib/pennmarc/helpers/inventory_entry/base.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module PennMARC
+  module InventoryEntry
+    # Base class for InventoryEntry classes, defines required interface
+    class Base
+      attr_reader :source, :field, :mapper
+
+      # @param [MARC::DataField] inventory_field
+      # @param [Symbol] source
+      def initialize(inventory_field, source)
+        @source = source
+        @field = inventory_field
+        @mapper = @source == :api ? Enriched::Api : Enriched::Pub
+      end
+
+      # @return [Hash]
+      def to_h
+        raise NotImplementedError
+      end
+    end
+  end
+end

data/lib/pennmarc/helpers/inventory_entry/electronic.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+require_relative 'base'
+
+module PennMARC
+  module InventoryEntry
+    # Represent a Electronic inventory entry - simple because the subfield specification is identical across
+    # entries returned by the API and Alma Publishing enrichment
+    class Electronic < Base
+      # @return [Hash{Symbol->Unknown}]
+      def to_h
+        { portfolio_id: field[mapper::ELEC_PORTFOLIO_ID],
+          url: field[mapper::ELEC_SERVICE_URL],
+          collection_name: field[mapper::ELEC_COLLECTION_NAME],
+          coverage: field[mapper::ELEC_COVERAGE_STMT],
+          note: field[mapper::ELEC_PUBLIC_NOTE] }
+      end
+    end
+  end
+end

data/lib/pennmarc/helpers/inventory_entry/physical.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require_relative 'base'
+
+module PennMARC
+  module InventoryEntry
+    # Represent a Physical inventory entry
+    class Physical < Base
+      # Call number from inventory entry
+      # @return [String (frozen)]
+      def call_num
+        if source == :pub
+          "#{field[mapper::HOLDING_CLASSIFICATION_PART]}#{field[mapper::HOLDING_ITEM_PART]}"
+        elsif source == :api
+          field[mapper::PHYS_CALL_NUMBER]
+        end
+      end
+
+      # Priority for inventory entry
+      # @note we currently don't return priority in our publishing enrichment
+      # @return [String, nil]
+      def priority
+        return nil if source == :pub
+
+        field[mapper::PHYS_PRIORITY]
+      end
+
+      # @return [Hash{Symbol->Unknown}]
+      def to_h
+        { holding_id: field[mapper::PHYS_HOLDING_ID],
+          location_name: field[mapper::PHYS_LOCATION_NAME],
+          location_code: field[mapper::PHYS_LOCATION_CODE],
+          call_num: call_num,
+          priority: priority }
+      end
+    end
+  end
+end

data/lib/pennmarc/helpers/language.rb

@@ -18,7 +18,8 @@ module PennMARC
         values = record.fields('546').map do |field|
           join_subfields field, &subfield_not_in?(%w[6 8])
         end
-        values + linked_alternate(record, '546', &subfield_not_in?(%w[6 8]))
+        language_values = values + linked_alternate(record, '546', &subfield_not_in?(%w[6 8]))
+        language_values.uniq
       end
 
       # Get language values for searching and faceting of a record. The values are extracted from subfields
@@ -33,7 +34,7 @@ module PennMARC
       # @param [MARC::Record] record
       # @param [Hash] iso_639_2_mapping iso-639-2 spec hash for language code translation
       # @param [Hash] iso_639_3_mapping iso-639-3 spec hash for language code translation
-      # @return [Array] array of language values
+      # @return [Array<String>] array of language values
       def values(record, iso_639_2_mapping: Mappers.iso_639_2_language, iso_639_3_mapping: Mappers.iso_639_3_language)
         values = record.fields('041').filter_map { |field|
           mapper = subfield_value?(field, '2', /iso639-3/) ? iso_639_3_mapping : iso_639_2_mapping

data/lib/pennmarc/helpers/location.rb

@@ -3,10 +3,12 @@
 module PennMARC
   # Methods that return Library and Location values from Alma enhanced MARC fields
   class Location < Helper
+    ONLINE_LIBRARY = 'Online library'
+
     class << self
       # Retrieves library location from enriched marc 'itm' or 'hld' fields, giving priority to the item location over
       # the holdings location. Returns item's location if available. Otherwise, returns holding's location.
-      # {PennMARC::EnrichedMarc} maps enriched marc fields and subfields created during Alma publishing.
+      # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing.
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param [MARC::Record] record
@@ -19,7 +21,7 @@ module PennMARC
       # Retrieves the specific location from enriched marc 'itm' or 'hld' fields, giving priority to the item location
       # over the holdings location. Returns item library location if available. Otherwise, returns holdings library
       # location.
-      # {PennMARC::EnrichedMarc} maps enriched marc fields and subfields created during Alma publishing.
+      # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing.
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param [MARC::Record] record
@@ -31,7 +33,7 @@ module PennMARC
 
       # Base method to retrieve location data from enriched marc 'itm' or 'hld' fields, giving priority to the item
       # location over the holdings location. Returns item location if available. Otherwise, returns holdings location.
-      # {PennMARC::EnrichedMarc} maps enriched marc fields and subfields created during Alma publishing.
+      # {PennMARC::Enriched} maps enriched marc fields and subfields created during Alma publishing.
       # @see https://developers.exlibrisgroup.com/alma/apis/docs/bibs/R0VUIC9hbG1hd3MvdjEvYmlicy97bW1zX2lkfQ==/
       #   Alma documentation for these added fields
       # @param [MARC::Record] record
@@ -61,7 +63,9 @@ module PennMARC
             location_map[subfield.value.to_sym][display_value.to_sym]
           }.flatten.compact_blank
         }.uniq
-        locations << 'Online library' if record.fields(PennMARC::EnrichedMarc::TAG_ELECTRONIC_INVENTORY).any?
+        if record.tags.intersect?([Enriched::Pub::ELEC_INVENTORY_TAG, Enriched::Api::ELEC_INVENTORY_TAG])
+          locations << ONLINE_LIBRARY
+        end
         locations
       end
 
@@ -82,17 +86,18 @@ module PennMARC
         # Since item records may reflect locations more accurately, we use them if they exist;
         # if not, we use the holdings.
 
-        tag = PennMARC::EnrichedMarc::TAG_HOLDING
-        subfield_code = PennMARC::EnrichedMarc::SUB_HOLDING_SHELVING_LOCATION
-
         # if the record has an enriched item field present, use it
-        if field_defined?(record, PennMARC::EnrichedMarc::TAG_ITEM)
-          tag = PennMARC::EnrichedMarc::TAG_ITEM
-          subfield_code = PennMARC::EnrichedMarc::SUB_ITEM_CURRENT_LOCATION
-        # otherwise, use the api enriched field value
-        elsif field_defined?(record, EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY)
-          tag = EnrichedMarc::AlmaApi::TAG_PHYSICAL_INVENTORY
-          subfield_code = EnrichedMarc::AlmaApi::SUB_HOLDING_LOCATION_CODE
+        if field_defined?(record, Enriched::Pub::ITEM_TAG)
+          tag = Enriched::Pub::ITEM_TAG
+          subfield_code = Enriched::Pub::ITEM_CURRENT_LOCATION
+        # if the record has API inventory tags, use them
+        elsif field_defined?(record, Enriched::Api::PHYS_INVENTORY_TAG)
+          tag = Enriched::Api::PHYS_INVENTORY_TAG
+          subfield_code = Enriched::Api::PHYS_LOCATION_CODE
+        # otherwise use Pub holding tags
+        else
+          tag = Enriched::Pub::PHYS_INVENTORY_TAG
+          subfield_code = Enriched::Pub::PHYS_LOCATION_CODE
         end
 
         { tag: tag, subfield_code: subfield_code }

data/lib/pennmarc/helpers/note.rb

@@ -16,11 +16,11 @@ module PennMARC
       # @return [Array<String>]
       def notes_show(record)
         notes_fields = %w[500 502 504 515 518 525 533 540 550 580 586 588]
-        record.fields(notes_fields + ['880']).filter_map do |field|
+        record.fields(notes_fields + ['880']).filter_map { |field|
           next if field.tag == '880' && subfield_value_not_in?(field, '6', notes_fields)
 
           join_subfields(field, &subfield_not_in?(%w[5 6 8]))
-        end
+        }.uniq
       end
 
       # Retrieve local notes for display from fields {https://www.oclc.org/bibformats/en/5xx/561.html 561},
@@ -38,11 +38,12 @@ module PennMARC
 
         additional_fields = %w[562 563 585 590]
 
-        local_notes + record.fields(additional_fields + ['880']).filter_map do |field|
+        notes = local_notes + record.fields(additional_fields + ['880']).filter_map do |field|
           next if field.tag == '880' && subfield_value_not_in?(field, '6', additional_fields)
 
           join_subfields(field, &subfield_not_in?(%w[5 6 8]))
         end
+        notes.uniq
       end
 
       # Retrieve provenance notes for display from fields {https://www.oclc.org/bibformats/en/5xx/561.html 561} and
@@ -63,7 +64,8 @@ module PennMARC
 
           join_subfields(field, &subfield_in?(%w[a]))
         end
-        provenance_notes + prefixed_subject_and_alternate(record, 'PRO')
+        notes = provenance_notes + prefixed_subject_and_alternate(record, 'PRO')
+        notes.uniq
       end
 
       # Retrieve contents notes for display from fields {https://www.oclc.org/bibformats/en/5xx/505.html 505} and
@@ -75,16 +77,16 @@ module PennMARC
           next if field.tag == '880' && subfield_value_not_in?(field, '6', %w[505])
 
           join_subfields(field, &subfield_not_in?(%w[6 8])).split('--')
-        }.flatten
+        }.flatten.uniq
       end
 
       # Retrieve access restricted notes for display from field {https://www.oclc.org/bibformats/en/5xx/506.html 506}.
       # @param [MARC::Record] record
       # @return [Array<String>]
       def access_restriction_show(record)
-        record.fields('506').filter_map do |field|
+        record.fields('506').filter_map { |field|
           join_subfields(field, &subfield_not_in?(%w[5 6]))
-        end
+        }.uniq
       end
 
       # Retrieve finding aid notes for display from field {https://www.oclc.org/bibformats/en/5xx/555.html 555} and its
@@ -161,7 +163,7 @@ module PennMARC
 
           sub3_and_other_subs(field, &subfield_in?(%w[a b c d e f]))
         end
-        system_details_notes
+        system_details_notes.uniq
       end
 
       private

data/lib/pennmarc/helpers/production.rb

@@ -8,21 +8,21 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array<String>]
       def show(record)
-        get_264_or_880_fields(record, '0')
+        get_264_or_880_fields(record, '0').uniq
       end
 
       # Retrieve distribution values for display from {https://www.oclc.org/bibformats/en/2xx/264.html 264 field}.
       # @param [MARC::Record] record
       # @return [Array<String>]
       def distribution_show(record)
-        get_264_or_880_fields(record, '2')
+        get_264_or_880_fields(record, '2').uniq
       end
 
       # Retrieve manufacture values for display from {https://www.oclc.org/bibformats/en/2xx/264.html 264 field}.
       # @param [MARC::Record] record
       # @return [Array<String>]
       def manufacture_show(record)
-        get_264_or_880_fields(record, '3')
+        get_264_or_880_fields(record, '3').uniq
       end
 
       # Retrieve publication values. Return publication values from
@@ -60,7 +60,7 @@ module PennMARC
 
           values += joined264
         end
-        values.filter_map { |value| value&.strip }
+        values.filter_map { |value| value&.strip }.uniq
       end
 
       # Retrieve publication values for display from fields
@@ -68,7 +68,7 @@ module PennMARC
       # {https://www.oclc.org/bibformats/en/2xx/260.html 260}-262 and their linked alternates,
       # and {https://www.oclc.org/bibformats/en/2xx/264.html 264} and its linked alternate.
       # @param [MARC::Record] record
-      # @return [Object]
+      # @return [Array<String>]
       def publication_show(record)
         values = record.fields('245').first(1).flat_map { |field| subfield_values(field, 'f') }
 
@@ -86,22 +86,22 @@ module PennMARC
         end
 
         values += get_264_or_880_fields(record, '1')
-        values.compact_blank
+        values.compact_blank.uniq
       end
 
       # Retrieve place of publication for display from {https://www.oclc.org/bibformats/en/7xx/752.html 752 field} and
       # its linked alternate.
       # @note legacy version returns array of hash objects including data for display link
       # @param [MARC::Record] record
-      # @return [Object]
+      # @return [Array<String>]
       def place_of_publication_show(record)
-        record.fields(%w[752 880]).filter_map do |field|
+        record.fields(%w[752 880]).filter_map { |field|
           next if field.tag == '880' && subfield_values(field, '6').exclude?('752')
 
           place = join_subfields(field, &subfield_not_in?(%w[6 8 e w]))
           place_extra = join_subfields(field, &subfield_in?(%w[e w]))
           "#{place} #{place_extra}"
-        end
+        }.uniq
       end
 
       private

data/lib/pennmarc/helpers/relation.rb

@@ -14,11 +14,11 @@ module PennMARC
       # in this field should be sufficient to locate host item record.
       #
       # @param [MARC::Record] record
-      # @return [Array] contained in values for display
+      # @return [Array<String>] contained in values for display
       def contained_in_show(record)
-        record.fields('773').map do |field|
+        record.fields('773').map { |field|
           join_subfields(field, &subfield_not_in?(%w[6 7 8 w]))
-        end
+        }.uniq
       end
 
       # Get "chronology" information from specially-prefixed 650 (subject) fields
@@ -56,7 +56,7 @@ module PennMARC
 
           values_with_title_prefix(field, sf_exclude: %w[0 4 6 8 i], relator_map: relator_map)
         end
-        values + record.fields('880').filter_map do |field|
+        relation_values = values + record.fields('880').filter_map do |field|
           next if field.indicator2.present?
 
           next unless subfield_value?(field, '6', /^(#{RELATED_WORK_FIELDS.join('|')})/)
@@ -65,6 +65,7 @@ module PennMARC
 
           values_with_title_prefix(field, sf_exclude: %w[0 4 6 8 i], relator_map: relator_map)
         end
+        relation_values.uniq
       end
 
       # Get "Contains" values from {CONTAINS_FIELDS} in the 7XX range. Must have indicator 2 value of 2 indicating an
@@ -72,20 +73,21 @@ module PennMARC
       # values in sf 0, 5, 6, and 8.
       # @param [MARC::Record] record
       # @param [Hash] relator_map
-      # @return [Array]
+      # @return [Array<String>]
       def contains_show(record, relator_map: Mappers.relator)
-        acc = record.fields(CONTAINS_FIELDS).filter_map do |field|
+        values = record.fields(CONTAINS_FIELDS).filter_map do |field|
           next unless field.indicator2 == '2'
 
           values_with_title_prefix(field, sf_exclude: %w[0 4 5 6 8 i], relator_map: relator_map)
         end
-        acc + record.fields('880').filter_map do |field|
+        contains_values = values + record.fields('880').filter_map do |field|
           next unless field.indicator2 == '2'
 
           next unless subfield_value?(field, '6', /^(#{CONTAINS_FIELDS.join('|')})/)
 
           values_with_title_prefix(field, sf_include: %w[0 5 6 8 i])
         end
+        contains_values.uniq
       end
 
       # Get "Constituent Unit" values from {https://www.oclc.org/bibformats/en/7xx/774.html MARC 774}. Include
@@ -93,10 +95,11 @@ module PennMARC
       # @param [MARC::Record] record
       # @return [Array]
       def constituent_unit_show(record)
-        acc = record.fields('774').filter_map do |field|
+        values = record.fields('774').filter_map do |field|
           join_subfields(field, &subfield_in?(%w[i a s t]))
         end
-        acc + linked_alternate(record, '774', &subfield_in?(%w[i a s t]))
+        constituent_values = values + linked_alternate(record, '774', &subfield_in?(%w[i a s t]))
+        constituent_values.uniq
       end
 
       # Get "Has Supplement" values from {https://www.oclc.org/bibformats/en/7xx/770.html MARC 770}. Ignore

data/lib/pennmarc/helpers/series.rb

@@ -31,7 +31,8 @@ module PennMARC
                  end || []
 
         values += remaining_show_entries(record, tags_present)
-        values + series_880_fields(record)
+        series_values = values + series_880_fields(record)
+        series_values.uniq
       end
 
       # Values from series fields for display.
@@ -75,7 +76,7 @@ module PennMARC
 
           filtered_values.map { |v| v.gsub(/\(|\)/, '') }.join(' ')
         end
-        values
+        values.uniq
       end
 
       # Information concerning the immediate predecessor of the target item (chronological relationship). When a note
@@ -83,9 +84,9 @@ module PennMARC
       # indicator position for display.
       # https://www.loc.gov/marc/bibliographic/bd780.html
       # @param [MARC::Record] record
-      # @return [String] continues fields string
+      # @return [Array<String>] continues fields string
       def get_continues_show(record)
-        continues(record, '780')
+        continues(record, '780').uniq
       end
 
       # Information concerning the immediate successor to the target item (chronological relationship). When a note is
@@ -93,9 +94,9 @@ module PennMARC
       # position for display.
       # https://www.loc.gov/marc/bibliographic/bd785.html
       # @param [MARC::Record] record
-      # @return [String] continued by fields string
+      # @return [Array<String>] continued by fields string
       def get_continued_by_show(record)
-        continues(record, '785')
+        continues(record, '785').uniq
       end
 
       private
@@ -127,7 +128,7 @@ module PennMARC
       # @note added 2017/04/10: filter out 0 (authority record numbers) added by Alma
       # @param [MARC::Record] record
       # @param [String] first_tag
-      # @return [Array<Hash>] array of author show entry hashes
+      # @return [Array<String>] array of author show entry strings
       def title_show_entries(record, first_tag)
         record.fields(first_tag).map do |field|
           series = join_subfields(field, &subfield_not_in?(%w[0 5 6 8 c e w v n]))
@@ -156,6 +157,7 @@ module PennMARC
       # links that field to the 880 field. The data in field 880 may be in more than one script. This function exists
       # because it differs than the tradition use of linked_alternate.
       # @param [MARC::Record] record
+      # @return [Array<String>]
       def series_880_fields(record)
         record.fields('880').filter_map do |field|
           next unless subfield_value?(field, '6', /^(800|810|811|830|400|410|411|440|490)/)
@@ -189,7 +191,7 @@ module PennMARC
       # Get subfields from a given field (continues or continued_by).
       # @param [MARC::Record] record
       # @param [String] tag
-      # @return [String] joined subfields
+      # @return [Array<String>] joined subfields
       def continues(record, tag)
         record.fields.filter_map do |field|
           next unless field.tag == tag || (field.tag == '880' && subfield_value?(field, '6', /^#{tag}/))