This should only be done `if(shouldReplaceAttachments())`, to avoid the API call if it will not be used (they will only be used at line 236).  Perhaps a lazy-load method to retrieve the attachments on first-use if they have not already been retrieved?