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

94.78

/app/controllers/orangelight/browsables_controller.rb

# frozen_string_literal: true

class Orangelight::BrowsablesController < ApplicationController
  # GET /orangelight/names
  # GET /orangelight/names.json

  def index
    # if rpp isn't specified default is 50
    # if rpp has values other than 10, 25, 50, 100 then set it to 50
    # previous/next page links need to pass
    # manually set rpp

    assign_values

    # makes sure valid page is displayed
    if !((@last_id - @rpp + 1)..@last_id).cover?(@start) && @is_last
      @start = @last_id - @rpp + 1
      @start = 1 if @start < 1 # catch for start ids higher than last id
    end

    assign_pagination_values

    @facet = facet
    @list_name = list_name

    respond_to do |format|
      format.html # index.html.erb
      format.json { render json: @orangelight_browsables }
    end
  end

  private

    def assign_values
      @model = model_table_name
      @rpp = rpp
      @page_link = page_link

      # @start gets the id of the first entry to display on page
      # specific ids are given based on search results
      @start = first_model_id

      populate_search_results

      # gets last page of table's results
      @last_id = last_model_id

      # makes sure no next page link is shown for last page
      @is_last = (@last_id - @rpp + 1) <= @start
    end

    def rpp
      if rpp_param.nil?
        50
      else
        validate_rpp(requested_rpp)
      end
    end

    def page_link
      if rpp_param.nil?
        '?'
      else
        size = validate_rpp(requested_rpp)
        "?rpp=#{size}&"
      end
    end

    def validate_rpp(size)
      rpp_range = [10, 25, 50, 100]
      if rpp_range.include? size
        size
      else
        50
      end
    end

    def populate_search_results
      return if query_param.nil?
      # 'subject_facet' is the default vocabulary type if there is no vocabulary type specified
      search_result = if vocabulary_search_on_facet == 'subject_facet'
                        model_param.where('sort <= ?', search_term).order('sort').last
                      else
                        model_param.where(sort: search_term).where(vocabulary: vocabulary_search_on_facet).first
                      end

      return if search_result.nil?

      populate_search_params(search_result)
    end

    def populate_search_params(search_result)
      @search_result = search_result.label
      @search_term = search_term
      @exact_match = search_term == search_result.sort
      @match = search_result.id
      @match_vocabulary = search_result.vocabulary if @model == 'subjects'
      @start = search_result.id - 1
      @start -= 1 if @exact_match
      @start = 1 if @start < 1
      @query = query_param
    end

    def assign_pagination_values
      @is_first = @start == 1

      @page_last = if (@start + @rpp - 1) > @last_id
                     @last_id
                   else
                     @start + @rpp - 1
                   end

      @prev = @start - @rpp
      @prev = 1 if @prev < 1
      @next = @start + @rpp

      @orangelight_browsables = model_param.where(id: @start..@page_last)
    end

    def facet
      if browsing_names?
        'author_s'
      elsif browsing_subjects?
        vocab_param
      elsif browsing_titles?
        'name_title_browse_s'
      end
    end

    # Retrieve the Model mapped to the request parameter
    # @return [Class]
    def model_param
      params[:model]
    end

    def vocab_param
      params[:vocab]
    end

    def vocabulary_search_on_facet
      # TODO: move vocab_types to a config file
      # the hash also exists in vocab_type method
      vocab_types = {
        'Library of Congress subject heading' => 'lc_subject_facet',
        'Library of Congress genre/form terms for library and archival materials' => 'lcgft_genre_facet',
        'Art & architecture thesaurus' => 'aat_genre_facet',
        'Homosaurus terms' => 'homoit_subject_facet',
        'Homosaurus genres' => 'homoit_genre_facet',
        'Rare books genre term' => 'rbgenr_genre_facet',
        'Chinese traditional subjects' => 'siku_subject_facet',
        'Locally assigned term' => 'local_subject_facet'
      }
      vocab_types_invert = vocab_types.invert
      vocab_types_invert.fetch(vocab_param, 'subject_facet')
    end

    # Generates the name of the table mapped to the model in the request
    # @return [String]
    def model_table_name
      model_name = model_param.name
      model_class = model_name.demodulize
      model_class.tableize
    end

    # Determines whether or not the client is requesting to browse call numbers
    # @return [Boolean]
    def browsing_call_numbers?
      model_table_name == 'call_numbers'
    end

    # Determines whether or not the client is requesting to browse names
    # @return [Boolean]
    def browsing_names?
      model_table_name == 'names'
    end

    # Determines whether or not the client is requesting to browse subjects
    # @return [Boolean]
    def browsing_subjects?
      model_table_name == 'subjects'
    end

    # Determines whether or not the client is requesting to browse titles
    # @return [Boolean]
    def browsing_titles?
      model_table_name == 'name_titles'
    end

    # Generates the name of the list (based upon the model for the request)
    # @return [String]
    def list_name
      value = model_table_name.humanize
      return 'author-title headings' if value == 'Name titles'
      value
    end

    # Retrieves the ID requested by the client
    # @return [String]
    def id_param
      params[:id]
    end

    # Retrieves the query transmitted by the client
    # @return [String]
    def query_param
      params[:q]
    end

    # Retrieves the ID for the first object requested by the client
    # @return [String]
    def start_param
      params[:start]
    end

    # Retrieves the ID of the first object for this request
    # @return [Integer]
    def first_model_id
      return 1 if start_param.nil?
      start_param.to_i
    end

    # Normalizes the query transmitted by the client
    # @return [String]
    def search_term
      return StringFunctions.cn_normalize(query_param) if browsing_call_numbers?

      query_param.normalize_em
    end

    # Retrieves the ID of the last object for this request
    # @return [Integer]
    def last_model_id
      return model_param.last.id if model_param.last
      1
    end

    # Retrieves the requested rows per page (rpp) from the client
    # @return [String]
    def rpp_param
      params[:rpp]
    end

    # Generates the requested rows per page as an Integer
    # @return [Integer]
    def requested_rpp
      rpp_param.to_i
    end

    # Use callbacks to share common setup or constraints between actions.
    def set_orangelight_browsable
      @orangelight_browsable = model_param.find(id_param) if model_param
    end

    # Never trust parameters from the scary internet; run the params we receive through an allowlist
    def orangelight_browsable_params
      params.require(:orangelight_browsable).permit(:model, :id)
    end
end

1	# frozen_string_literal: true
2
3	class Orangelight::BrowsablesController < ApplicationController	1✔
4	# GET /orangelight/names
5	# GET /orangelight/names.json
6
7	def index	1✔
8	# if rpp isn't specified default is 50
9	# if rpp has values other than 10, 25, 50, 100 then set it to 50
10	# previous/next page links need to pass
11	# manually set rpp
12
13	assign_values	23✔
14
15	# makes sure valid page is displayed
16	if !((@last_id - @rpp + 1)..@last_id).cover?(@start) && @is_last	23✔
17	@start = @last_id - @rpp + 1	8✔
18	@start = 1 if @start < 1 # catch for start ids higher than last id	8✔
19	end
20
21	assign_pagination_values	23✔
22
23	@facet = facet	23✔
24	@list_name = list_name	23✔
25
26	respond_to do \|format\|	23✔
27	format.html # index.html.erb	23✔
28	format.json { render json: @orangelight_browsables }	41✔
29	end
30	end
31
32	private	1✔
33
34	def assign_values	1✔
35	@model = model_table_name	23✔
36	@rpp = rpp	23✔
37	@page_link = page_link	23✔
38
39	# @start gets the id of the first entry to display on page
40	# specific ids are given based on search results
41	@start = first_model_id	23✔
42
43	populate_search_results	23✔
44
45	# gets last page of table's results
46	@last_id = last_model_id	23✔
47
48	# makes sure no next page link is shown for last page
49	@is_last = (@last_id - @rpp + 1) <= @start	23✔
50	end
51
52	def rpp	1✔
53	if rpp_param.nil?	23✔
54	50	10✔
55	else
56	validate_rpp(requested_rpp)	13✔
57	end
58	end
59
60	def page_link	1✔
61	if rpp_param.nil?	23✔
62	'?'	10✔
63	else
64	size = validate_rpp(requested_rpp)	13✔
65	"?rpp=#{size}&"	13✔
66	end
67	end
68
69	def validate_rpp(size)	1✔
70	rpp_range = [10, 25, 50, 100]	26✔
71	if rpp_range.include? size	26✔
72	size	18✔
73	else
74	50	8✔
75	end
76	end
77
78	def populate_search_results	1✔
79	return if query_param.nil?	23✔
80	# 'subject_facet' is the default vocabulary type if there is no vocabulary type specified
81	search_result = if vocabulary_search_on_facet == 'subject_facet'	7✔
82	model_param.where('sort <= ?', search_term).order('sort').last	7✔
83	else
84	model_param.where(sort: search_term).where(vocabulary: vocabulary_search_on_facet).first	×
85	end
86
87	return if search_result.nil?	7✔
88
89	populate_search_params(search_result)	6✔
90	end
91
92	def populate_search_params(search_result)	1✔
93	@search_result = search_result.label	6✔
94	@search_term = search_term	6✔
95	@exact_match = search_term == search_result.sort	6✔
96	@match = search_result.id	6✔
97	@match_vocabulary = search_result.vocabulary if @model == 'subjects'	6✔
98	@start = search_result.id - 1	6✔
99	@start -= 1 if @exact_match	6✔
100	@start = 1 if @start < 1	6✔
101	@query = query_param	6✔
102	end
103
104	def assign_pagination_values	1✔
105	@is_first = @start == 1	23✔
106
107	@page_last = if (@start + @rpp - 1) > @last_id	23✔
108	@last_id	3✔
109	else
110	@start + @rpp - 1	20✔
111	end
112
113	@prev = @start - @rpp	23✔
114	@prev = 1 if @prev < 1	23✔
115	@next = @start + @rpp	23✔
116
117	@orangelight_browsables = model_param.where(id: @start..@page_last)	23✔
118	end
119
120	def facet	1✔
121	if browsing_names?	23✔
122	'author_s'	1✔
123	elsif browsing_subjects?	22✔
124	vocab_param	4✔
125	elsif browsing_titles?	18✔
126	'name_title_browse_s'	1✔
127	end
128	end
129
130	# Retrieve the Model mapped to the request parameter
131	# @return [Class]
132	def model_param	1✔
133	params[:model]	204✔
134	end
135
136	def vocab_param	1✔
137	params[:vocab]	11✔
138	end
139
140	def vocabulary_search_on_facet	1✔
141	# TODO: move vocab_types to a config file
142	# the hash also exists in vocab_type method
143	vocab_types = {	7✔
144	'Library of Congress subject heading' => 'lc_subject_facet',
145	'Library of Congress genre/form terms for library and archival materials' => 'lcgft_genre_facet',
146	'Art & architecture thesaurus' => 'aat_genre_facet',
147	'Homosaurus terms' => 'homoit_subject_facet',
148	'Homosaurus genres' => 'homoit_genre_facet',
149	'Rare books genre term' => 'rbgenr_genre_facet',
150	'Chinese traditional subjects' => 'siku_subject_facet',
151	'Locally assigned term' => 'local_subject_facet'
152	}
153	vocab_types_invert = vocab_types.invert	7✔
154	vocab_types_invert.fetch(vocab_param, 'subject_facet')	7✔
155	end
156
157	# Generates the name of the table mapped to the model in the request
158	# @return [String]
159	def model_table_name	1✔
160	model_name = model_param.name	128✔
161	model_class = model_name.demodulize	128✔
162	model_class.tableize	128✔
163	end
164
165	# Determines whether or not the client is requesting to browse call numbers
166	# @return [Boolean]
167	def browsing_call_numbers?	1✔
168	model_table_name == 'call_numbers'	19✔
169	end
170
171	# Determines whether or not the client is requesting to browse names
172	# @return [Boolean]
173	def browsing_names?	1✔
174	model_table_name == 'names'	23✔
175	end
176
177	# Determines whether or not the client is requesting to browse subjects
178	# @return [Boolean]
179	def browsing_subjects?	1✔
180	model_table_name == 'subjects'	22✔
181	end
182
183	# Determines whether or not the client is requesting to browse titles
184	# @return [Boolean]
185	def browsing_titles?	1✔
186	model_table_name == 'name_titles'	18✔
187	end
188
189	# Generates the name of the list (based upon the model for the request)
190	# @return [String]
191	def list_name	1✔
192	value = model_table_name.humanize	23✔
193	return 'author-title headings' if value == 'Name titles'	23✔
194	value	22✔
195	end
196
197	# Retrieves the ID requested by the client
198	# @return [String]
199	def id_param	1✔
200	params[:id]	×
201	end
202
203	# Retrieves the query transmitted by the client
204	# @return [String]
205	def query_param	1✔
206	params[:q]	48✔
207	end
208
209	# Retrieves the ID for the first object requested by the client
210	# @return [String]
211	def start_param	1✔
212	params[:start]	37✔
213	end
214
215	# Retrieves the ID of the first object for this request
216	# @return [Integer]
217	def first_model_id	1✔
218	return 1 if start_param.nil?	23✔
219	start_param.to_i	14✔
220	end
221
222	# Normalizes the query transmitted by the client
223	# @return [String]
224	def search_term	1✔
225	return StringFunctions.cn_normalize(query_param) if browsing_call_numbers?	19✔
226
227	query_param.normalize_em	×
228	end
229
230	# Retrieves the ID of the last object for this request
231	# @return [Integer]
232	def last_model_id	1✔
233	return model_param.last.id if model_param.last	23✔
234	1	×
235	end
236
237	# Retrieves the requested rows per page (rpp) from the client
238	# @return [String]
239	def rpp_param	1✔
240	params[:rpp]	72✔
241	end
242
243	# Generates the requested rows per page as an Integer
244	# @return [Integer]
245	def requested_rpp	1✔
246	rpp_param.to_i	26✔
247	end
248
249	# Use callbacks to share common setup or constraints between actions.
250	def set_orangelight_browsable	1✔
251	@orangelight_browsable = model_param.find(id_param) if model_param	×
252	end
253
254	# Never trust parameters from the scary internet; run the params we receive through an allowlist
255	def orangelight_browsable_params	1✔
256	params.require(:orangelight_browsable).permit(:model, :id)	×
257	end
258	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