Refactoring to not have so many diff services accessing the request context to get at the current active site

Instead, they can just use a method on the SiteService object, which makes more sense API wise