4c391e0e-519a-40cb-8ad3-354445f4ce03

Committed 12 Aug 2025 08:47PM UTC coverage: 85.348% (-10.0%) from 95.335%

Build # 4c391e0e-519a-40cb-8ad3-354445f4ce03

Build Type

push

circleci

Committed by

web-flow

Commit Message

[#5143] Use access restriction note as Aeon ItemInfo1 if available (#5173)

With this commit, if a user visits a record with an access
restrictions note and presses the Reading Room Request
button, they will get to an Aeon form with the 'Restrictions'
field pre-filled with the restriction note.

If the record does not have an access restrictions note,
the field will be pre-filled with 'Reading Room Access Only',
as it has been previously.

Closes #5143

Run Details

5493 of 6436 relevant lines covered (85.35%)

251.82 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

42.86

/app/components/holdings/plain_text_component.rb

# frozen_string_literal: true

# A View Component that is responsible for formatting a record's holdings
# (including bound-with holdings) in a way suitable for plain-text contexts,
# like a plain-text email.
class Holdings::PlainTextComponent < ViewComponent::Base
  def initialize(document)
    @document = document
  end

  def call
    return if formatted_holdings_data.blank?
    "Holdings:\n#{formatted_holdings_data}"
  end

  private

    # :reek:DuplicateMethodCall
    # :reek:FeatureEnvy
    def formatted_holdings_data
      @formatted_holdings_data ||= document.holdings_all_display&.values&.map do |holdings_data|
        location_statement = "\n\tLocation: #{holdings_data['library']}"
        location_statement << " - #{holdings_data['location']}" if holdings_data['location']
        location_statement << "\n\tCall number: #{holdings_data['call_number']}" if holdings_data['call_number']
        location_statement
      end&.join("\n")
    end
    attr_reader :document
end

1	# frozen_string_literal: true
2
3	# A View Component that is responsible for formatting a record's holdings
4	# (including bound-with holdings) in a way suitable for plain-text contexts,
5	# like a plain-text email.
6	class Holdings::PlainTextComponent < ViewComponent::Base	1✔
7	def initialize(document)	1✔
8	@document = document	×
9	end
10
11	def call	1✔
12	return if formatted_holdings_data.blank?	×
13	"Holdings:\n#{formatted_holdings_data}"	×
14	end
15
16	private	1✔
17
18	# :reek:DuplicateMethodCall
19	# :reek:FeatureEnvy
20	def formatted_holdings_data	1✔
21	@formatted_holdings_data \|\|= document.holdings_all_display&.values&.map do \|holdings_data\|	×
22	location_statement = "\n\tLocation: #{holdings_data['library']}"	×
23	location_statement << " - #{holdings_data['location']}" if holdings_data['location']	×
24	location_statement << "\n\tCall number: #{holdings_data['call_number']}" if holdings_data['call_number']	×
25	location_statement	×
26	end&.join("\n")
27	end
28	attr_reader :document	1✔
29	end

pulibrary / orangelight / 4c391e0e-519a-40cb-8ad3-354445f4ce03

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous