Refactor and ensure all waffle flags to include a request object

[zandercymatics]

Issue description

A few of our flag_is_active checks are passing in "None" for the underlying HttpRequest object. This is because the request isn't accessible in the context that it is being used in.

However, doing so breaks some functionality. All instances of these should be refactored such that we are now passing in a HttpRequest object

Acceptance criteria

• All checks using flag_is_active for "disable_email_sending" pass in a request object
• Double check that all of our checks for "organization_feature" pass in a request object
• Double check that all of our checks for "multiple_portfolios" pass in a request object

Additional context

One approach to this may be to create a bogus request and pass it to the user context. However, this would need to be tested thoroughly.

request = HttpRequest()
request.user = self.user
flag_is_active(request, "profile_feature")

An alternative would be to write a custom function in WaffleFlags that essentially does what flag_is_active does, but it takes the user and the flag name rather than the request and the flag name.

If you decide to go this route, remember to also refactor the other bogus requests created for our organization_feature flag

Links to other issues

• blocked by Add new waffle flags and use them for showing features #2667 (so we aren't messing with multiple waffle flag refactors at the same time)