Personally, i feel like this is cleaner and easier to change if we ever need to. Calling a method keeps us from violating DRY in our views, especially if we ever need to change the group membership logic.

That said, you need to document you inputs and returns.