diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index cb61749082..3e96ce25b2 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -649,12 +649,6 @@ Style/RedundantBegin: Exclude: - 'lib/blacklight/configuration/fields.rb' -# Offense count: 3 -# Cop supports --auto-correct. -Style/RedundantFreeze: - Exclude: - - 'app/helpers/blacklight/render_partials_helper_behavior.rb' - # Offense count: 5 # Cop supports --auto-correct. # Configuration parameters: EnforcedStyle, AllowInnerSlashes. diff --git a/app/helpers/blacklight/render_partials_helper_behavior.rb b/app/helpers/blacklight/render_partials_helper_behavior.rb index 6c85effd62..460925e4ec 100644 --- a/app/helpers/blacklight/render_partials_helper_behavior.rb +++ b/app/helpers/blacklight/render_partials_helper_behavior.rb @@ -132,8 +132,8 @@ def document_partial_name(document, base_name = nil) def type_field_to_partial_name(_document, display_type) # using "_" as sep. to more closely follow the views file naming conventions # parameterize uses "-" as the default sep. which throws errors - underscore = '_'.freeze - Array(display_type).join(' '.freeze).tr('-'.freeze, underscore).parameterize(separator: underscore) + underscore = '_' + Array(display_type).join(' ').tr('-', underscore).parameterize(separator: underscore) end ##