hyperstack-org / hyperstack-org/hyperstack

for policy white and black lists ignore nils, and flatten arrays

Đang mở
#319 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
enhancement good first issue
Ngôn ngữ chính
JavaScript
Star
538
Fork
41
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

for example
`policy.send_all_but(nil).to(Application)`
should work.

It doesn't because it tries to convert nil to a symbol and fails.

This is handy for a case like this:

`policy.send_all_but(production_job? && :ship_by_date).to(Application)`

Otherwise the code looks like this:

```ruby
if production_job?
policy.send_all_but(:ship_by_date).to(Application)
else
policy.send_all.to(Application)
end

# or

policy.send_all_but(*[*(production_job? && :ship_by_date)]).to(Application)
```

Allowing an array of symbols to be passed in might also be handy in cases as well so basically we want to flatten and compress the parameters (which is already a splat) to `send_all_but` and `send_only`. We use `reject(&:!)` instead of `compact` so that any falsy value will be removed (false as well as nil.)

```ruby
module Hyperstack
class InternalPolicy
def send_all_but(*exceptions)
SendSet.new(self, exclude: exceptions.flatten.reject(&:!))
end

def send_only(*white_list)
SendSet.new(self, white_list: white_list.flatten.reject(&:!))
end
end
end
```

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.