fix rubocop offenses

Author: scambra

lib/active_scaffold/actions/field_search.rb

@@ -206,14 +206,16 @@ def field_search_ignore?
       def setup_joins_for_filtered_columns(filtered_columns)
         if grouped_search? || active_scaffold_config.list.user.count_includes.present?
           active_scaffold_outer_joins.concat filtered_columns.map(&:search_joins).flatten.uniq.compact
-          if grouped_search? && search_group_column&.search_joins.present? && filtered_columns.exclude?(search_group_column)
-            active_scaffold_joins << search_group_column.search_joins
-          end
+          active_scaffold_joins << search_group_column.search_joins if grouped_column_needs_joins?(filtered_columns)
         else
           set_outer_joins_for_search filtered_columns
         end
       end
 
+      def grouped_column_needs_joins?(filtered_columns)
+        grouped_search? && search_group_column&.search_joins.present? && filtered_columns.exclude?(search_group_column)
+      end
+
       def field_search_formats
         (default_formats + active_scaffold_config.formats + active_scaffold_config.field_search.formats).uniq
       end

lib/active_scaffold/finder.rb

@@ -517,12 +517,12 @@ def active_scaffold_joins
     end
 
     def active_scaffold_habtm_joins
-      ActiveScaffold.deprecator.warn "use active_scaffold_joins"
+      ActiveScaffold.deprecator.warn 'use active_scaffold_joins'
       active_scaffold_joins
     end
 
     def active_scaffold_habtm_joins=(value)
-      ActiveScaffold.deprecator.warn "use active_scaffold_joins="
+      ActiveScaffold.deprecator.warn 'use active_scaffold_joins='
       self.active_scaffold_joins = value
     end