I'd refactor this so that current_user returns a user object (versus an integer id). I'd do this because that's generally conventional