not sure it matters but reverting the order of these might be more effective : 

```ruby
group.privacy == 'public' || group.members.include?(user) 
```

because it returns true if the first one evaluates as true and doesn't check the second one I think