Prefer delegated Project#member? over Project.team#member?

What does this MR do and why?

This MR refactors production code (no specs have been changed/harmed) to use project.member? instead of project.team.member? because member? is delegated.

See #408201 (closed)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

• I have evaluated the MR acceptance checklist for this MR.

Related to #408201 (closed)