To me, this seems better:

```ruby
product_ids = Product.readable(current_organization).where(:id => product_ids).pluck(:id)
```

Than this:

```ruby
product_ids = Product.readable(current_organization).where(:id => product_ids).select(:id).collect(&:id)
```

But that's my opinion.