If we delegated subscribed_to_proposal_closure_notifications? from user to email_preferences then we would not need to change this throughout the codebase. 

something like
class User
  ..
  delegate :subscribed_to_proposal_closure_notifications? to: :email_preferences, prefix: false
  ..
end