diff options
author | Matt Jankowski <mjankowski@thoughtbot.com> | 2017-05-19 05:43:20 -0400 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-05-19 11:43:20 +0200 |
commit | 8e4fc5d5d2e5152df35e0486f13356ec76d590f3 (patch) | |
tree | 2fd197191a98f0fe65ddd32e995726aad98df2b7 /app/models/concerns | |
parent | b8b7b506a27ddc1373032078a4f1cfd47242627a (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