Unfloyded/Legamunt
0
0
Fork 0
Code Activity

Add relevant_params to ReportFilter (matches account filter) (#32136)

Browse source
This commit is contained in:
Matt Jankowski 2024-10-02 09:24:40 -04:00 • committed by GitHub
parent cf859d151b
commit 2151dfb8d6
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 26 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

14
app/models/report_filter.rb
View file

@ -18,13 +18,25 @@ class ReportFilter
def results
scope = Report.unresolved
params.each do |key, value|
relevant_params.each do |key, value|
scope = scope.merge scope_for(key, value)
end
scope
end
private
def relevant_params
params.tap do |args|
args.delete(:target_origin) if origin_is_remote_and_domain_present?
end
end
def origin_is_remote_and_domain_present?
params[:target_origin] == 'remote' && params[:by_target_domain].present?
end
def scope_for(key, value)
case key.to_sym
when :by_target_domain