should this be checking membership instead of ownership?

`if not @group.memberships.exists?(:user_id => current_user.id)`