Ok, that's fine.

The try(:id) is essentially unrelated to this concern, as the try_spree_current_user would've passed a nil into the mailer method before this change, which would've caused it to blow up at the mailer itself.

As this is a method in an admin controller, the invocation is already guarded by filters.  So I think we can just remove the try(:id) and replace it with an id call, and the controller itself will ensure the user is properly authenticated.