about summary refs log tree commit diff
path: root/app/models/concerns
diff options
context:
space:
mode:
authorMatt Jankowski <mjankowski@thoughtbot.com>2017-05-19 05:43:20 -0400
committerEugen Rochko <eugen@zeonfederated.com>2017-05-19 11:43:20 +0200
commit8e4fc5d5d2e5152df35e0486f13356ec76d590f3 (patch)
tree2fd197191a98f0fe65ddd32e995726aad98df2b7 /app/models/concerns
parentb8b7b506a27ddc1373032078a4f1cfd47242627a (diff)
Improve how params from controller are permitted in filter helper (#3129)
The `params` variable here was quite overloaded.

It exists via the controller to hold the request params, and was sometimes being
used in this helper as that object, but other times was being used as a local
variable, or to pass to another method, and this was confusing.

This change renames the args for a method away from `params` for more clarity,
and extracts the actual usage of the controller-provided `params` to a
better-named method for clarity.
Diffstat (limited to 'app/models/concerns')
0 files changed, 0 insertions, 0 deletions