diff options
| author | Dana Sherson <robot@dana.sh> | 2019-06-05 17:25:18 +1000 |
|---|---|---|
| committer | Dana Sherson <robot@dana.sh> | 2019-06-05 18:18:17 +1000 |
| commit | c8847c17a7c9ae75c44c522c56ccd9c5fca25ea7 (patch) | |
| tree | 057901f401f10fb98615b9bca73c88d110c13fba /activesupport/CHANGELOG.md | |
| parent | ee10e05af27facf42b1c49f8aaa0463d3af6989c (diff) | |
| download | rails-c8847c17a7c9ae75c44c522c56ccd9c5fca25ea7.tar.gz rails-c8847c17a7c9ae75c44c522c56ccd9c5fca25ea7.tar.bz2 rails-c8847c17a7c9ae75c44c522c56ccd9c5fca25ea7.zip | |
Add compact_blank shortcut for reject(&:blank?)
I frequently find myself having to .compact but for blank. which means
on an array reject(&:blank?) (this is fine), or,
on a hash `.reject { |_k, v| v.blank? }` which is slightly more
frustrating and i usually write it as .reject(&:blank?) first and am
confused when it's trying to check if the keys are blank.
I've added the analagous .compact_blank! where there's a reject! to
build on (there's also a reject! in Set, but there's no other core_ext
touching Set so i've left that alone)
Diffstat (limited to 'activesupport/CHANGELOG.md')
| -rw-r--r-- | activesupport/CHANGELOG.md | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/activesupport/CHANGELOG.md b/activesupport/CHANGELOG.md index 955eb7eef9..6256fb8c86 100644 --- a/activesupport/CHANGELOG.md +++ b/activesupport/CHANGELOG.md @@ -1,3 +1,8 @@ +* Add `compact_blank` for those times when you want to remove #blank? values from + an Enumerable (also `compact_blank!` on Hash, Array, ActionController::Parameters) + + *Dana Sherson* + * `delegate_missing_to` would raise a `DelegationError` if the object delegated to was `nil`. Now the `allow_nil` option has been added to enable the user to specify they want `nil` returned in this case. |