Use public_send rather than send

Author: jcoyne

app/factories/blacklight_field_configuration_factory.rb

@@ -17,7 +17,7 @@ def initialize(config:, field:, field_group:)
   def field_config
     return null_field unless config.respond_to?(:"#{field_group}s")
 
-    config.send(:"#{field_group}s").fetch(field) { null_field }
+    config.public_send(:"#{field_group}s").fetch(field) { null_field }
   end
 
   private

app/models/arclight/document_downloads.rb

@@ -74,7 +74,7 @@ def href
       def size
         return data['size'] if data['size']
 
-        document.send(data['size_accessor'].to_sym) if data['size_accessor']
+        document.public_send(data['size_accessor'].to_sym) if data['size_accessor']
       end
 
       private
@@ -105,7 +105,7 @@ def template_variables
 
       def escaped_document_interpolations
         non_custom_template_variables.index_with do |method_name|
-          escape_non_url_doc_value(document.send(method_name))
+          escape_non_url_doc_value(document.public_send(method_name))
         end
       end
 

app/models/arclight/requests/aeon_external_request.rb

@@ -30,7 +30,7 @@ def static_mappings
 
       def dynamic_mappings
         config['request_mappings']['accessor'].transform_values do |v|
-          @document.send(v.to_sym)
+          @document.public_send(v.to_sym)
         end
       end
 

app/views/arclight/repositories/_in_person_repository.html.erb

@@ -9,9 +9,9 @@
 <div class='al-in-person-repository-location'>
   <address>
     <% %i[building address1 address2 city_state_zip_country].each do |f| %>
-      <% if repository.send(f).present? %>
+      <% if repository.public_send(f).present? %>
         <div class='al-repository-contact-<%= f %>'>
-          <%= repository.send(f) %>
+          <%= repository.public_send(f) %>
         </div>
       <% end %>
     <% end %>

app/views/arclight/repositories/_repository.html.erb

@@ -14,9 +14,9 @@
             <address>
               <div class="al-repository-street-address">
                 <% %i[building address1 address2 city_state_zip_country].each do |f| %>
-                  <% if repository.send(f).present? %>
+                  <% if repository.public_send(f).present? %>
                     <div class='al-repository-street-address-<%= f %>'>
-                      <%= repository.send(f) %>
+                      <%= repository.public_send(f) %>
                     </div>
                   <% end %>
                 <% end %>

app/views/arclight/repositories/_repository_contact.html.erb

@@ -1,8 +1,8 @@
 <div class='al-in-person-repository-contact'>
   <% %i[phone contact_info].each do |f| %>
-    <% if repository.send(f).present? %>
+    <% if repository.public_send(f).present? %>
       <div class='al-repository-contact-<%= f %>'>
-        <%= repository.send(f) %>
+        <%= repository.public_send(f) %>
       </div>
     <% end %>
   <% end %>

app/views/arclight/requests/_google_form.html.erb

@@ -3,7 +3,7 @@
 
 <%= form_tag google_form.url, method: :get, class: 'al-request-form' do |f| %>
   <% google_form.form_mapping.each do |key, value| %>
-    <%= hidden_field_tag value, google_form.send(key) %>
+    <%= hidden_field_tag value, google_form.public_send(key) %>
   <% end %>
   <!-- Note: previous button class was 'al-request-button btn-link' -->
   <button type='submit' class='btn btn-secondary btn-sm'>