<commit_before># frozen_string_literal: true

module Admin::FilterHelper
  ACCOUNT_FILTERS = %i(local remote by_domain silenced suspended recent username display_name email ip).freeze
  REPORT_FILTERS = %i(resolved account_id target_account_id).freeze

  FILTERS = ACCOUNT_FILTERS + REPORT_FILTERS

  def filter_link_to(text, more_params)
    new_url = filtered_url_for(more_params)
    link_to text, new_url, class: filter_link_class(new_url)
  end

  def table_link_to(icon, text, path, options = {})
    link_to safe_join([fa_icon(icon), text]), path, options.merge(class: 'table-action-link')
  end

  private

  def filter_params(more_params)
    params.permit(FILTERS).merge(more_params)
  end

  def filter_link_class(new_url)
    filtered_url_for(params) == new_url ? 'selected' : ''
  end

  def filtered_url_for(params)
    url_for filter_params(params)
  end
end<commit_msg>Improve how params from controller are permitted in filter helper (#3129)<commit_after># frozen_string_literal: true

module Admin::FilterHelper
  ACCOUNT_FILTERS = %i(local remote by_domain silenced suspended recent username display_name email ip).freeze
  REPORT_FILTERS = %i(resolved account_id target_account_id).freeze

  FILTERS = ACCOUNT_FILTERS + REPORT_FILTERS

  def filter_link_to(text, more_params)
    new_url = filtered_url_for(more_params)
    link_to text, new_url, class: filter_link_class(new_url)
  end

  def table_link_to(icon, text, path, options = {})
    link_to safe_join([fa_icon(icon), text]), path, options.merge(class: 'table-action-link')
  end

  private

  def filter_params(more_params)
    controller_request_params.merge(more_params)
  end

  def filter_link_class(new_url)
    filtered_url_for(controller_request_params) == new_url ? 'selected' : ''
  end

  def filtered_url_for(url_params)
    url_for filter_params(url_params)
  end

  def controller_request_params
    params.permit(FILTERS)
  end
end
{'fraction_non_alphanumeric': 0.08359293873312565, 'fraction_numerical': 0.0020768431983385254, 'mean_word_length': 5.338815789473684, 'pattern_counts': {'":': 0, '<': 3, '<?xml version=': 0, '>': 3, 'https://': 0, 'lorem ipsum': 0, 'www.': 0, 'xml': 0}, 'pii_count': 0, 'substrings_counts': 0, 'word_list_counts': {'cursed_substrings.json': 0, 'profanity_word_list.json': 0, 'sexual_word_list.json': 0, 'zh_pornsignals.json': 0}}
{'dir': 'git-commits-cleaned', 'id': '7569107', 'n_tokens_mistral': 659, 'n_tokens_neox': 634, 'n_words': 139}