pass user_id to mailer instead of user object
move application helpers to where they belong
use scopes in models