diff --git a/app/models/filter/params.rb b/app/models/filter/params.rb index 69cd07bb4..8938840eb 100644 --- a/app/models/filter/params.rb +++ b/app/models/filter/params.rb @@ -25,10 +25,10 @@ module Filter::Params end end - def params_without(name, value) + def params_without(key, value) to_params.tap do |params| - params[name].delete(value) if params[name].is_a?(Array) - params.delete(name) if params[name] == value + params[key].delete(value) if params[key].is_a?(Array) + params.delete(key) if params[key] == value end end end diff --git a/test/models/filter_test.rb b/test/models/filter_test.rb index 49d7c3167..4933f003b 100644 --- a/test/models/filter_test.rb +++ b/test/models/filter_test.rb @@ -100,7 +100,7 @@ class FilterTest < ActiveSupport::TestCase assert_equal "Most discussed, tagged #Mobile, and assigned to JZ in all projects", filters(:jz_assignments).plain_summary end - test "params without a name-value pair" do + test "params without a key-value pair" do filter = users(:david).filters.new indexed_by: "most_discussed", assignee_ids: [ users(:jz).id, users(:kevin).id ] expected = { indexed_by: "most_discussed", assignee_ids: [ users(:kevin).id ] }