Revert bp_group_is_visible() to check access.

In r11761, `bp_group_is_visible()` was changed to rely on the `BP_Groups_Group->is_visible` property. However, `bp_group_is_visible()` has been used to check access, not visibility, making the new logic, even though it seems sensible, wrong.

This changeset also replaces `bp_group_is_visible()` in the BP Nouveau template parts, to start retiring this no-longer-correctly-named function.

Props r-a-y.

See #7608.